Wireshark-commits: [Wireshark-commits] master 2780a30: GSM MAP: do not use a global variable to sto

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: Tue, 3 Mar 2015 14:51:44 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2780a303ab0e8d293bd9be88b523de1bb02b413c
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

2780a30 by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    GSM MAP: do not use a global variable to store OID
    
    Fixes an ASAN failure reported by Alexis
    
    Change-Id: I63b07ec4b6eb0698c4abf27f8fb0b1f11a19396d
    Reviewed-on: https://code.wireshark.org/review/7510
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    

Actions performed:

    from  8933cbb   HTTP: Fix small typo in a field description
    adds  2780a30   GSM MAP: do not use a global variable to store OID


Summary of changes:
 asn1/gsm_map/gsm_map.cnf               |    8 ++-
 asn1/gsm_map/packet-gsm_map-template.c |    1 -
 epan/dissectors/packet-gsm_map.c       |  118 +++++++++++++++++---------------
 3 files changed, 66 insertions(+), 61 deletions(-)