分段错误（核心转储）C答案

【问题标题】：Seguementation Fault (Core Dump) C分段错误（核心转储）C
【发布时间】：2022-01-14 03:52:31
【问题描述】：

我正面临分段错误的问题。我知道这与记忆有关。我做了一些更改，但一直出错。我正在尝试读取一个 25kb 的大文件。我需要阅读一个最大的 csv 并将其逐行放入结构中。我用链表做到了。我已经尝试过使用列表。如果我在不拆分字符串的情况下读取文件，我没有错误。

#include <string.h>
#include <math.h>
#include <stdio.h>
#include <stdlib.h>



struct gh_user {
    char *id;
    char *login;
    char *type;
    char *createdat;
    char *followers;
    char *followerlist;
    char *following;
    char *followinglist;
    char *public_gists;
    char *pub_repos;
    struct gh_user *next;
};

typedef struct gh_user *GH_USER;

void separador (char *lines, GH_USER k) {

    k->id = strdup(strsep(&lines, ";"));
    k->login = strdup(strsep(&lines, ";"));
    k->type = strdup(strsep(&lines, ";"));
    k->createdat = strdup(strsep(&lines, ";"));
    k->followers = strdup(strsep(&lines, ";"));
    k->followerlist = strdup(strsep(&lines, ";"));
    k->following = strdup(strsep(&lines, ";"));
    k->followinglist = strdup(strsep(&lines, ";"));
    k->public_gists = strdup(strsep(&lines, ";"));
    k->pub_repos = strdup(strsep(&lines, ";"));
    free(lines);
}


int conta_bots()
{
    FILE *usr;
    usr = fopen("../entrada/users.csv","rb");
    
    
    int contador = 0;
    
    char *lines = malloc(1024);
    GH_USER k = malloc(sizeof(struct gh_user));
    int fileSize = 0;

    while (fgets(lines, 1024,usr)!=NULL)
    {  
    
    if(lines != NULL)
    {  
        separador (lines,k->next);
        printf("%s", k->type);
    }
    
    }
    free(k);
    fclose(usr);   
    free(lines);
    
    return contador;
}


int main()
{
    conta_bots();
   
    return 0;
}

【问题讨论】：

能分享一下调试结果吗？
还有其他问题，例如：if(lines != NULL)：此检查毫无意义。 typedef struct gh_user *GH_USER;: 不要在 typedef 后面隐藏指针类型，这只会造成混乱。
实际上释放未分配的指针是未定义的。 strsep 将 lines 的值更新为与其开始时不同的地址，因此 free(lines) 可能会表现出未定义的行为。
最好使用调试器找出它落在哪一行。如果您对代码进行了更改，请edit您问题中的代码，并在必要时更新观察到的行为。
如果您有新代码和/或新信息，请将其添加到您的问题中。不要删除旧内容，因为这会使 cmets 和（可能）答案无用。

标签： c memory

【解决方案1】：

如果没有数据，很难说它到底在哪里崩溃了，但让我们尝试解决我在不运行代码的情况下看到的问题：

#include <string.h>
#include <math.h>
#include <stdio.h>
#include <stdlib.h>

struct gh_user {
    char *id;
    char *login;
    char *type;
    char *createdat;
    char *followers;
    char *followerlist;
    char *following;
    char *followinglist;
    char *public_gists;
    char *pub_repos;
    struct gh_user *next;
};

char *n_strdup(char *in) {
    return strdup(in ? in : "");
}

void separador (char *lines, struct gh_user *k) {
    k->id = n_strdup(strsep(&lines, ";"));
    k->login = n_strdup(strsep(&lines, ";"));
    k->type = n_strdup(strsep(&lines, ";"));
    k->createdat = n_strdup(strsep(&lines, ";"));
    k->followers = n_strdup(strsep(&lines, ";"));
    k->followerlist = n_strdup(strsep(&lines, ";"));
    k->following = n_strdup(strsep(&lines, ";"));
    k->followinglist = n_strdup(strsep(&lines, ";"));
    k->public_gists = n_strdup(strsep(&lines, ";"));
    k->pub_repos = n_strdup(strsep(&lines, ";"));
}

/*yes, you need to free all that*/
void gh_user_free(struct gh_user *ptr) {
    if (!ptr) return;
    free(ptr->id);
    free(ptr->login);
    free(ptr->type);
    free(ptr->createdat);
    free(ptr->followers);
    free(ptr->followerlist);
    free(ptr->following);
    free(ptr->followinglist);
    free(ptr->public_gists);
    free(ptr->pub_repos);
    free(ptr);
}


int conta_bots()
{
    FILE *usr;
    usr = fopen("../entrada/users.csv","rb");

    int contador = 0;

    char lines[1024];

    while (fgets(lines, 1024,usr)!=NULL)
    {  
        struct gh_user *k = calloc(1, sizeof(struct gh_user));
        separador (lines,k);
        printf("%s", k->type);
        gh_user_free(k);
    }
    fclose(usr);
    
    return contador; /*possibly that var was supposed to be updated in some way?*/
}


int main()
{
    conta_bots();
    return 0;
}

您的代码示例没有使用链表，所以我省略了。

【讨论】：