From 3624e4e923ab5ecacff21c4b4c4a17ab41ae45bb Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Wed, 7 May 2008 03:04:58 +0000 Subject: [PATCH] - if we can't parse the INI file we can't set ini->data so return NULL instead of an invalid ini file SVN revision: 34493 --- legacy/efreet/src/lib/efreet_ini.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/legacy/efreet/src/lib/efreet_ini.c b/legacy/efreet/src/lib/efreet_ini.c index 7a4692b44a..bd6a1017b0 100644 --- a/legacy/efreet/src/lib/efreet_ini.c +++ b/legacy/efreet/src/lib/efreet_ini.c @@ -55,6 +55,7 @@ efreet_ini_new(const char *file) if (!ini) return NULL; ini->data = efreet_ini_parse(file); + if (!ini->data) FREE(ini); return ini; } @@ -79,8 +80,7 @@ efreet_ini_parse(const char *file) f = fopen(file, "rb"); if (!f) return NULL; - if (fstat(fileno(f), &file_stat) - || file_stat.st_size < 1) + if (fstat(fileno(f), &file_stat) || file_stat.st_size < 1) { fclose(f); return NULL; @@ -274,7 +274,7 @@ efreet_ini_save(Efreet_Ini *ini, const char *file) EAPI int efreet_ini_section_set(Efreet_Ini *ini, const char *section) { - if (!ini || !ini->data || !section) return 0; + if (!ini || !(ini->data) || !section) return 0; ini->section = ecore_hash_get(ini->data, section); return (ini->section ? 1 : 0);