fclose () вызывает ошибку сегментации

Question

У меня есть текстовый файл с разделителями табуляции, который я анализирую. Его первый столбец содержит строки формата chrX, где X обозначает набор строк, например, "1", "2", ..., "X", "Y".

Каждый из них сохраняется в char*, называемом chromosome, так как файл анализируется.

Текстовый файл отсортирован по первому столбцу лексикографически, т. Е. У меня будет ряд строк, начинающихся с "chr1", а затем "chr2" и т. Д.

В каждой записи "chrX" мне нужно открыть другой файл, связанный с этой записью:

FILE *merbaseIn;

// loop through rows...

if (chromosome == NULL)                                                                                                                                                   
    openSourceFile(&chromosome, fieldArray[i], &merbaseIn, GENPATHIN);                                                                                                      
else {                                                                                                                                                                    
    if (strcmp(chromosome, fieldArray[i]) != 0) { // new chromosome                                                                                                   
        fclose(merbaseIn); // close old chromosome FILE ptr                                                                                                                                                                                                                                    
        free(chromosome); // free old chromosome ptr                                                                                                                          
        openSourceFile(&chromosome, fieldArray[i], &merbaseIn, GENPATHIN); // set up new chromosome FILE ptr                                                                  
    }                                                                                                                                                                       
}  
// parse row

У меня есть функция openSourceFile, которая определяется следующим образом:

void openSourceFile (char** chrome, const char* field, FILE** filePtr, const char *path) {
    char filename[100];                                                                                                                                                           
    *chrome = (char *) malloc ((size_t) strlen(field));
    if (*chrome == NULL) {                                                                                                                                                        
        fprintf(stderr, "ERROR: Cannot allocate memory for chromosome name!");                                                                                                      
        exit(EXIT_FAILURE);                                                                                                                                                         
    }                                                                                                                                                                             

    strcpy(*chrome, field);                                                                                                                                                       
    sprintf(filename,"%s%s.fa", path, field);                                                                                                                                     

    *filePtr = fopen(filename, "r");                                                                                                                                              
    if (*filePtr == NULL) {                                                                                                                                                       
        fprintf(stderr, "ERROR: Could not open fasta source file %s\n", filename);                                                                                                  
        exit(EXIT_FAILURE);                                                                                                                                                         
    }                                                                                                                                                                             
}      

Проблема в том, что мое приложение закрывается с ошибкой сегментации, идущей от первой хромосомы ко второй (от chr1 до chr2) в следующей строке, где я закрываю первый файл хромосомы, который я открыл:

fclose(merbaseIn);

Я знаю, что не передаю fclose указатель NULL, потому что вплоть до ошибки сегментации я читаю данные из этого файла. Я могу даже обернуть это в условное, и я все еще получаю Ошибка:

if (merbaseIn != NULL) {
    fclose(merbaseIn);
}

Кроме того, я знаю, что openSourceFile работает (по крайней мере, для chr1, при настройке первого дескриптора файла FILE*), потому что мое приложение анализирует строки chr1 и считывает данные из исходного файла FILE* правильно .

Что это за вызов fclose, который вызывает ошибку сегментации?

Answer 1

valgrind --db-attach=yes --leak-check=yes --tool=memcheck --num-callers=16 --leak-resolution=high ./yourprogram args

Весьма вероятно, что ошибка вызвана повреждением памяти в куче, а не чем-либо, что влияет на местных жителей. Valgrind немедленно покажет вам первый неверный доступ, который вы сделаете.

Редактировать: параметр --db-attach для valgrind устарел с момента выпуска 3.10.0 в 2014 году. Состояние примечаний к выпуску:

The built-in GDB server capabilities are superior and should be used
instead. Learn more here:

http://valgrind.org/docs/manual/manual-core-adv.html#manual-core-adv.gdbserver

Answer 2

одна ошибка, которую я замечаю, это строка:

 *chrome = (char *) malloc ((size_t) strlen(field));

которое должно быть:

 *chrome = (char *) malloc ((size_t) strlen(field)+1);

Это потому что строка имеет закрывающий 0 в конце, который вы также должны освободить место для

Answer 3

В дополнение к ошибке, обнаруженной reinier , я подозреваю, что:

free(chromosome);

должно сопровождаться:

chromosome = NULL;

для предотвращения возможного использования недействительного значения.

Answer 4

Общая проблема с указателем

C - отличный язык, но он требует, чтобы вы не забивали свою собственную память. Помимо ранее отмеченной проблемы, когда malloc имеет длину 1 байт, у вас могут быть и другие проблемы с указателями.

Я бы предложил с использованием отладчика памяти . В прошлом Electric Fence был довольно популярен, но сейчас я слышу больше о valgrind . Есть много других вариантов.

Answer 5

Лучше всего предположить, что какая-то другая часть вашего кода повреждает память из-за переполнения буфера или подобной ошибки.

Хотя это вряд ли является причиной, у вас есть потенциальное переполнение в массиве имен файлов, когда полное имя файла превышает 100 символов.

Я бы предложил использовать отладчик для отслеживания изменений в ячейке памяти, используемой переменной merbaseIn.

Answer 6

Просмотрите каждое использованное вами место "malloc" и посмотрите, не ошиблись ли вы.

Например, я помещал строки, прочитанные из файла, в символ **, но я неправильно называет это как:

my_list = malloc(sizeof(char) * num_lines_found_in_file);

Когда это должно было быть:

my_list = malloc(sizeof(char*)* num_lines_found_in_file);

Answer 7

valgrind с memcheck - определенно правильный инструмент для обнаружения причины ошибки сегментации. Чтобы использовать отладчик с valgrind, обратите внимание, что опция --db-attach для valgrind устарела с момента выпуска Valgrind 3.10.0 в 2014 году. Состояние примечаний к выпуску:

The built-in GDB server capabilities are superior and should be used
instead. Learn more here:

http://valgrind.org/docs/manual/manual-core-adv.html#manual-core-adv.gdbserver

Answer 8

почему этот FILE ** filePtr, если только этого FILE * filePtr будет достаточно? просто мысль ...