C - segfault при попытке использовать strdup

Question

Я использовал strdup() в прошлом так же, как и я здесь. Я передаю token2 в 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 неверное число байтов (хотя в коде отправил вас никогда не пытаться хранить что-либо в этом хранилище, так что это не вызовет ошибку только пока).