Ethereal-dev: [Ethereal-dev] Special thanks to Guy Harris

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Charles Wilkinson <c_s_wilkinson@xxxxxxxxxxx>
Date: Wed, 31 Aug 2005 21:03:13 +0100 (BST)
Thanks Guy I'll try your suggestions tommorow; Thanks
for the help, and the good humour with which you
treated upon this subject. BTW I agree Nortel should
have got their own ethertype, but this dissector is
only meant for an in-house solution. I work in their
r'n'd and support offices in the lovely south of
france, so IPLT shouldn't be a problem, and it is
sunny enough down here to live with the problem if it
arises:)

But since my dad is a member of the IEE I agree they
should have applied for their 'own' ethertype.
Cheers all the best and thanks 
Charles
--- Guy Harris <gharris@xxxxxxxxx> wrote:

> Charles Wilkinson wrote:
> 
> > Right having decided that the only way to be
> > understood is to send out the source
> 
> Good idea.
> 
> OK, it appears that you're special-casing the IP
> protocol number field, 
> and calling a completely separate dissector for this
> "Optical Ethernet" 
> protocol.  (I *still* think Nortel should have
> gotten a separate 
> Ethertype for this protocol....)
> 
> That's probably a reasonable way to handle this; it
> avoids having to 
> have a preference setting (unless the "IPLT"
> protocol, to which IP 
> protocol number 129 is assigned, according to
> 
> 	http://www.iana.org/assignments/protocol-numbers
> 
> is something other than "Optical Ethernet", and
> somebody contributes a 
> dissector for "IPLT").
> 
> It also appears that "hlen" is, indeed, the Optical
> Ethernet equivalent 
> of the IP header length, meaning that using "hlen -
> 26" as the payload 
> length is wrong, as per my other mail, and that the
> total length field 
> is used to set the length of the tvbuff, as per my
> other mail, so try 
> setting next_tvb with
> 
> 	next_tvb = tvb_new_subset(tvb, 26, -1, -1);
> 
> > There are still 5 bytes that aren't dissected
> > but that shouldn't matter as long as I create a
> new
> > tvb with the right length
> 
> That's the problem - you probably weren't doing
> that.
> 
> _______________________________________________
> Ethereal-dev mailing list
> Ethereal-dev@xxxxxxxxxxxx
>
http://www.ethereal.com/mailman/listinfo/ethereal-dev
> 



		
___________________________________________________________ 
To help you stay safe and secure online, we've developed the all new Yahoo! Security Centre. http://uk.security.yahoo.com