在释放结构内部的内存时程序崩溃

时间:2015-10-11 17:49:55

标签: c struct malloc free

我有文件解析程序,就像这样,我的程序在释放内存时崩溃了。我必须检查值是否为空,然后我必须释放其他malloced变量。

struct db_handle_st {
    char *server;
    char *user;
};

int main()
{   
    char srv_conf_file[] = "C:\\\\Users\\admin\\Documents\\Visual Studio 2010\\Projects\\abcd\\abcd\\service.config";
    FILE *fp = NULL;
db_handle_st db_details;
fp = fopen(srv_conf_file, "r");
    if (fp != NULL) {
        /* Look for key value pairs. */
        while (fgets(line, sizeof(line), fp) != NULL) {
            /* Get key */
            key = line;
            key[strlen(key) - 1] = '\0'; /* trim the newline. */
            if ((value = strstr(line, "=")) != NULL) {
                *value = '\0';
                value++;
            }
            if (key && value)
                printf("    %s: %s\n", key, value);
            else
                continue;
            if (!strncmp(key, "SERVER", strlen("SERVER"))&&(strcmp(value,""))) {
                /*  if(strcmp(value,"")==0) {
                goto err;
                }*/
                db_details.server = (char *)malloc(strlen(value)+1);
                strcpy(db_details.server, value);
                printf("db_details.server is %s\n",db_details.server);
            }
            if (!strncmp(key, "USER", strlen("USER"))&&(strcmp(value,""))) {
                db_details.user = (char *)malloc(strlen(value)+1);
                strcpy(db_details.user, value);
                printf("db_details.user is %s\n",db_details.user);
            }
}
}
if((db_details.user!=NULL) || (db_details.server!=NULL)) {
        printf("something is zero\n");
        if(db_details.user) {
            free(db_details.user);}
        if(db_details.server) {
            free(db_details.server);}
    }
}

我的配置看起来像这样

SERVER=localhost
USER=

当我运行这个程序时,我得到了

  

“某事为零”   程序崩溃了。

2 个答案:

答案 0 :(得分:1)

您需要将struct db_handle_st中的char指针初始化为NULL,否则您可能会释放一些随机数据(否则它们的值未定义),这将导致崩溃。

PS:在致电NULL之前,您无需检查free free NULL上什么都不做)您也应该fclose该文件你打开了。

答案 1 :(得分:0)

你从未初始化db_handle_st db_details;所以指针是不确定的,崩溃表明不确定的值不是空指针。

使用:

db_handle_st db_details = { 0, 0 };