Wireshark-commits: [Wireshark-commits] master e317421: Make uat_fld_chk_num_hex() more like other u

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: Sat, 20 Oct 2018 05:59:47 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e317421aaaec5767ef4b3757002215028ede04df
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

e317421 by Guy Harris (guy@xxxxxxxxxxxx):

    Make uat_fld_chk_num_hex() more like other uat_fld_chk_num routines.
    
    Allow a space after the number.
    
    Return "Invalid value" for strings that aren't valid numbers and "Value
    too large" for values that don't fit in a 32-bit integer.
    
    Change-Id: Iff616330968bf434fc7daf822a09ffc4f768105b
    Reviewed-on: https://code.wireshark.org/review/30272
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  3fdc40f   Use ws_basetostru32(), rather than strtoul(), when parsing uint preferences.
     add  e317421   Make uat_fld_chk_num_hex() more like other uat_fld_chk_num routines.


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