Wireshark-commits: [Wireshark-commits] master-1.10 5514231: Don't intentionally use -1 as an hfinde
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: Mon, 24 Feb 2014 21:53:03 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=551423110b94730ed25a4193df5e30fbb5998b95
Submitter: Gerald Combs (gerald@xxxxxxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

5514231 by Jeff Morriss (jeff.morriss.ws@xxxxxxxxx):

    Don't intentionally use -1 as an hfindex: while only naughty before, with the
    changes from r49644 you'll get complaints about the hf being unregistered...
    
    Note that this code also intentionally passes -1 as an ett_index.  It doesn't
    currently complain so I'm leaving it alone for now...
    
    svn path=/trunk/; revision=49648
    (cherry picked from commit 813dcf79a15d6527cce8a6a7bac4234de6ab577f)
    Change-Id: I2bca70df606397264df54e88ce6fa85815aaf1b9
    Reviewed-on: https://code.wireshark.org/review/338
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    

Actions performed:

    from  e45b095   NAS EPS: fix dissection of CS service notification message
    adds  5514231   Don't intentionally use -1 as an hfindex: while only naughty before, with the changes from r49644 you'll get complaints about the hf being unregistered...


Summary of changes:
 asn1/t124/packet-t124-template.c |    4 +++-
 epan/dissectors/packet-t124.c    |    8 +++++---
 2 files changed, 8 insertions(+), 4 deletions(-)