Wireshark mailing list archives

Re: [PATCH] dependent_frames memleak?


From: Jeff Morriss <jeff.morriss.ws () gmail com>
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 is good.

ps. welcome back!
___________________________________________________________________________
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


Current thread: