Wireshark-bugs: [Wireshark-bugs] [Bug 7224] Custom model for packet details
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Fri, 4 May 2012 01:03:20 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7224

Jakub Zawadzki <darkjames-ws@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #8368|                            |review_for_checkin?
              Flags|                            |

--- Comment #1 from Jakub Zawadzki <darkjames-ws@xxxxxxxxxxxx> 2012-05-04 01:03:19 PDT ---
Created attachment 8368
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=8368
Clear treestore before freeing edt

When building current data for packet details treeview we store two things.
- Generated string with item label
- Pointer to node field_info structure

After epan_dissect_{free, cleanup} pointer to field_info node is no longer
valid so we should store before freeing.

This is not *critial* in current version cause even if gdk/gtk thread fire up 
cell renderer is using generated string.

I can commit it by myself, just need someone to review ;-)

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