lib: talloc: Allow destructors to reparent the object they're called on.
authorJeremy Allison <jra@samba.org>
Wed, 4 Mar 2015 01:02:47 +0000 (17:02 -0800)
committerStefan Metzmacher <metze@samba.org>
Tue, 10 Mar 2015 09:55:40 +0000 (10:55 +0100)
commit024dd8368da2b2e01754f9ca7339a24b47ba0dd7
tree050fddd730b21626ac69770b3767700dec700599
parent0b702df708b9496ab6408d1bb6a789f44c3bb82b
lib: talloc: Allow destructors to reparent the object they're called on.

If a destructor returns failure (-1) when freeing a child, talloc
must then reparent the child.

Firstly it tries the owner of any reference, next the parent of the
current object calling _talloc_free_children_internal(), and finally
the null context in the last resort.

If a destructor reparented its own object, which can be a very
desirable thing to do (a destructor can make a decision it isn't
time to die yet, and as the parent may be going away it might
want to move itself to longer-term storage) then this new parent
gets overwritten by the existing reparenting logic.

This patch checks when freeing a child if it already reparented
itself, and if it did doesn't then overwrite the new parent.

Makes destructors more flexible.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ira Cooper <ira@samba.org>
(cherry picked from commit cc4e5481ea060db7f6d8a83619d859b2e002eb90)
lib/talloc/talloc.c