Good morning, On Thu, Aug 31, 2017 at 12:17:16AM -0700, Carsten Haitzler wrote: > raster pushed a commit to branch master. > > http://git.enlightenment.org/core/efl.git/commit/?id=65d2dfc8925a183bf7f2709677d87edcd759e9c3 > > commit 65d2dfc8925a183bf7f2709677d87edcd759e9c3 > Author: Carsten Haitzler (Rasterman) <[email protected]> > Date: Thu Aug 31 16:12:57 2017 +0900 > > elm focus manager - dont leak child lists when updating focus order > > this should fix T5800 > > @fix > --- > src/lib/elementary/elm_box.c | 1 + > src/lib/elementary/elm_grid.c | 1 + > src/lib/elementary/elm_table.c | 1 + > 3 files changed, 3 insertions(+) > > diff --git a/src/lib/elementary/elm_box.c b/src/lib/elementary/elm_box.c > index b6a617dd2b..1c77ccd882 100644 > --- a/src/lib/elementary/elm_box.c > +++ b/src/lib/elementary/elm_box.c > @@ -29,6 +29,7 @@ _focus_order_flush(Eo *obj, Elm_Box_Data *pd EINA_UNUSED) > Eina_List *order = evas_object_box_children_get(wpd->resize_obj); > > efl_ui_focus_manager_calc_update_order(wpd->focus.manager, obj, order); > + eina_list_free(order);
Noep, you are passing the ownership of the list to the focus manager, you should NOT use the list after that anymore, esp. not freeing it, same for the other usages ... :) Greetings Marcel Hollerbach > } > > static void * > diff --git a/src/lib/elementary/elm_grid.c b/src/lib/elementary/elm_grid.c > index 7c85648882..913ffe3b2f 100644 > --- a/src/lib/elementary/elm_grid.c > +++ b/src/lib/elementary/elm_grid.c > @@ -20,6 +20,7 @@ _focus_order_flush(Eo *obj) > Eina_List *order = evas_object_grid_children_get(wpd->resize_obj); > > efl_ui_focus_manager_calc_update_order(wpd->focus.manager, obj, order); > + eina_list_free(order); > } > > > diff --git a/src/lib/elementary/elm_table.c b/src/lib/elementary/elm_table.c > index 48e6dfa62a..48bc137b23 100644 > --- a/src/lib/elementary/elm_table.c > +++ b/src/lib/elementary/elm_table.c > @@ -21,6 +21,7 @@ _focus_order_flush(Eo *obj) > Eina_List *order = evas_object_table_children_get(wpd->resize_obj); > > efl_ui_focus_manager_calc_update_order(wpd->focus.manager, obj, order); > + eina_list_free(order); > } > > EOLIAN static Eina_Bool > > -- > > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
