c - 标记化数组上的realloc（）：信号SIGABRT错误

Question

在第56行，我试图调整数组的大小：

tokenArray = (char**) realloc(tokenArray, tokSize * (sizeof(char)));

我收到错误：

  

（11972,0x7fff7ca4f300）malloc： *对象0x100105598的错误：释放对象的校验和不正确 - 对象可能在被释放后被修改。   * 在malloc_error_break中设置断点以进行调试

这是一个类的编程任务，我已经被特别指示动态分配我的数组，然后根据需要进行扩展。我已经广泛搜索了另一个相同的线程，这个线程对我来说太难以理解，没有运气......所以希望我能得到一些帮助。谢谢！这是我的代码：

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

#define MAX_ROW_SIZE 81

void strInput(char str[], int numElem);


int main(int argc, const char * argv[])
{
    printf("Enter a string of any number of integers separated by spaces or tabs.\n");
    printf("Maximum string length is 80 characters.\n");
    printf("Enter an empty string to conclude input.\n");


    int arrSize = 10, tokSize = 10, i = 0, j = 0;

    char** inputArray = malloc(arrSize * (sizeof(char)));
    char** tokenArray = malloc(tokSize * (sizeof(char)));


    do {
        inputArray[i] = malloc(MAX_ROW_SIZE * sizeof(int));

        strInput(inputArray[i], arrSize);

        if ((inputArray[i][0] != '\0') && (i == (arrSize - 1)))
        {
            arrSize = arrSize * 2;
            inputArray = (char**) realloc(inputArray, arrSize * (sizeof(char)));
        }

        while (inputArray[i][j] != '\0')
        {
            printf("%c", inputArray[i][j]);
            j++;
        }
        j = 0;

        i++;
    } while (inputArray[i-1][0] != '\0');

    i = 0;

    while (inputArray[i][0] != '\0')
    {

        if ((tokenArray[j] = strtok(inputArray[i], " \t")))
            j++;
        while ((tokenArray[j] = strtok(NULL, " \t")))
        {
            if (j == (tokSize - 1))
            {
                tokSize = 2 * tokSize;

 //This is the line where I get the error
                tokenArray = (char**) realloc(tokenArray, tokSize * (sizeof(char)));
            }
            j++;
        }
        i++;
    }

    printf("printing the tokenized arrays: ");
    for (i = 0; i < j; i++)
        printf("%s ", tokenArray[i]);

    free(inputArray);
    free(tokenArray);

    return 0;
}

void strInput(char str[], int numElem)
{


    int j, k = 0;

    j = k;

    while ((str[k] = getchar()) != '\n')
    {
        k++;
    }

    if (str[k] == '\n')
        str[k] = '\0';
}

Answer 1

1. Do not cast the result of malloc and friends.

它充其量是无用的，最坏的是危险的。

2。注意你malloc的大小。

char** inputArray = malloc(arrSize * (sizeof(char)));

这没有任何意义，可能是偶然的。根据经验，您malloc的类型以及您指向结果存储的指针应该只有一个间接不同。我：

char** inputArray = malloc(arrSize * sizeof(char*));
//  ^^ Double-pointer     vs     Single pointer ^

更好的经验法则，让编译器搞清楚。 sizeof可以从表达式中推断出它应该测量的类型。

char **inputArray = malloc(arrSize * sizeof(*inputArray));

这是有效的，因为sizeof的操作数是未评估的上下文。指针实际上不会被解除引用，只会推断它的类型 附注：表达式周围不需要sizeof＆＃39}括号，但为了清晰起见，我已将它们留下了。一旦你感到舒服，就把它们移走。

3。检查分配是否成功。

malloc，如果遇到问题，朋友会返回NULL。你应该检查一下。

4。修复您的realloc

inputArray = realloc(inputArray, /*...*/);

这是错误的。如上所述，如果realloc失败，它将返回NULL 并且不执行任何操作。这意味着inputArray仍指向其先前的存储空间。也就是说，直到你用刚才返回的NULL realloc来敲击这个指针，然后泄漏所述存储。糟糕。

始终存储，检查，然后指定realloc的结果。

char **inputArray_ = realloc(inputArray, /*...*/);
if(!inputArray_) {
    /* Allocation failure, handle it and break out */
}
inputArray = inputArray_;