Wireshark · Wireshark-dev: Re: [Wireshark-dev] [PATCH] dependent_frames memleak?

Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] [PATCH] dependent_frames memleak?

From: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>

Date: Mon, 02 Apr 2012 21:43:33 -0400

On 04/02/2012 03:07 PM, Jakub Zawadzki wrote:

Hi,

Looking at r41216 I think dependent_frames list should be always freed,
not only in add_packet_to_packet_list().

Can someone review attached patch? Jeff?

Oops, yes, you're right--it leaks memory! :-( And I think your patch isgood.


ps. welcome back!

References:
- [Wireshark-dev] [PATCH] dependent_frames memleak?
  - From: Jakub Zawadzki

Prev by Date: [Wireshark-dev] [PATCH] dependent_frames memleak?
Next by Date: Re: [Wireshark-dev] [Wireshark-commits] rev 41790: /trunk/ /trunk/: configure.in
Previous by thread: [Wireshark-dev] [PATCH] dependent_frames memleak?
Next by thread: Re: [Wireshark-dev] [Wireshark-commits] rev 41790: /trunk/ /trunk/: configure.in
Index(es):
- Date
- Thread

© Wireshark Foundation · Privacy Policy

Facebook·Mastodon·Twitter·YouTube