我有文件解析程序,就像这样,我的程序在释放内存时崩溃了。我必须检查值是否为空,然后我必须释放其他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=
当我运行这个程序时,我得到了
“某事为零” 程序崩溃了。
答案 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 };