Сегментация Fault Использование strcat()

Question

Я имею ошибки сегментации на моем strcat в коде ниже:

csv* lines_into_csv(int m,char* array[LINES_MAX][COLUMNS]){ 
    char *first_string, *final_string; 
    first_string = (char*)calloc(400,sizeof(char)); 
    final_string = (char*)calloc(400,sizeof(char)); 
    csv *earthquake = malloc(sizeof(csv)*LINES_MAX); 
    int n, j; 

    for(j = 0; j < m; j++){ 
    for(n = 0; n < COLUMNS; n++){ 
     if (array[j][n] != NULL) { 
     if(n < 4){ 
      strcat(first_string, array[j][n]); 
     } 
     else if(n == 4){ 

      earthquake[j].mag = atof(array[j][n]); 
     } 
     else { 

      strcat(final_string, array[j][n]); 
     } 
     } 
    } 
    earthquake[j].start_of_line = (char*)calloc(200,sizeof(char)); 
    earthquake[j].end_of_line = (char*)calloc(200,sizeof(char)); 
    earthquake[j].start_of_line = strdup(first_string); 
    earthquake[j].end_of_line = strdup(final_string); 
    free(first_string);free(final_string); 
    } 


    return earthquake; 
} 

структура типа CSV состоит из двух полукокса * и поплавка, который является числом от того, где я сортировкой файл.

Answer 1

С кодом здесь

csv *earthquake = malloc(sizeof(csv*)*LINES_MAX); 

вы были отнесены массив указателей на Csv структур, но вы, кажется, предположить, что вы можете получить доступ к объектам структуры CSV здесь:

earthquake->start_of_line = (char*)calloc(200,sizeof(char)); 
earthquake->end_of_line = (char*)calloc(200,sizeof(char)); 

вы, вероятно, означало

csv *earthquake = malloc(sizeof(csv)*LINES_MAX); 

EDIT:

Проверьте в параметре, что длины в нем не превышают выделенную вами длину. Также убедитесь, что строки in-string правильно завершены.

Answer 2

У вас возникла проблема с использованием strcat(), потому что вы не инициализируете ни одну из строк, которую вы используете в качестве цели.

strcat(first_string, tmp); 
... 
strcat(final_string, tmp); 

strcat() рассчитывает найти Нуль-терминатор, добавляемых затем следующую строку в. Если он не найдет его, он будет продолжать пахать память до тех пор, пока это не произойдет, тогда кто знает, куда он скопирует вашу другую строку?

Одним из решений было бы использовать calloc(), который записывает 0 в память, которую он выделяет.

Существует еще одна проблема, утечка памяти. Вы выделили память здесь, но только для первого элемента массива.

earthquake->start_of_line = malloc(100*sizeof(char*)); 
earthquake->end_of_line = malloc(200*sizeof(char*)); 

, что приводит к утечке памяти, когда вскоре после того, вы

earthquake[j].start_of_line = strdup(first_string); 
earthquake[j].end_of_line = strdup(final_string); 

Вы должны знать, что strdup() выделяет больше памяти и возвращает указатель, который затем перезаписать исходный указатель, возвращаемый malloc(), так невозможно освободить эту память, поскольку у вас больше нет указателей.

Переходя к аргументу функции array[][], который вы используете для dup(), был ли он инициализирован или он тоже случайный?