我尝试使用Realloc函数分配并获取错误

Question

typedef struct{
   char** strings_cmd;
   int size_cmd;
}parseInfo;

....

parseInfo* parse(char* cmd){
    char* temp = strdup(cmd);
    char* temp_split = strtok(temp," ");
    int i = 0;
    char** strings = (char**)malloc(sizeof(char*));
    if(strings == NULL){
        printf("no memory allocated strings parse()\n");
        exit(1);
    }
    while(temp_split != NULL){
        strings[i++] = strdup(temp_split);
        strings = realloc(strings,i * sizeof(char*));
        if(strings == NULL){
            printf("no memory allocated strings (while) parse()\n");
            exit(1);
        }   
        temp_split = strtok(NULL," ");
    }
    strings[i] = NULL;
    parseInfo* info = (parseInfo*)malloc(sizeof(parseInfo));
    if(info == NULL){
        printf("no memory allocated info parse()\n");
        exit(1);
    }
    info->strings_cmd = strings;
    info->size_cmd = i;
    return info;
}

大家好，我收到错误：

realloc()：下一个大小无效。

我尝试做的是输入一个字符串并将其分解为单词 例如我输入=“Hello World”。 并分割它=“Hello”，“World”，

但是当我传递4个单词时，我得到了这个错误......

Answer 1

对于初学者来说，该函数存在内存泄漏，因为在函数的开头分配了

parseInfo* parse(char* cmd){
    char* temp = strdup(cmd);
    //...

未释放的内存。

在这个 while 循环中，

while(temp_split != NULL){
    strings[i++] = strdup(temp_split);
    strings = realloc(strings,i * sizeof(char*));
    if(strings == NULL){
        printf("no memory allocated strings (while) parse()\n");
        exit(1);
    }   
    temp_split = strtok(NULL," ");

您需要编写

strings = realloc(strings, ( i + 1 ) * sizeof(char*));

为该语句中使用的终止空指针保留一个元素

strings[i] = NULL;

，并且您需要在函数开头释放动态分配的内存，就像

free( temp );

}

您分配一个少一个元素的指针数组一样这是必需的。

Answer 2

这行代码很糟糕：

        strings = realloc(strings,i * sizeof(char*));

这行代码将数组大小调整为 i 元素。
然后，在下一次迭代中，某个值被存储到数组的第 i 个元素（由 strings 指向）。该数组只有 i 个元素（0 到 i-1），因此这是超出范围的访问。

分配足够的元素来修复：

        strings = realloc(strings,(i + 1) * sizeof(char*));

另请注意 malloc() 系列的转换结果为 被认为是一种不好的做法。