Wireshark-commits: [Wireshark-commits] master 1859b35: PROFINET: Fix related to redundancy and PNIO

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 20 Nov 2019 14:33:02 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1859b35f5a7231ba538263c1dab6f6ad221cb91f
Submitter: "Roland Knall <rknall@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

1859b35 by Basak Kalfa (basakkalfa@xxxxxxxxx):

    PROFINET: Fix related to redundancy and PNIOStatus
    
    According to specification, redundancy bit values in
    packet-pn-rt.c are swapped. Moreover, endpoint type
    was changed by Change 30821 and became ENDPOINT_NONE.
    It has caused an error since related conversation can
    not be found with its endpoint (ENDPOINT_UDP). Finally,
    it is detected that when PNIOStatus is error, dissection
    still continues but there is no data to dissect. This
    defect is also fixed by this commit. (This change also
    includes one fix for dead store.)
    
    Change-Id: I09a07fd0027c4485ba84651e969b3de9d0012b5c
    Reviewed-on: https://code.wireshark.org/review/35158
    Petri-Dish: Graham Bloice <graham.bloice@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Roland Knall <rknall@xxxxxxxxx>
    

Actions performed:

    from  4dacdd9   Update to DICOM Edition 2019e
     add  1859b35   PROFINET: Fix related to redundancy and PNIOStatus


Summary of changes:
 plugins/epan/profinet/packet-dcerpc-pn-io.c | 9 ++++++---
 plugins/epan/profinet/packet-pn-rt.c        | 2 +-
 2 files changed, 7 insertions(+), 4 deletions(-)