Wireshark-commits: [Wireshark-commits] master 4569772: eap: fix which tokens we think contain mncNN
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 13 Aug 2020 21:50:11 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4569772457b92c4b25c8c2e7bd4bac5256038e41
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

4569772 by Guy Harris (gharris@xxxxxxxxx):

    eap: fix which tokens we think contain mncNNN and mccNNN.
    
    tokens[] contains two tokens - the part of the identity before @ and the
    part of the identity after @.
    
    realm_tokens[] contain five tokens - the "."-separated parts of the part
    of the identity after @.
    
    The latter include "mncNNN" and "mncNNN".
    
    This fixes a crash.
    
    Change-Id: I4b13dd90977a626a823cb53958412301abf8addb
    Reviewed-on: https://code.wireshark.org/review/38158
    Petri-Dish: Guy Harris <gharris@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    

Actions performed:

    from  c3066b6   make-version.pl: Update sample output in comments to more modern git version 2.24.3 (Apple Git-128)
     add  4569772   eap: fix which tokens we think contain mncNNN and mccNNN.


Summary of changes:
 epan/dissectors/packet-eap.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)