From 9c264fa0288a463f0587e2423c04c8b1403f264c Mon Sep 17 00:00:00 2001 From: Tom Hacohen Date: Tue, 24 May 2016 19:14:59 +0100 Subject: [PATCH] Eo: Fix issue of too many unrefs in some cases. This problem was that because the refcount is now shared between the parent and the programmer in some cases we would get a double unref. An example way of triggering it is creating a button and putting it in a box. The box has a callback registered that when the button is deleted it would delete itself too. The problem is that the delete callback is called the button is removed from the box thus causing the box to unref it again (because of the parent), although the refcount was already accounted for. There is another more convoluted scenario that I have yet to fix. Thanks to raster for reporting. --- src/lib/eo/eo_base_class.c | 7 +++---- src/lib/eo/eo_private.h | 2 -- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/src/lib/eo/eo_base_class.c b/src/lib/eo/eo_base_class.c index fa5222d888..1743695019 100644 --- a/src/lib/eo/eo_base_class.c +++ b/src/lib/eo/eo_base_class.c @@ -488,8 +488,9 @@ _eo_base_parent_set(Eo *obj, Eo_Base_Data *pd, Eo *parent_id) pd->parent, obj); } - /* Only unref if we don't have a new parent instead. */ - if (!parent_id) + /* Only unref if we don't have a new parent instead and we are not at + * the process of deleting the object.*/ + if (!parent_id && !eo_obj->del_triggered) { _eo_unref(eo_obj); } @@ -1432,8 +1433,6 @@ _eo_base_destructor(Eo *obj, Eo_Base_Data *pd) if (pd->parent) { - /* A bit ugly, but unparent unrefs, so we need to ref before. */ - eo_ref(obj); eo_parent_set(obj, NULL); } diff --git a/src/lib/eo/eo_private.h b/src/lib/eo/eo_private.h index dd5176cb2e..3cba394896 100644 --- a/src/lib/eo/eo_private.h +++ b/src/lib/eo/eo_private.h @@ -332,8 +332,6 @@ _eo_unref(_Eo_Object *obj) obj->del_triggered = EINA_TRUE; _eo_del_internal(__FILE__, __LINE__, obj); - - obj->del_triggered = EINA_FALSE; #ifdef EO_DEBUG /* If for some reason it's not empty, clear it. */ while (obj->xrefs)