diff options
author | Chris Michael <cp.michael@samsung.com> | 2013-07-08 11:25:59 +0100 |
---|---|---|
committer | Chris Michael <cp.michael@samsung.com> | 2013-07-08 11:25:59 +0100 |
commit | 57924e5824b4f38d2cddf4321d1da35798ee423e (patch) | |
tree | 5b598c51d5dcf04473cce4a27f1177a9b90ab1a7 /src/lib/efreet/efreet_menu.c | |
parent | aeb4b7f7b90def0d92c566e9a1288a2dce665bec (diff) |
Fix memleak reported by Coverity: Close the descriptor returned by
popen if we are going to error out.
NB: Fixes Coverity CID1039184
Signed-off-by: Chris Michael <cp.michael@samsung.com>
Diffstat (limited to '')
-rw-r--r-- | src/lib/efreet/efreet_menu.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/lib/efreet/efreet_menu.c b/src/lib/efreet/efreet_menu.c index 098f8c79c8..88559c272d 100644 --- a/src/lib/efreet/efreet_menu.c +++ b/src/lib/efreet/efreet_menu.c | |||
@@ -462,6 +462,7 @@ efreet_menu_kde_legacy_init(void) | |||
462 | if (!fgets(buf, sizeof(buf), f)) | 462 | if (!fgets(buf, sizeof(buf), f)) |
463 | { | 463 | { |
464 | ERR("Error initializing KDE legacy information"); | 464 | ERR("Error initializing KDE legacy information"); |
465 | pclose(f); | ||
465 | return 0; | 466 | return 0; |
466 | } | 467 | } |
467 | s = buf; | 468 | s = buf; |