1
0
mirror of https://github.com/samba-team/samba.git synced 2025-01-11 05:18:09 +03:00

tevent: don't force the nesting flag to false in the destructor

the tevent destructor is called in tevent_re_initialise(), to reset
the event context back to its original state for creating child
processes. We need the nesting flag to stay the same

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Fri Aug 12 05:18:25 CEST 2011 on sn-devel-104
This commit is contained in:
Andrew Tridgell 2011-08-12 11:56:28 +10:00
parent f444eeb51c
commit 4aba775219

View File

@ -185,9 +185,13 @@ int tevent_common_context_destructor(struct tevent_context *ev)
tevent_cleanup_pending_signal_handlers(se);
}
/* clean up nesting or we get an abort when nesting
* is not allowed. -- SSS */
ev->nesting.allowed = false;
/* removing nesting hook or we get an abort when nesting is
* not allowed. -- SSS
* Note that we need to leave the allowed flag at its current
* value, otherwise the use in tevent_re_initialise() will
* leave the event context with allowed forced to false, which
* will break users that expect nesting to be allowed
*/
ev->nesting.level = 0;
ev->nesting.hook_fn = NULL;
ev->nesting.hook_private = NULL;