Wireshark-commits: [Wireshark-commits] master 613c1bd: editcap: clean up the parsing of fractional
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 6 Aug 2020 09:26:22 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=613c1bd93147576df74a660e04449fd02a3e46de
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

613c1bd by Guy Harris (gharris@xxxxxxxxx):

    editcap: clean up the parsing of fractional time in -A and -B.
    
    Use ws_strtou32() so that we're guaranteed to produce a value that fits
    in a 32-bit integer (and don't get a narrowing warning on LP64 platforms
    for converting a long to an int), and then make sure it's less than one
    billion.
    
    Note, while we're at it, that you can, for example, specify a date of
    2020-10-40 and it won't produce an error.
    
    Change-Id: I26c36d346cfa5c2bdc9ecdbdf821a9bc3529c940
    Reviewed-on: https://code.wireshark.org/review/38065
    Petri-Dish: Guy Harris <gharris@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    

Actions performed:

    from  75884bd   packet-bthci-cmd: For LE Supported Features, fix loop to exit after 8 items.
     add  613c1bd   editcap: clean up the parsing of fractional time in -A and -B.


Summary of changes:
 editcap.c | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)