Wireshark-commits: [Wireshark-commits] rev 37539: / /trunk/epan/dissectors/: Makefile.common packet
Date: Sat, 04 Jun 2011 15:45:56 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=37539

User: etxrab
Date: 2011/06/04 08:45 AM

Log:
 From Michael Mann:
 I made the following improvements to the Modbus/TCP dissector:
 
 1. Implemented protocol to the latest specification (v1.1b).  See
 http://www.modbus.org/specs.php.
 2. Upgraded to "tcp_dissect_pdus" instead of having the dissector do it
 manually.  This also provides TCP packet reassembly support.
 3. Removed support for UDP port 502 because it's not supported by the protocol
 specification.  I believe "Decode As..." could be used in its place.
 4. Added dissector support for the following function codes:
   a) 8 - Diagnostics
   b) 11 - Event Counter
   c) 12 - Event Log
   d) 43 - Encapsulation Transport (mostly for 43/14)
 5. Removed support for function codes not in the protocol specification. None
 of them were really being parsed, they just offered a "name" for the function
 code.
 6. Moved protocol #defines to header file for access from other dissectors.  I
 plan to have other dissectors use this, but one patch at a time.
 7. Created "modbus" dissector that is accessible to other dissectors.
 8. Renamed base "display filter name" to reflect PROTOABBREV.
 
 I removed defines for value_strings, I think it's better to export the vaöue strings if needed.
 
 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5923

Directory: /trunk/epan/dissectors/
  Changes    Path               Action
  +1 -0      Makefile.common    Modified
  +946 -448  packet-mbtcp.c     Modified
  +101 -0    packet-mbtcp.h     Added