重新分配二维字符数组

Question

我有以下代码，

 int wordLenght = 256, arrayLength = 2, i = 0, counter = 0;
 char **stringArray = NULL; 

 stringArray = calloc(arrayLength, sizeof(*stringArray));

 for(counter; counter<wordLenght; counter++) 
    stringArray[counter] = calloc(wordLenght, sizeof(stringArray));

 while(1)
 {
   printf("Input: ");
   fgets(stringArray[i], wordLenght, stdin);

   printf("stringArray[%d]: %s\n", i, stringArray[i]);

   if(i == arrayLength)
   {
     printf("Reallocation !!!\n");
     arrayLength *= 2;

     stringArray = realloc(stringArray, arrayLength*sizeof(*stringArray));

   } 

   i++;
 }    

但出现此重新分配错误：

*** glibc detected *** ./stringArray: realloc(): invalid next size: 0x0000000000b49010 ***
======= Backtrace: =========
/lib/libc.so.6(+0x775b6)[0x7f4dd12565b6]
/lib/libc.so.6(+0x7dd66)[0x7f4dd125cd66]
/lib/libc.so.6(realloc+0xf0)[0x7f4dd125d080]
./stringArray[0x4007f9]
/lib/libc.so.6(__libc_start_main+0xfd)[0x7f4dd11fdc4d]
./stringArray[0x400629]

我的问题是什么？？？

谢谢，问候

Answer 1

 stringArray = calloc(arrayLength, sizeof(*stringArray));

这里你可能想使用 sizeof(char*)

for(counter; counter<wordLenght; counter++) stringArray[counter] = calloc(wordLenght, sizeof(stringArray));

这里你循环了 256 次 (wordLenght) 但你应该只循环 2 次 (arrayLength)。此外，您可能想使用 sizeof(char) 而不是 sizeof(stringArray)。

if(i == arrayLength) {...}

这个检查应该在调用 fgets 之前完成，因为现在您首先使用内存，然后再分配它们。

此外，在重新分配 stringArray 之后，您需要使用类似的方法分配其余字符串

for(counter = i; counter<arrayLength; counter++) stringArray[counter] = (char*)calloc(wordLenght, sizeof(char));

，最后您需要在退出应用程序之前释放所有分配的内存。

Answer 2

您可能不是指 sizeof(*stringArray)

事实上，我相信您可能也想重新查看 calloc 调用，我认为您正在分配指针的大小那里（字长乘以）。

Answer 3

第一次执行此行后：

stringArray = realloc(stringArray, arrayLength*sizeof(*stringArray));

那么 stringArray[arrayLength/2] 将是一个垃圾值 - 您尚未将其设置为指向该单词的存储。

这部分应该使用 use sizeof(**stringArray)，或者 1 因为 **stringArray 是 char，并且计数器应该只达到 arrayLength：

 for(counter; counter<wordLenght; counter++) 
     stringArray[counter] = calloc(wordLenght, sizeof(stringArray));

而不是在一个块中分配：

 char* block = malloc(wordLength * arrayLength);

 for ( counter; counter < arrayLength; ++counter ) 
     stringArray[counter] = block + ( counter * wordLength );

目前，后面可能有一些空格stringArray，当您调用它们时，将（wordLength-arrayLength）额外指针存储到其中，并且 realloc 不会移动 stringArray。

很可能 0xb49010 是您调用的指针之一，并且您覆盖了 malloc 保留其块大小的内存。

但是由于您正在注销 stringArray 的末尾，因此您无论如何都会陷入未定义的行为。

Answer 4

好的，这是整个解决方案：

int wordLength = 256, arrayLength = 2, i = 0, counter = 0;
    char **stringArray = NULL;
    char buffer[wordLength];

    stringArray = calloc(arrayLength, sizeof(char*));
    for(counter; counter<arrayLength; counter++) stringArray[counter] = (char*)calloc(wordLength, sizeof(char));

    while(1)
    {
        if(i == arrayLength)
        {
            printf("Reallocation !!!\n");
            arrayLength *= 2;

            stringArray = realloc(stringArray, arrayLength*sizeof(char*));
            for(counter = i; counter<arrayLength; counter++) stringArray[counter] = (char*)calloc(wordLength, sizeof(char));
        }   

        printf("Input: ");
        fgets(buffer, wordLength, stdin);

        if(!strcmp(buffer,"q\n")) break; // also free here      
        else stringArray[i] = buffer;

        printf("stringArray[%d]: %s\n", i, stringArray[i]);

        i++;
    }

释放空间的最佳方法是什么？！