FCLOSE는 SEG 나는 경우에만 나는 파일을 닫으려고하면 독방 감금 오류가 계속 C

Question

에 결함으로 하여금. 어떤 제안?

gdb의 오류가 여기로 리디렉션됩니다. 모든 변수가 선언 된 함수라고 가정합니다. 또한 gdb는 내가 사용하지도 않는 strtol을 비난합니다. 파일을 닫을 때 나는 항상 그것을 할

if (outFilePtr) {fclose(outFilePtr); outFilePtr = NULL;} 

와 나는 또한 포인터를 넣어 :

int t; 
    char line[50]; 

      for (t = 0; t < lines; t++){ 
       fgets(line, 50, filePtr); 
      strcpy(*string[t], strtok(line, " ")); 
       *(num1 + t) = atoi(strtok(NULL, " ")); 
       *(num2 + t) = atoi(strtok(NULL, " ")); 
      } 

는 메모리 할당 기능

void dynamicArray(int** num1, int** num2, char*** str, int size) 
{ 
    int i = 0; 

*(num1) = (int*)malloc(sizeof(int) * size); 
*(num2) = (int*)malloc(sizeof(int) * size); 

*(str) = (char**)malloc(sizeof(char*) * size); 

for(i = 0; i < size; i++){ 
    *(*(str) + i) = (char*)malloc(sizeof(char) *size); 
} 

return; 
} 

Answer 1

그냥 확실하게는, outFilePtr가 null이 아닌 확인 동일한 파일을 두 번 닫으 려하지 않으려면 NULL로 설정하십시오 (문제가 발생할 수 있음).

하지만 가장 일반적인 원인은 일부 메모리 누수 또는 정의되지 않은 동작으로 인해 상황이 엉망이고 segfault가 fclose()에 의해 방금 트리거됩니다.

Answer 2

내 생각에 fclose()에 도달하면 outFilePtr 값이 보존되지 않습니다.

코드 스 니펫이 너무 짧아서 중요한 요소 일 수있는 너무 많은 다른 것들을 놓치게됩니다 ... 문자열과 num2는 무엇입니까? 그들이 얼마나 큰 지 할당 받았다. ...

또한 * (num2 + t)에 대한 첫 번째 저장소는 두 번째 * (num2 + t)로 덮어 씁니다.

도 참조하십시오. ... num2 [t]는 * (num2 + t)보다 읽기가 쉽고 동일한 작업을 수행합니다.

Answer 3

끔찍한 배열 액세스 구문을 제외하고. 입력 문자열 줄을 복사하는 것을 잊었습니다. strtok은 항상 모든 행에서 변경되는 동일한 버퍼를 가리 킵니다. 당신이 정말로 5 바이트를 할당하는 경우는 대신 50 만 5 바이트를 할당하면 할당 코드에서

int t; 
char line[50]; 

     for (t = 0; t < lines; t++){ 
      fgets(line, 50, filePtr); 
      strings[t] = strdup(strtok(line, " "))); 
      num1[t] = atoi(strtok(NULL, " ")); 
      num2[t] = atoi(strtok(NULL, " ")); 
     } 

는, 당신은 힙을 clobbert 이것은 fclose 충돌에 의해 종종 나타난다.

void dynamicArray(int** num1, int** num2, char*** str, int size) 
{ 
int i = 0; 

*num1 = malloc(sizeof(int) * size); 
*num2 = malloc(sizeof(int) * size); 

*str = malloc(sizeof(char*) * size); 

for(i = 0; i < size; i++) 
    (*str)[i] = malloc(50);  /* sizeof (char) is by definition 1 */ 

return; 
}