strdup를 사용하려고 할 때 C - segfault

Question

이전에 strdup()을 사용했습니다. 여기에서 사용하는 것과 같습니다. token2를 strdup에 전달합니다.이 strdup는 유효한 포인터가있는 char * 유형이며, "name = strdup (token2);"줄을 실행하려고 할 때 사용됩니다. 내 프로그램 segfaults 그리고 나는 확실히 확신 할 수 없다. 누구든지 나를 도울 수 있다면 크게 감사하겠습니다. 나는 또한 내 코드가 적절한 타입을 반환하지 않는다는 것을 알고있다. 나는 아직도 그것을 작성하는 중이다.

struct YelpDataBST* create_business_bst(const char* businesses_path, const char* reviews_path){ 

    if(fopen(businesses_path,"r") == NULL || fopen(reviews_path,"r") == NULL) 
    return NULL; 
    FILE* fp_bp = fopen(businesses_path, "r"); 
    FILE* fp_rp = fopen(reviews_path, "r"); 

    struct YelpDataBST* yelp = malloc(sizeof(struct YelpDataBST*)); 

    int ID = -1; 
    int tempID; 
    long int addressOffset; 
    long int reviewOffset; 
    char line[2000]; 
    char line2[2000]; 
    char temp[2000]; 
    char temp2[2000]; 
    char* token; 
    char* token2; 
    char* name; 
    int len; 

    BusList* busNode = NULL; 
    BusList* busList = NULL; 
    BusTree* busTreeNode = NULL; 
    BusTree* busTree = NULL; 

    ID = -1; 
    tempID = 0; 
    fgets(line,2000,fp_rp); 
    fgets(line2,2000,fp_bp); 
    fseek(fp_rp,0, SEEK_SET); 
    fseek(fp_bp,0,SEEK_SET); 
    int ct = 0; 
    while(!feof(fp_rp)){ 

    len = strlen(line); 
    token = strtok(line, "\t"); 
    //printf("line: %s\n", line); 
    token2 = strtok(line2, "\t"); 
    tempID = atoi((char*)strdup(token)); 
    if(ct == 0){ 
     tempID = 1; 
     ct++; 
    } 

    if((ID != tempID || (ID < 0)) && tempID != 0){ 
    if(tempID == 1) 
     tempID = 0; 
    token2 = strtok(NULL, "\t"); 
    //name = strdup(token2); 
    reviewOffset = ftell(fp_rp); 
    if(tempID != 0) 
     reviewOffset -= len; 
    addressOffset = ftell(fp_bp); 
    ID = atoi((char*)strdup(token)); 
    busList = BusNode_insert(busList, addressOffset, reviewOffset); //replace with create node for tree 
    token2 = strtok(NULL, "\t"); 
    token2 = strtok(NULL, "\t"); 
    token2 = strtok(NULL, "\t"); 
    token2 = strtok(NULL, "\t"); 
    token2 = strtok(NULL, "\t"); 
    token2 = strtok(NULL, "\n"); 
    fgets(line2,2000,fp_bp); 
    } 
    token = strtok(NULL, "\t"); 
    token = strtok(NULL, "\t"); 
    token = strtok(NULL, "\t"); 
    token = strtok(NULL, "\t"); 
    token = strtok(NULL, "\n"); 
    fgets(line,2000,fp_rp); 

    } 

    //BusList_print(busList); 

} 

Answer 1

strdup(token) segfaulting은 대부분 token 존재의 NULL에 의해 설명된다. (당신은 어쨌든 여기에 strdup 필요가 없습니다). 코드의 부분 변경 :

if (token == NULL) 
{ 
    fprintf(stderr, "Invalid data in file.\n"); 
    exit(EXIT_FAILURE); // or some other error handling 
} 

tempID = atoi(token); 

은 그러나 주변의 코드에 더 큰 문제는 한 번에 두 번 strtok을 사용하려고한다는 것입니다. 내부 상태를 유지하며 한 번에 하나만 strtok을 "진행 중"으로 설정할 수 있습니다. 두 번째 것은 첫 번째 것을 취소합니다. 해당 코드 섹션을 다시 디자인해야합니다. 또한

---

, while(!feof(fp_rp)) is wrong하고 yelp mallocs 바이트의 잘못된 번호 (코드를 게시 실제로는 아직 오류가 발생하지 않을 수 있도록, 해당 스토리지에 아무것도 저장하지 마십시오 있지만).