hermet pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=3f0963f7b020bd83da348615e7582ebef1a4a5d0

commit 3f0963f7b020bd83da348615e7582ebef1a4a5d0
Author: Sungtaek Hong <[email protected]>
Date:   Thu Dec 8 13:26:55 2016 +0900

    efreet_desktop: fix potential error in efreet_desktop_cache_create
    
    Summary:
     - old_file_ids is freed but not set as NULL.
       If it goes to error code, old_file_ids will be freed again.
    
    Reviewers: jpeg, cedric, Hermet
    
    Reviewed By: Hermet
    
    Subscribers: conr2d
    
    Differential Revision: https://phab.enlightenment.org/D4467
---
 src/bin/efreet/efreet_desktop_cache_create.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/bin/efreet/efreet_desktop_cache_create.c 
b/src/bin/efreet/efreet_desktop_cache_create.c
index c6b1877..fe4a476 100644
--- a/src/bin/efreet/efreet_desktop_cache_create.c
+++ b/src/bin/efreet/efreet_desktop_cache_create.c
@@ -504,6 +504,7 @@ main(int argc, char **argv)
     {
         eina_hash_free(old_file_ids->hash);
         free(old_file_ids);
+        old_file_ids = NULL;
     }
 
     eina_hash_free(file_ids);

-- 


Reply via email to