On Mon, Apr 30, 2007 at 09:13:48AM -0700, Jeremy Allison wrote: > Nope - looks good. What you missed is that there > are other areas in this loop that return without > freeing current_ace, so your fix is good but not > quite general enough :-). > > I'll add the extra SAFE_FREE's needed :-).
I'm definately having a bad day :-). There are no extra SAFE_FREE's needed as we've added current_ace to the linked list.... Doh ! :-). Your fix is perfectly correct, thanks ! Jeremy. -- To unsubscribe from this list go to the following URL and read the instructions: https://lists.samba.org/mailman/listinfo/samba
