ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 37112: /trunk/epan/ /trunk/epan/: re

From: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>
Date: Fri, 13 May 2011 10:23:28 -0400
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.