Wireshark-commits: [Wireshark-commits] master deaac11: iso7816: check for minimum atr len

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 11 Jul 2016 09:22:10 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=deaac110db74ad6de7b3d7a6416d1b0df7c51fe1
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

deaac11 by Martin Kaiser (wireshark@xxxxxxxxx):

    iso7816: check for minimum atr len
    
    and reject the packet if it is shorter
    
    make sure that we don't throw an exception when we read the first byte
    and haven't yet decided if the packet contains an iso7816 atr
    
    Change-Id: I7b4c93cc7c55489467b46241f07a1bb5ddfd927a
    Reviewed-on: https://code.wireshark.org/review/16377
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  f2584b2   BGP: Add support of Vendor Capability 131 (Multisession Cisco)
    adds  deaac11   iso7816: check for minimum atr len


Summary of changes:
 epan/dissectors/packet-iso7816.c |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)