oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

cmdline: bunch of fixes related to config file selection

Browse Source 
- fix potential memory leak of system_path

  'Potential' because currently g_get_system_config_dirs()
  returns single entry on Windows, but that might change.

- remove incorrect g_free() call

  It's not required at all because
  g_get_system_config_dirs() returns GLib owned memory.

- remove extra semicolon
This commit is contained in:
Denis Krjuchkov 2012-10-17 23:47:21 +06:00 committed by Max Kellermann
parent a9a5907a0f
commit 01a45a53aa
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/cmdline.c
View File

@ -213,12 +213,12 @@ parse_cmdline(int argc, char **argv, struct options *options,
if(g_file_test(system_path, if(g_file_test(system_path,
G_FILE_TEST_IS_REGULAR)) { G_FILE_TEST_IS_REGULAR)) {
ret = config_read_file(system_path,error_r); ret = config_read_file(system_path,error_r);
g_free(system_path);
break; break;
} } else
++i;; g_free(system_path);
++i;
} }
g_free(system_path);
g_free(&system_config_dirs);
} }
#else /* G_OS_WIN32 */ #else /* G_OS_WIN32 */
char *path2; char *path2;