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

[Jeff Morriss <jeff.morriss.ws () gmail com>]

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 () wireshark org [mailto:wireshark-dev-bounces () wireshark org] On Behalf Of Jeff Morriss
> Sent: den 13 maj 2011 16:12
> To: wireshark-dev () wireshark org
> Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 37112: /trunk/epan/ /trunk/epan/: reassemble.c
>
> etxrab () wireshark org 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.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:wireshark-dev-request () wireshark org?subject=unsubscribe