Почему этот код segfault? (Арифметика указателя)

Question

По какой-то причине этот код является неполадки по причине, которую я не могу найти.

char *read_line(FILE *fp) 
{ 
    char *out; 
    int counter = 0; 
    char c = getc(fp); 

    while (c != '\n' && c != EOF) 
    { 
     *(out + counter) = c; 
     counter++; 
     c = getc(fp); 
    } 

    if (c == EOF || feof(fp)) 
    { 
     return NULL; 
    } 

    *(out + counter) = '\0'; 
    return out; 
} 

Я уже попытался запустить его в gdb, и сказал мне, что это в выдаёт ошибку сегментации *(out + counter) = c;. Я не могу понять, что я делаю неправильно, может ли кто-нибудь еще?

Answer 1

Вы не назначаете никакого значения out, поэтому, вероятно, это указывает на некорректный адрес памяти.

То, что вы, вероятно, хотите сделать, это найти длину данных, а затем выделить требуемый объем памяти, а затем прочитать фактические данные в выделенной памяти:

char *out; 
int counter = 0; 
char c = getc(fp); 
while (c != '\n' && c != EOF) { 
    counter++; 
    c = getc(fp); 
} 

out = malloc(counter+1); 
fseek(fp,0,SEEK_SET); 

counter = 0; 
c = getc(fp); 
while (c != '\n' && c != EOF) { 
    *(out + counter) = c; 
    counter++; 
    c = getc(fp); 
} 
*(out + counter) = 0; 

И не забудьте до free(out), когда вы закончите использовать его ...

BTW, а не второй цикл for, вы можете просто использовать fgets(out,count,fp).