对char数组的访问冲突

Question

我在刚创建的char数组上遇到访问冲突new.

DispatchCommand(char* cmdStr)
        {
            // Dispatch
            for(int i = 0; i < sizeof(_lpCommands); i++)
            {
                const int len = strlen(_lpCommands[i].szCommand);
                char* cmdblip = new char[len + 1];
                memcpy(&cmdblip, cmdStr, len);
                cmdblip[len] = '\0';  // Access Violation

                if(strcmp(cmdblip, _lpCommands[i].szCommand) == 0)
                {
                    if(strlen(cmdStr) > strlen(_lpCommands[i].szCommand))
                        (*_lpCommands[i].cbCallback)(&cmdStr[strlen(_lpCommands[i].szCommand)]);
                    else
                        (*_lpCommands[i].cbCallback)("");

                    delete cmdblip;
                    return;
                }

                delete cmdblip;
            }

            // Error and return
            *Out::ServerInfo<<"Command not found!"<<ENDL;
        }

_lpCommands是一个Command结构数组:

struct Command
{
    char* szCommand;
    CommandCallback cbCallback;
};

生成的错误消息是:

Program.exe中0x012219cf处的未处理异常:0xC0000005:访问冲突写入位置0x66647366.

这是对使用的类似代码的重写memcmp,最终给了我一个访问冲突,而不是做了memcpy.

是什么赋予了？

Answer 1

不要传递&cmdblip给memcpy.您应该将指针传递给目标缓冲区,而不是指向该指针的指针.通过cmdblip代替.

编辑:我同意一般来说,std :: string应该在C++中使用.此代码崩溃的技术原因仍然是memcpy破坏cmdblip指针,使其指向实际由复制字符串的前4个字节组成的内存位置.然后,cmdblip[len]导致内存位置不在分配的缓冲区(或任何其他合法分配的缓冲区)内,因此崩溃.因此,如果您想编写更好的代码,请使用C++类.如果您想了解给定代码崩溃的原因,请考虑以上内容.

Answer 2

这个问题唯一可能有用的答案是"使用std::string".您现在遇到的具体问题只会在每次修改此函数或写入其他类似函数时重新出现或相同.在一般情况下解决问题的唯一方法是转向基于类的解决方案,这是一种为您提供标准的解决方案.例如,您的当前代码是异常不安全的,除了提供访问冲突的任何内容之外,更不用说它是不可读的并且要求许多其他错误,例如一个一个错误,没有正确的NULL终止,双删除和内存泄漏.哦,和UB,因为你delete是你new[].