Wireshark-commits: [Wireshark-commits] master ee1dd89: iso7816: fix the dissection of the class byt
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 05 Jun 2019 03:40:16 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ee1dd899d0a3350c5d700e0a5845b8a98146475d
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

ee1dd89 by Martin Kaiser (wireshark@xxxxxxxxx):

    iso7816: fix the dissection of the class byte
    
    Update dissect_iso7816_class() to return 1 only if both APDU structure
    and coding are compliant with ISO 7816. In this case, the iso7816 dissector
    can continue dissecting the APDU.
    
    Change-Id: I73d4246fbc234779fceb337c788dd0b680102d61
    Reviewed-on: https://code.wireshark.org/review/33480
    Reviewed-by: Martin Kaiser <wireshark@xxxxxxxxx>
    Petri-Dish: Martin Kaiser <wireshark@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  4e1a5f6   iso7816: correct the descriptions of the class byte
     add  ee1dd89   iso7816: fix the dissection of the class byte


Summary of changes:
 epan/dissectors/packet-iso7816.c | 47 ++++++++++++++++++++++++++++------------
 1 file changed, 33 insertions(+), 14 deletions(-)