spoolss talloc problem

Andreas Schneider asn at samba.org
Mon Aug 8 07:13:12 MDT 2011


On Monday 08 August 2011 18:42:16 tridge at samba.org wrote:
> Hi Simo,

Hi Tridge,

> http://git.samba.org/?p=tridge/samba.git;a=commitdiff;h=d24af812d76059c2f3b
> 917b791c77175b388811a
> 
> it just ensures that a talloc chunk that is part way through being
> freed doesn't have an invalid tc->prev pointer.
> 
> I'm still testing the fix, but it does seem to fix your issue. It is
> also faster than the existing code (as we no longer call the expensive
> talloc_parent_chunk() call when we don't need it).

I've tested the fix here and I wasn't able to reprodue the problem. However I 
have a patch which fixes the problem in a different way and makes sure the 
memory context doesn't get so huge.

http://git.samba.org/?p=asn/samba.git;a=commitdiff;h=7dbe148c0d43fe662490f940b27fc561c069d491

Feel free to push both :)

Cheers,


	-- andreas

-- 
Andreas Schneider                   GPG-ID: F33E3FC6
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list