[prev in list] [next in list] [prev in thread] [next in thread] 

List:       enlightenment-devel
Subject:    [E-devel]   [Patch] small patch for elm_gengrid.c
From:       Jeonghyun Yun <jh0506.yun () samsung ! com>
Date:       2010-12-31 2:24:43
Message-ID: 017801cba891$e2790f60$a76b2e20$%yun () samsung ! com
[Download RAW message or body]

Hello All.

I re-attach patch.

I found one small bug in elm_gengrid.c

"item" is used after "free(item)" in elm_gengrid_item_del() function.

Can anybody check about this ? 

Thanks.

["elm_gengrid.patch.txt" (text/plain)]

Index: src/lib/elm_gengrid.c
===================================================================
--- src/lib/elm_gengrid.c	(revision 55786)
+++ src/lib/elm_gengrid.c	(working copy)
@@ -242,6 +242,7 @@ static void      _item_hilight(Elm_Gengrid_Item *i
 static void      _item_unrealize(Elm_Gengrid_Item *item);
 static void      _item_select(Elm_Gengrid_Item *item);
 static void      _item_unselect(Elm_Gengrid_Item *item);
+static void      _calc_job(void *data);
 static void      _on_focus_hook(void        *data,
                                 Evas_Object *obj);
 static Eina_Bool _item_multi_select_up(Widget_Data *wd);
@@ -1177,6 +1178,8 @@ _item_del(Elm_Gengrid_Item *item)
      item->tooltip.del_cb((void *)item->tooltip.data, item->base.widget, item);
    item->wd->walking -= item->walking;
    item->wd->count--;
+   if (item->wd->calc_job) ecore_job_del(item->wd->calc_job);
+   item->wd->calc_job = ecore_job_add(_calc_job, item->wd);
    elm_widget_item_del(item);
 }
 
@@ -1859,9 +1862,6 @@ elm_gengrid_item_del(Elm_Gengrid_Item *item)
      }
 
    _item_del(item);
-
-   if (item->wd->calc_job) ecore_job_del(item->wd->calc_job);
-   item->wd->calc_job = ecore_job_add(_calc_job, item->wd);
 }
 
 /**


------------------------------------------------------------------------------
Learn how Oracle Real Application Clusters (RAC) One Node allows customers
to consolidate database storage, standardize their database environment, and, 
should the need arise, upgrade to a full multi-node Oracle RAC database 
without downtime or disruption
http://p.sf.net/sfu/oracle-sfdevnl

_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[prev in list] [next in list] [prev in thread] [next in thread]