Wireshark · Wireshark-commits: [Wireshark-commits] master ec781e9: proto: Add a debug check to sanity-check range_string

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ec781e9749c1329efb6a288668a1809b9d418ec0
Submitter: "Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

ec781e9 by Martin Mathieson (martin.mathieson@xxxxxxxxxxxx):

    proto: Add a debug check to sanity-check range_string
    
    Check is enabled by #ifdef ENABLE_CHECK_FILTER
    Remaining issues found by this check are fixed here,
    along with a documentation note that the entries
    are checked in order and the first match is used.
    
    The only issue not yet fixed is in packet-isup.c,
    where the spec was not available to me.
    
    Change-Id: Ife747cda9b91a265bc2b81ce0a53f55f3389919e
    Reviewed-on: https://code.wireshark.org/review/36708
    Petri-Dish: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    

Actions performed:

    from  361f82f   XnAP: upgrade dissector to v15.7.0
     add  ec781e9   proto: Add a debug check to sanity-check range_string


Summary of changes:
 doc/README.dissector                               |  5 +++-
 .../asn1/gsm_map/packet-gsm_map-template.c         |  1 -
 epan/dissectors/packet-gsm_map.c                   | 13 ++++-----
 epan/dissectors/packet-hislip.c                    |  2 +-
 epan/dissectors/packet-ieee1722.c                  |  2 +-
 epan/dissectors/packet-isup.c                      |  2 +-
 epan/dissectors/packet-optommp.c                   |  2 +-
 epan/dissectors/packet-tn5250.c                    |  2 +-
 epan/proto.c                                       | 33 ++++++++++++++++++++++
 9 files changed, 48 insertions(+), 14 deletions(-)