Wireshark-bugs: [Wireshark-bugs] [Bug 5130] new_packet_list: crash in add_byte_views from decryp

Date: Fri, 27 Jan 2012 09:22:27 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5130

Kundok Park <kundokpark@xxxxxxxxx> changed:

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

--- Comment #5 from Kundok Park <kundokpark@xxxxxxxxx> 2012-01-27 09:22:24 PST ---
Created attachment 7737
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=7737
Patch to make filter change crash disappear

I experienced crash problems with 6LoWPAN captures and the symptom seems to be
identical to the description in this bug item.
With the modification in the attached patch (patch_on_rev_38625.patch), my
crash problems disappeared.
The cause of the crash I saw was that the add_byte_views() function in
main_proto_draw.c relies on output from previous dissector run while the
function may eventually trigger dissector to run again which wipes out the
previous output.
The patch copies the output of the dissector before calling add_byte_tab() so
that even when add_byte_tab() updates the dissector output, the loop continues
with previous dissector output.

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