Question

“CRT检测到应用程序写入堆缓冲区的内存末尾”错误。它到达free时会崩溃。任何帮助表示赞赏。

int messageFunction(char* message) {
   char* sPtr = strstr(message,"Subject:");
   char* cPtr = strstr(message,"Content:");

   char* messageSubject = (char*) malloc(cPtr - sPtr - strlen("Subject:"))
   char* messageContent = (char*) malloc(strlen(cPtr + strlen("Content:")))

   strncpy(messageSubject, 
          stPtr + strlen("Subject:"), 
          cPtr - sPtr - strlen("Subject:"));

   messageSubject[cPtr - sPtr - strlen("Subject:")] = '\0';

   strncpy(messageContent, 
           cPtr + strlen("Content:"), 
           strlen(cPtr + strlen("Content:")));
   ...
   free(messageSubject);
   free(messageContent);
   }


void main() {
  char* message = "Subject:HelloWorldContent:MessageContent";
  int result = messageFunction(message);
 }

Answer 1

您正在分配一个字节太短的内存。您的计算是针对例如数据的长度。 “Subject：”和“Content：”但不考虑字符串中对null终止符的需要。然后，当您手动添加空终止符时，您将通过写入数组末尾来调用未定义的行为。

将代码更改为以下内容应该可以解决问题。

char* messageSubject = malloc(cPtr - sPtr - strlen("Subject:") + 1)
char* messageContent = malloc(strlen(cPtr + strlen("Content:")) + 1)

您也不会在“...”部分显示代码，因此您可能在其中有一个未终止的字符串，如果字符串库例程正在处理它可能会导致问题。

Answer 2

如果你这样做：

char* v = malloc(n);

然后v的有效下标范围从v[0]到v[n-1]。特别是， v [n]永远不会有效。这是一般规则。如果再次查看代码，您应该会看到问题。

几点说明：

您的代码假定主题：位于内容之前：并且它们都存在。在某些情况下，这种假设是不正确的。你应该在开始malloc'ing大量内存之前检查（因为小的负数变成巨大的正无符号数）。你还应该确保你的mallocs不会返回0，而不是在他们这样做时发生segfaulting。
strdup（和strndup）通常会让您免于尴尬的“哎呀，我没有为NUL字节分配足够的空间”错误。它们也不需要几乎同样多的东西，使您的代码更简单，更可靠，更容易理解。了解他们。他们将是你的朋友。
如果没有其他工作，valgrind可以帮助您找到这样的错误。

检测到堆腐蚀：正常阻塞后

2 个答案: