Wireshark-commits: [Wireshark-commits] master 29be26c: FTAM: get rid of an evil global variable

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, 10 Feb 2015 12:35:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=29be26c4b3b387027abcab3c859643f0ee12e5cf
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

29be26c by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    FTAM: get rid of an evil global variable
    
    Fixes an ASAN failure reported by Alexis
    
    Change-Id: Id0ddad6451e22ffc53d42985e9b2071c7388d15b
    Reviewed-on: https://code.wireshark.org/review/7049
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  3b29542   Add a "Sum" column to the Service Response Time (SRT) window.
    adds  29be26c   FTAM: get rid of an evil global variable


Summary of changes:
 asn1/ftam/ftam.cnf               |   18 ++++++++++-----
 asn1/ftam/packet-ftam-template.c |    1 -
 epan/dissectors/packet-ftam.c    |   45 +++++++++++++++++++++-----------------
 3 files changed, 37 insertions(+), 27 deletions(-)