当释放内存中的内存时程序崩溃

-3

我有文件解析程序是这样的，我的程序在释放内存时崩溃。我必须检查值是否为空然后我必须释放其他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=

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

“的东西是零” 和程序崩溃。

来源

2015-10-11 user3206557

什么调试器说什么？ –

你没有在'while（fgets（line，sizeof（line），fp）！= NULL）{'中显示变量'line'。这可能是麻烦来源，或者它可能完全无害。（你也没有显示'key'和'value'的声明。）但是，如果你发布可编译的代码，你应该瞄准向我们展示一个MCVE（[如何创建一个最小，完整和可验证的例子？]（http://stackoverflow.com/help/mcve））。 –

需要初始化在struct db_handle_st的字符指针，否则为NULL，你可能会释放一些随机数据（其价值是不确定的，否则），这将导致崩溃。

PS：你不需要调用free（free上NULL只是什么都不做）之前检查NULL也应该fclose你打开的文件。

来源

2015-10-11 17:52:38 MrTux

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

用途：

db_handle_st db_details = { 0, 0 };

来源

2015-10-11 17:58:21

当释放内存中的内存时程序崩溃

回答

相关问题