«перераспределить(): недействительный следующий размер» после нескольких успешных запусков

Question

я получаю эту ошибку после того, как массив уже хранится несколько строк из пользовательского ввода, который говорит мне, что это, вероятно, повреждена память из-за линии:

poemArray = (char **)realloc(poemArray, count + 1); 

Любая идея, что я делаю неправильно? Конкретное решение было бы высоко оценено!

line = (char *)malloc(MaxLineLen); 

    fgets(line, MaxLineLen, stdin); 
    /*Get current line from user input*/ 
    if(count == 0) 
    { 
     poemArray = malloc(sizeof(char *)); 
     printf("1\n"); 
    } 
    if(line[0] == '.'){ 
     break; 
    } 
    line = (char *)realloc(line, strlen(line)); 

    printf("2\n"); 

    if(count != 0) 
    { 
     poemArray = (char **)realloc(poemArray, count + 1); 
    } 
    poemArray[count] = line; 

    ++count; 

Answer 1

Это

poemArray = (char **)realloc(poemArray, count + 1); 

должен быть actualy

poemArray = realloc(poemArray, (count + 1) * sizeof(char *)); 

Кроме того, этот

line = (char *)realloc(line, strlen(line)); 

, вероятно, следует

line = realloc(line, strlen(line) + 1); 

Также неясно, почему вы делаете первоначальный malloc для poemArrayперед тем потенциал break. Таким образом, вы можете получить poemArray как неинициализированный массив размера 1. В чем смысл его неинициализации?

---

Кроме того, обратите внимание, что realloc предназначен для правильной обработки нулевого указателя в качестве первого аргумента. В таких случаях realloc по существу эквивалентен malloc. Воспользовавшись этой особенностью realloc, вы можете исключить выделенную обработку для состояния count == 0, в результате получив гораздо более компактный и элегантный код.

P.S. Почему некоторые звонки в malloc включают явное приведение, а другие нет? В любом случае нет смысла обрезать результаты функций выделения памяти в C.