Wireshark-commits: [Wireshark-commits] master-2.2 ad5952a: Just use strncmp() to check the prefix o

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: Wed, 14 Dec 2016 23:04:27 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ad5952ae2354bfb0219c9921ab79e0fab587b9be
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

ad5952a by Guy Harris (guy@xxxxxxxxxxxx):

    Just use strncmp() to check the prefix of the file's first line.
    
    Bug: 13246
    Change-Id: I9df35596aa8dcb937f6a03cf60b5d0fbe9dce1ec
    Reviewed-on: https://code.wireshark.org/review/19276
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 1afbab69956c32b341001dd20ff667036fe1f8f5)
    Reviewed-on: https://code.wireshark.org/review/19277
    

Actions performed:

    from  13c8918   Don't assume we've read a line long enough to contain a magic number.
    adds  ad5952a   Just use strncmp() to check the prefix of the file's first line.


Summary of changes:
 wiretap/daintree-sna.c |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)