删除字符串时堆损坏

Question

这是我的代码：

std::string readString()
{
     int strLen = Read<int>();
     char* rawString = new char[strLen];
     Read(rawString, strLen);
     rawString[strLen] = '\0';
     std::string retVal(rawString);
     delete [] rawString;
     return retVal;
 }

第一行读取字符串的长度。
第二行创建一个新的字符数组（c-string），其长度为字符串
第三行读取字符串（从文件中读取）
第 4 行在末尾添加 NULL。
第 5 行从 c 字符串中创建一个 std::string。
第 6 行删除 c 字符串（HEAP CORRUPTION HAPPENS HERE）
第 7 行返回字符串，但由于错误，它永远不会到达这一点。

在第 6 行，我收到堆损坏错误： CRT 检测到应用程序在堆缓冲区末尾后写入内存。

我的问题可能很明显，但为什么我会出现堆损坏？当我创建 std::string 时，它应该复制该字符串，并且我应该可以安全地删除 c 字符串。

目前，我怀疑 std::string 在删除 c 字符串后试图访问它。

有什么想法吗？

Answer 1

您正在访问超过字符串的保留字节。您保留了 strLen 字符，但在字符 strLen 处放置了 \0。从 0 开始计算 C 数组，字符 strLen 位于位置 strLen + 1，因此您将一个值放在字符串的保留空间之外。您应该在 main 的第二行中保留 strLen + 1 以使代码正常工作。

Answer 2

更改：

char* rawString = new char[strLen];

至：

char* rawString = new char[strLen + 1];

Answer 3

int strLen = Read()可能仅返回非空终止字符串的长度，并且当您尝试写入 \0< /code> byte 到字符串，你会遇到缓冲区溢出问题。

您应该检查 strLen 是什么，很可能您必须像这样分配：

char *rawString = new char[strlen+1];

或者使用 std::string(const char *, size_t n) 的重载构造函数像这样：

std::string retVal(rawString, strlen);

Answer 4

由于数组在 C++ 中是从 0 索引的，因此当您创建一个大小为 strLen 的数组，然后在位置 strLen 处放置一个 0 时，您将在您分配的数组。

Answer 5

到目前为止有很多建议，但没有一个解决异常安全问题：如何消除潜在的内存泄漏？

有两种方法可以避免使用 new 进行分配（从而避免内存泄漏）。第一个非常简单，使用称为 VLA 的编译器扩展来表示可变长度数组：

std::string readString()
{
  int strLen = Read<int>();
  char rawString[strLen+1]; // VLA: the length is determined at runtime
                            // but the array is nonetheless on the stack
  Read(rawString, strLen);
  rawString[strLen] = '\0';

  std::string retVal(rawString);
  return retVal;
}

另一个符合标准：string 有一个可以访问的内部缓冲区（感谢 GMan，< code>data 不是正确的访问方法）

std::string readString()
{
  int strLen = Read<int>();

  std::string retVal(strLen, '\0'); // no need to allocate extra space

  Read(&retVal[0], strLen);      // &retVal[0] gives access to the buffer

  return retVal;
}

我确实相信最后一个版本要好得多。不再涉及任何复制:)

Answer 6

 rawString[strLen] = '\0';

将 NUL 从您分配的空间末尾写掉。

如果strLen是10，那么你分配10个字符的空间，读取10个字符，并将这个NUL写入位置11。哎呀