Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 37112: /trunk/epan/ /trunk/epan/: reassemble.c

[Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>]

Backing out I can do :-)

Anders Broman wrote:
> Hi,
> Lets back it out, Jakub may have a better solution :-)
> I'll do it later today/tomorrow if no one beats me to it.
> Regards
> Anders 
>
> -----Original Message-----
> From: wireshark-dev-bounces@xxxxxxxxxxxxx [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Jeff Morriss
> Sent: den 13 maj 2011 16:12
> To: wireshark-dev@xxxxxxxxxxxxx
> Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 37112: /trunk/epan/ /trunk/epan/: reassemble.c
>
> etxrab@xxxxxxxxxxxxx wrote:
> > http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=37112
> >
> > User: etxrab
> > Date: 2011/05/12 10:19 PM
> >
> > Log:
> >  We should get rid of g_mem_chunk_alloc() as it leaks memory  
> > http://www.wireshark.org/lists/wireshark-dev/200910/msg00074.html
> >  g_slice allocing the keys should make it possible to walk the  
> > fragment table and free the fragments once they are g_slice_alloced.
> >  It remains fo figure out how to do that.
>
> This is causing plenty of fuzz failures (see bug 5915).
>
> I'm out of time to look at it for now.