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

Wireshark-bugs: [Wireshark-bugs] [Bug 4141] Reassemble.c leaks memory for GLIB > 2.8

Date: Wed, 8 Aug 2012 19:18:34 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4141

--- Comment #18 from Bill Meier <wmeier@xxxxxxxxxxx> 2012-08-08 19:18:34 PDT ---
(In reply to comment #5)

>       Also note that currently for free_all_reassembled_fragments()
>       that the 'key_arg' points to se_alloc'd memory which has already
>       been freed.


> For the reassembled_table I think we'll need to change the key to be slice
> allocated 

As Anders suggested, I'm going to change the reassembled_table keys to be slice
allocated (even tho not strictly required) and then free them appropriately.

reassembled_table_init() is currently working (when freeing entries in the
hash) because the invalid keys are never actually referenced but it's never a
good thing to have dangling pointers.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.