[pulseaudio-commits] src/modules

[Tanu Kaskinen]

 src/modules/module-augment-properties.c |   13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

New commits:
commit e5588e08a132f0c82efc6b4e53889900e1ed5035
Author: Tanu Kaskinen <tanuk at iki.fi>
Date:   Thu Dec 7 01:26:29 2017 +0200

    augment-properties: fix a memory leak
    
    If the desktop file is not found, fn was not being freed after the last
    loop iteration.
    
    CID: 1462477

diff --git a/src/modules/module-augment-properties.c b/src/modules/module-augment-properties.c
index f0584328..3eb503aa 100644
--- a/src/modules/module-augment-properties.c
+++ b/src/modules/module-augment-properties.c
@@ -137,10 +137,13 @@ static char * find_desktop_file_in_dir(struct rule *r, const char *desktop_file_
     pa_assert(st);
 
     fn = pa_sprintf_malloc("%s" PA_PATH_SEP "%s.desktop", desktop_file_dir, r->process_name);
-    if (stat(fn, st) == 0) {
+    if (stat(fn, st) == 0)
         return fn;
-    } else {
+
+    pa_xfree(fn);
+
 #ifdef DT_DIR
+    {
         DIR *desktopfiles_dir;
         struct dirent *dir;
 
@@ -152,18 +155,20 @@ static char * find_desktop_file_in_dir(struct rule *r, const char *desktop_file_
                     || pa_streq(dir->d_name, ".."))
                     continue;
 
-                pa_xfree(fn);
                 fn = pa_sprintf_malloc("%s" PA_PATH_SEP "%s" PA_PATH_SEP "%s.desktop", desktop_file_dir, dir->d_name, r->process_name);
 
                 if (stat(fn, st) == 0) {
                     closedir(desktopfiles_dir);
                     return fn;
                 }
+
+                pa_xfree(fn);
             }
             closedir(desktopfiles_dir);
         }
-#endif
     }
+#endif
+
     return NULL;
 }