On Aug 14, 2012, at 12:49 AM, Andrei Emeltchenko wrote:

> Hi Joerg,
> 
> On Mon, Aug 06, 2012 at 05:37:35PM +0200, Joerg Mayer wrote:
>> Hello,
>> 
>> can you please open a bug at bugs.wireshark.org and attach the patch there?
> 
> https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7633
> 
>> Does your patch distinguish between an 802.3/LLC/SNAP encapsulated frame
>> of length 3 and Ethertype 3?
> 
> Sorry did not get what you mean here.

As per my comment in the bug:

0x0003 is not an Ethernet type, so the EAPOL dissector should *NOT* be 
registered in the "ethertype" table with a value of 0x0003.

(Nothing less than 0x0600 can be a valid Ethernet type; see section 3.2.6 
"Length/Type field" of IEEE Std 802.3-2008, for example.  Values in the 
type/length field of an Ethernet packet that are less than or equal to 0x05DC - 
1500 - are *length* values, not *type* values, so if an Ethernet packet had a 
value of 0x0003, it'd be a length value, hence the packet would be an 
802.3/LLC-encapsulated frame with a length of 3.)

Instead, it's a protocol ID in the space of SNAP protocol IDs for the OUI value 
00:19:58, i.e. OUI_BLUETOOTH, so what you want to do is to:

        register a dissector table for the OUI value OUI_BLUETOOTH - for an 
example of how to do this, see proto_register_cisco_oui() in 
epan/dissectors/packet-cisco-oui.c;

        in that dissector table, register the EAPOL dissector in *THAT* 
dissector table with a protocol ID of 0x0003.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

Reply via email to