2

asprintf我有以下代码在同时使用和时不起作用realloc。

我得到的错误是：

*** glibc detected *** a.out: realloc(): invalid old size: 0x006f1430 ***

根据我的研究，当我使用asprintf它时，它看起来会覆盖一些realloc使用的内存。这对我来说没有意义，因为asprintf它应该是安全的并且使用适当的字符串长度动态分配。不使用asprintf会导致程序运行良好，但我需要asprintf我的项目的功能。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main() {
  int ifCount = 1;
  int stringCount = 1;
  char** IFs = NULL;

  //Broken code
  char* message;
  asprintf(&message, "Hello: %d", stringCount);

  //Working code, but not the alternative I want to take
  //char* message = "Hello";

  IFs = (char**) realloc(IFs, sizeof(char*) * ifCount);
  IFs[ifCount - 1] = (char*) realloc(IFs[ifCount - 1], sizeof(char) * strlen(message));
  strcpy(IFs[ifCount - 1], message);

  printf("Message: %s\n", message);
  printf("Copy: %s\n", IFs[ifCount - 1]);
  free(message);
}
4

2 回答 2

5

这：

IFs[ifCount - 1] = (char*) realloc(IFs[ifCount - 1], sizeof(char) * strlen(message));

正在将未初始化的指针传递给realloc()，这是错误的原因。

还：

  1. 请记住，字符串需要终止空间，上面分配的strlen(message)字符太少了 1。这将导致strcpy()复制时缓冲区溢出。
  2. 请记住realloc()，像所有分配堆内存的函数一样，可能会失败。这也是真的asprintf()。
  3. 不要强制转换realloc()in C的返回值。
  4. 避免sizeof (char)，因为它始终为 1，它为代码增加的价值很少。
于 2012-11-12T15:20:28.983 回答
0

不要使用realloc或未NULL初始化的第一个参数，只需使用malloc开头。

如果在realloc调用中需要IFs[ifCount - 1] = (char*) realloc(...)调用，则在前一行中使用calloc- 这将至少将分配的内存清零，以便realloc获得正确的NULL指针。

于 2012-11-12T15:25:17.440 回答