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:
- [PATCH] dependent_frames memleak? Jakub Zawadzki (Apr 02)
- Re: [PATCH] dependent_frames memleak? Jeff Morriss (Apr 02)