Wireshark-bugs: [Wireshark-bugs] [Bug 4722] New: PPP/USB dissection doesn't work if USB frames a

Date: Wed, 28 Apr 2010 15:20:51 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4722

           Summary: PPP/USB dissection doesn't work if USB frames are of
                    interface class, "Vendor Specific"
           Product: Wireshark
           Version: SVN
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Medium
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: christopher.maynard@xxxxxxxxx


Chris Maynard <christopher.maynard@xxxxxxxxx> changed:

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

Created an attachment (id=4586)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=4586)
Patch to allow PPP/USB dissection to work if USB frames are of interface class,
"Vendor Specific"

Build Information:
SVN 32591
--
When I submitted the USB and PPP changes, I had not previously tested with a
full insertion of a particular USB device.  Now that I have, I've found that
what used to be reported as an unknown interface class is now reported as a
vendor specific interface class.  Thus, in order for PPP/USB dissection to
work, the ppp dissector must register for that class as well.  The attached
trivial patch does just that.

However ... this brings up another point.  Currently there are 19 IF_CLASS_'s
defined in packet-ppp.h.  Should we register for all of them instead of just
these 2?  I don't have other USB devices to test with, so I don't know if
they're needed or not.  For now, I'm just adding what I need.

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