char 数组的访问冲突

Question

我刚刚使用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[]。