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

[Anders Broman <anders.broman@xxxxxxxxxxxx>]

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.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe