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

Question

我有这样的文件解析程序，我的程序在释放内存时崩溃。我必须检查值是否为空，然后我必须释放另一个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] = ''; /* trim the newline. */
            if ((value = strstr(line, "=")) != NULL) {
                *value = '';
                value++;
            }
            if (key && value)
                printf("    %s: %sn", 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 %sn",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 %sn",db_details.user);
            }
}
}
if((db_details.user!=NULL) || (db_details.server!=NULL)) {
        printf("something is zeron");
        if(db_details.user) {
            free(db_details.user);}
        if(db_details.server) {
            free(db_details.server);}
    }
}

我的配置是这样的

SERVER=localhost
USER=

当我运行这个程序时，我得到

"某物为零"程序崩溃

Answer 1

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

PS:在调用free之前，您不需要检查NULL (free对NULL只是什么都不做)，您还应该fclose您打开的文件。

Answer 2

您从未初始化db_handle_st db_details;，因此指针是不确定的，并且崩溃表明不确定的值不是空指针。

使用:

db_handle_st db_details = { 0, 0 };