edje: fix edje_part_helpers refcounting

Summary:
the reference from efl_reuse was forgotten & the parent relation was not
correctlty setted, which led to the fact that NOREF was never emitted.
This caused that thte object never really was destructed probebly, and
thus the del_interceptor_cb was not executed, and the object simply
leaked.

The test checks that those properties are correctly set, additionally a
error is printed in the efl code when a part has not the expected
reference properties. This also enforces errors when users are doing
wrong things with objects returned by efl_part.

Reviewers: ManMower, zmike

Reviewed By: zmike

Subscribers: cedric, #committers

Tags: #efl

Differential Revision: https://phab.enlightenment.org/D6254

1 files changed, 6 insertions, 0 deletions

diff --git a/src/lib/efl/interfaces/efl_interfaces_main.c b/src/lib/efl/interfaces/efl_interfaces_main.c
index d9b10e89d2..f5b8634d68 100644
--- a/src/lib/efl/interfaces/efl_interfaces_main.c
+++ b/src/lib/efl/interfaces/efl_interfaces_main.c
@@ -96,6 +96,12 @@ efl_part(const Eo *obj, const char *name)
    if (!r) return NULL;
 
    efl_event_callback_add(r, EFL_EVENT_NOREF, _noref_death, NULL);
+
+   //ensure that the parts that we have here are never leaked
+   //by checking theire references and ownership details
+   EINA_SAFETY_ON_NULL_RETURN_VAL(efl_parent_get(r), r);
+   EINA_SAFETY_ON_FALSE_RETURN_VAL(efl_ref_count(r) == 1, r);
+
    ___efl_auto_unref_set(r, EINA_TRUE);
 
    return efl_ref(r);