Wireshark · Ethereal-dev: Re: [Ethereal-dev] [Patch][packet-enip.c]

["Magnus Hansson" <mah@xxxxxx>]

Sorry for that.
Attached is a new diff against last version in CVS.
(Show_cdf() shall not be called for NOP packets).

Changes:
* Added decoding of Transport type/trigger
* Updated service code's text to match specification
* Added new vendor IDs from ODVA
* Added service etc to info column, formatted info column for easier overview
* Added actual time out calculation for Forward close, Unconnected send
* Fixed bug, port not shown for extended addresses
* Added Network Segment to EPATH decode
* NOP packets not decoded as they contain Common data Format

Regards
Magnus Hansson

-----Original Message-----
From: Guy Harris [mailto:guy@xxxxxxxxxxxx]
Sent: den 3 februari 2004 20:11
To: Ethereal development
Subject: Re: [Ethereal-dev] [Patch][packet-enip.c]


On Tue, Feb 03, 2004 at 04:20:07PM +0100, Magnus Hansson wrote:
> Attached is a patch for the packet-enip.c (EtherNet/IP industrial
> protocol).

...but not for the current version in the Ethereal CVS repository; in
the current CVS version, the

	switch( encap_cmd )
	{

		...

	}

code already handles NOPs by calling "show_cdf()", for example.

_______________________________________________
Ethereal-dev mailing list
Ethereal-dev@xxxxxxxxxxxx
http://www.ethereal.com/mailman/listinfo/ethereal-dev

Attachment: packet-enip.diff
Description: packet-enip.diff