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 3290] New: proto_add_item() doesn't test for bound errors

Date: Fri, 27 Feb 2009 22:21:59 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3290

           Summary: proto_add_item() doesn't test for bound errors if tree
                    is NULL
           Product: Wireshark
           Version: SVN
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Major
          Priority: Low
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: dgautheron@xxxxxxxx



didier gautheron <dgautheron@xxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2787|                            |review_for_checkin?
               Flag|                            |


Created an attachment (id=2787)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=2787)
 always test for bound errors in proto_add_item()

Build Information:
tested with SVN r27565.
--
Hi,

It's an issue if the error is in the last proto_add_item() call, the packet
throw an exception if selected in the first pane but 'malformed' filter fails.  

I suspect it's bug #3285 cause.

Attached an other example with AFP and a patch.

I'm not sure about the patch, it works for me though.


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