Wireshark-commits: [Wireshark-commits] master-1.10 58f1ab4: GSM MAP: do not use a global variable t
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: Tue, 3 Mar 2015 16:29:55 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=58f1ab4d4877c791e53b1a5c63a2d16ca5564064
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

58f1ab4 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>
    (cherry picked from commit 2780a303ab0e8d293bd9be88b523de1bb02b413c)
    Conflicts:
    	asn1/gsm_map/gsm_map.cnf
    	epan/dissectors/packet-gsm_map.c
    Reviewed-on: https://code.wireshark.org/review/7514
    

Actions performed:

    from  e59c517   Prep for 1.10.13.
    adds  58f1ab4   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       |  116 +++++++++++++++++---------------
 3 files changed, 65 insertions(+), 60 deletions(-)