kimcinoo
/
efl
forked from enlightenment/efl
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

From: Jeonghyun Yun <jh0506.yun@samsung.com> 

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 ? 



SVN revision: 55841
This commit is contained in:
Jeonghyun Yun 2011-01-04 10:05:27 +00:00 committed by Carsten Haitzler
parent 6e782c1702
commit dda2eae31a
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
legacy/elementary/src/lib/elm_gengrid.c
View File

@ -242,6 +242,7 @@ static void _item_hilight(Elm_Gengrid_Item *item);
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);
}
/**