Wireshark-commits: [Wireshark-commits] master fa75848: packet-kerberos: Encrypted{AuthorizationData

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: Wed, 18 Mar 2020 18:30:21 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=fa7584898d86b38b961dd4d140a1aa2886a55df4
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

fa75848 by Stefan Metzmacher (metze@xxxxxxxxx):

    packet-kerberos: Encrypted{AuthorizationData,Authenticator} are not the same
    
    We need to dissect them differently...
    
    Change-Id: Idb6d65800b1787b9cb6fca2630373547b9b7b1bc
    Signed-off-by: Stefan Metzmacher <metze@xxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/36466
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  95d0ab1   packet-kerberos: we only need PA-PAC-REQUEST once
     add  fa75848   packet-kerberos: Encrypted{AuthorizationData,Authenticator} are not the same


Summary of changes:
 epan/dissectors/asn1/kerberos/KerberosV5Spec2.asn  |   8 +-
 epan/dissectors/asn1/kerberos/kerberos.cnf         |   9 ++
 .../asn1/kerberos/packet-kerberos-template.c       |  37 +++++
 epan/dissectors/packet-kerberos.c                  | 161 +++++++++++++++------
 4 files changed, 173 insertions(+), 42 deletions(-)