Wireshark-commits: [Wireshark-commits] master 8bb8d5f: [packet-iec104.c] - Add dissection for IEC 6
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 26 Apr 2019 03:47:13 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8bb8d5f86c81ff5aa40facbecc8cb14c5fce09cc
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

8bb8d5f by Chris Bontje (cbontje@xxxxxxxxx):

    [packet-iec104.c] - Add dissection for IEC 60870-5-101 Messages
    
    IEC 60870-5-101 is the traditional serial version of '104.  The headers are different but the ASDU dissection is identical.
    Changes made to the '104 dissector to accommodate '101 are as follows:
    - Added in a new protocol dissector 'iec60870_101'.  This dissector handles the '101 header and calls the ASDU dissector when required.
    - The existing '104acpi' dissector has been renamed to 'iec60870_104' to better align with the '101 addition
    - The '104asdu' protocol has been renamed to 'iec60870_asdu' in order to make it more generalized between the two variants.  Updated variable names and display filter fields as needed.
    - 3 preferences exist in the iec60870_101 dissector to allow for configurable length of the COT, ASDU Addr and IOA fields.  These are fixed their max length in '104 (2, 2 and 3 octets respectively) but are configurable in '101.
    - The ASDU dissector has been modified to accept a data parameter that contains the fixed/configurable lengths of COT, ASDU Addr and IOA fields.
    
    Bug: 15688
    Change-Id: Ib0c918a40d24967caa8588067fa9e9a240af4ca5
    Reviewed-on: https://code.wireshark.org/review/32802
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  64a56cf   macOS: Add a workaround for libbrotli.
     add  8bb8d5f   [packet-iec104.c] - Add dissection for IEC 60870-5-101 Messages


Summary of changes:
 epan/dissectors/packet-iec104.c | 652 +++++++++++++++++++++++++++++++---------
 1 file changed, 508 insertions(+), 144 deletions(-)