Wireshark-bugs: [Wireshark-bugs] [Bug 3313] BACnet MS/TP checksums

Date: Sun, 8 Mar 2009 11:05:43 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3313


Steve Karg <steve@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2826|review_for_checkin?         |
               Flag|                            |
Attachment #2826 is|0                           |1
           obsolete|                            |
   Attachment #2830|                            |review_for_checkin?
               Flag|                            |




--- Comment #2 from Steve Karg <steve@xxxxxxxxx>  2009-03-08 11:05:40 PDT ---
Created an attachment (id=2830)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=2830)
packet-mstp.c.patch

Thank you for the direction, Jaap.
1. Changed to use SET_GENERATED instead of SET_HIDDEN.
2. Added Checksum [Good: True], [Bad: False] combos.
Visually tested with good/bad BACnet MS/TP captures.  Fuzz tested with BACnet
MS/TP captures.


-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.