devilhorns pushed a commit to branch master.

commit d92edb51a4cc425d79973d2266f6695aae72abbd
Author: Chris Michael <[email protected]>
Date:   Fri Jun 28 13:39:45 2013 +0100

    Fix memleak reported by klockwork:
    
    If a jpeg image is rotated and we have allocated space for ptr_rotate,
    then we should free it Always ... Not Only if the degrees are
    180...because we allocate it always
    
    Signed-off-by: Chris Michael <[email protected]>
---
 src/modules/evas/loaders/jpeg/evas_image_load_jpeg.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/modules/evas/loaders/jpeg/evas_image_load_jpeg.c 
b/src/modules/evas/loaders/jpeg/evas_image_load_jpeg.c
index c3e3a40..96de9c6 100644
--- a/src/modules/evas/loaders/jpeg/evas_image_load_jpeg.c
+++ b/src/modules/evas/loaders/jpeg/evas_image_load_jpeg.c
@@ -1083,11 +1083,11 @@ done:
                        to += hw;
                     }
                }
-             if (ptr_rotate)
-               {
-                  free(ptr_rotate);
-                  ptr_rotate = NULL;
-               }
+          }
+        if (ptr_rotate)
+          {
+             free(ptr_rotate);
+             ptr_rotate = NULL;
           }
         if (region)
           {

-- 

------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev

Reply via email to