stefan pushed a commit to branch master.

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

commit c9e761da548342aa6271352de1d015ed2ebf42a1
Author: Stefan Schmidt <[email protected]>
Date:   Mon Dec 12 16:18:18 2016 +0100

    ethumb: fix resource leak introduced in file path sanitization rework
    
    This extra snaitized path assign path does not make sense and leads to a 
leak
    in the end. We already have sanitized_path and handle that correctly.
    
    Fixes 62a0c41fd3bf8796efec55db52395f13a78fb27f
    
    CID: 1366925
---
 src/lib/ethumb/ethumb.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/lib/ethumb/ethumb.c b/src/lib/ethumb/ethumb.c
index 7b8f4f0..b66c3f4 100644
--- a/src/lib/ethumb/ethumb.c
+++ b/src/lib/ethumb/ethumb.c
@@ -1098,7 +1098,6 @@ ethumb_thumb_path_set(Ethumb *e, const char *path, const 
char *key)
    else
      {
         sanitized_path = eina_file_path_sanitize(path);
-        path = eina_file_path_sanitize(path);
         eina_stringshare_replace(&e->thumb_path, sanitized_path);
         eina_stringshare_replace(&e->thumb_key, key);
         free(sanitized_path);

-- 


Reply via email to