tasn pushed a commit to branch master.

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

commit 392df9479f816a641fabbf333326bdab4b9cfc28
Author: Subodh Kumar <s7158.ku...@samsung.com>
Date:   Wed Nov 18 10:52:34 2015 +0000

    Evas text: Fix memory leak
    
    Summary:
    Fix memory leak
    
    Position is getting returned without
    freeing the temporary logical item list.
    
    @fix
    
    Test Plan: NA
    
    Reviewers: tasn, raster
    
    Reviewed By: raster
    
    Subscribers: raster, cedric
    
    Differential Revision: https://phab.enlightenment.org/D3345
---
 src/lib/evas/canvas/evas_object_text.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/lib/evas/canvas/evas_object_text.c 
b/src/lib/evas/canvas/evas_object_text.c
index 2cf1a54..97c0711 100644
--- a/src/lib/evas/canvas/evas_object_text.c
+++ b/src/lib/evas/canvas/evas_object_text.c
@@ -271,6 +271,7 @@ _evas_object_text_last_up_to_pos(const Evas_Object *eo_obj,
 {
    Evas_Object_Protected_Data *obj = eo_data_scope_get(eo_obj, 
EVAS_OBJECT_CLASS);
    Evas_Object_Text_Item *it;
+   int pos = 0;
 
 #ifdef BIDI_SUPPORT
    /*FIXME: not very efficient, sort the items arrays. */
@@ -291,15 +292,17 @@ _evas_object_text_last_up_to_pos(const Evas_Object 
*eo_obj,
           {
              if ((x <= cx) && (cx < x + it->adv))
                {
-                  return it->text_pos + ENFN->font_last_up_to_pos(ENDT,
+                  pos = it->text_pos + ENFN->font_last_up_to_pos(ENDT,
                         o->font,
                         &it->text_props,
                         cx - x,
                         cy);
+                  break;
                }
              x += it->adv;
           }
         eina_list_free(logical_it);
+        return pos;
      }
    else
 #endif

-- 


Reply via email to