diff options
author | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2001-01-10 09:34:08 +0000 |
---|---|---|
committer | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2001-01-10 09:34:08 +0000 |
commit | 8a91331df3ef0ffac6164cd56fc56e7a5429d350 (patch) | |
tree | d10df1aeefe5d87b528c1d4d587587e5e7a0723e /ppptypes.h | |
parent | 1464352c075bd63cbe2aefd3cf5e7bbe062ddb73 (diff) |
Update a comment.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2856 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'ppptypes.h')
-rw-r--r-- | ppptypes.h | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/ppptypes.h b/ppptypes.h index bf4ea5fd9b..ba9083ea3a 100644 --- a/ppptypes.h +++ b/ppptypes.h @@ -1,7 +1,7 @@ /* ppptypes.h * Defines PPP packet types. * - * $Id: ppptypes.h,v 1.3 2001/01/10 09:07:35 guy Exp $ + * $Id: ppptypes.h,v 1.4 2001/01/10 09:34:08 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -73,14 +73,21 @@ * Protocol types for the Cisco HDLC format. * * As per the above, according to RFC 1547, these are "standard 16 bit - * Ethernet protocol type code[s]", but 0x8035 is Reverse ARP, so, unless - * Cisco SLARP looks just like Reverse ARP, that's not quite true. + * Ethernet protocol type code[s]", but 0x8035 is Reverse ARP, and + * that is (at least according to the Linux ISDN code) not the + * same as Cisco SLARP. + * * In addition, 0x2000 is apparently the Cisco Discovery Protocol, but * on Ethernet those are encapsulated inside SNAP with an OUI of * OUI_CISCO, not OUI_ENCAP_ETHER. * - * If it *is* true, that'd be lovely, as we could just use - * "ethertype()" to handle Cisco HDLC. + * Perhaps we should set up a protocol table for those protocols + * that differ between Ethernet and Cisco HDLC, and have the PPP + * code first try that table and, if it finds nothing in that + * table, call "ethertype()". (Unfortunately, that means that - + * assuming we had a Cisco SLARP dissector - said dissector were + * disabled, SLARP packets would be dissected as Reverse ARP + * packets, not as data. */ #define CISCO_SLARP 0x8035 /* Cisco SLARP protocol */ |