當釋放內存中的內存時程序崩潰

-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

當釋放內存中的內存時程序崩潰

回答

相關問題