From 57924e5824b4f38d2cddf4321d1da35798ee423e Mon Sep 17 00:00:00 2001 From: Chris Michael Date: Mon, 8 Jul 2013 11:25:59 +0100 Subject: [PATCH] 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 --- src/lib/efreet/efreet_menu.c | 1 + 1 file changed, 1 insertion(+) 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) if (!fgets(buf, sizeof(buf), f)) { ERR("Error initializing KDE legacy information"); + pclose(f); return 0; } s = buf;