Wireshark-commits: [Wireshark-commits] master 9145acb: uat: tighten up dec/hex uat field validity c
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 20 Nov 2014 13:47:16 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9145acb68ac5485d9abe9be23d02964755890c67
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

9145acb by Bill Meier (wmeier@xxxxxxxxxxx):

    uat: tighten up dec/hex uat field validity checking
    
    Specifically:
    - Use the proper code for testing strtol() result;
    - Also: Values greater than 32-bits treated as an error
       (on LP64 architectures);
    
    Change-Id: I56e8e734fbb9a22dbd9ed4112e24327ffd7ee3c0
    Reviewed-on: https://code.wireshark.org/review/5394
    Reviewed-by: Bill Meier <wmeier@xxxxxxxxxxx>
    Petri-Dish: Bill Meier <wmeier@xxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  ead585f   Give better hints as to where the "expert info" fields go.
    adds  9145acb   uat: tighten up dec/hex uat field validity checking


Summary of changes:
 epan/uat.c |   37 ++++++++++++++++++++++---------------
 1 file changed, 22 insertions(+), 15 deletions(-)