diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2009-09-22 10:32:10 +0930 |
---|---|---|
committer | Volker Lendecke <vl@samba.org> | 2009-09-22 10:50:28 +0200 |
commit | 455d44d9618afdc1d1f4198ae9f64ccc56dee62f (patch) | |
tree | c17e653e265dc903df3169f2b0ae02da16bdea7b /source3/utils/net_ads_gpo.c | |
parent | f54ef5f20fa4809847c89ce0c393111807ca31a2 (diff) | |
download | samba-455d44d9618afdc1d1f4198ae9f64ccc56dee62f.tar.gz samba-455d44d9618afdc1d1f4198ae9f64ccc56dee62f.tar.bz2 samba-455d44d9618afdc1d1f4198ae9f64ccc56dee62f.zip |
lib/tevent: a cleaner fix for be4ac227842530d484659f2db683453366326d8b segv
Revert 23abcd2318c69753aa2a144e1dc0f9cf9efdb705 and fix logic bug.
The current code loops through the event contexts, when it sees a different
one, it notifies the current one (ev) and updates ev to point to the new one.
This is dumb, because:
(1) ev starts as NULL, so this code crashes, and
(2) The final context will not be notified.
The correct fix for this is to update ev to the new one, then notify it.
Volker's fix works because we currently always have one event context.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'source3/utils/net_ads_gpo.c')
0 files changed, 0 insertions, 0 deletions