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

Wireshark-bugs: [Wireshark-bugs] [Bug 6573] tvbuff_t values leak if they have a child tvbuff_t (

Date: Mon, 12 Dec 2011 14:09:32 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6573

--- Comment #4 from Bill Meier <wmeier@xxxxxxxxxxx> 2011-12-12 17:09:30 EST ---
Thanks for the patch with the suggested fixes !

You've obviously spent some time looking at and understanding the tvbuff code.

That being said, upon spending some time looking at the tvbuff.c code myself, I
came to the conclusion that maybe it's not really necessary to keep
usage_counts & used_in lists at all given the way Wireshark uses tvbuffs.

If so, then the code can be simplified and we don't have to go thru contortions
to try to get everything correct.

I've posted an Email in wireshark-dev and attached a revised version of
tvbuff.c and etc which implement my ideas.

http://www.wireshark.org/lists/wireshark-dev/201112/msg00064.html

If you have time to look at the code, any comments are welcome.

Bill Meier

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