Wireshark-commits: [Wireshark-commits] master-2.6 cf61dd42: Check whether you've hit the end of the

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: Thu, 31 Jan 2019 07:40:55 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cf61dd42f616afb6179463566281a08cab32176a
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-2.6
Repository: wireshark

Commits:

cf61dd42 by Guy Harris (guy@xxxxxxxxxxxx):

    Check whether you've hit the end of the packet data before going further.
    
    If we found no bytes with the low-order bit set in the packet data,
    there's no point in checking the non-existent "next" byte to see if it's
    a HDLC-style UI control byte (0x03).
    
    Bug: 15463
    Change-Id: Ibfd186e5b81d8ce229362e23f00b31a27900831a
    Reviewed-on: https://code.wireshark.org/review/31824
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 4ac27e24cdd5ab32639029d6ff440af9f7111ae7)
    Reviewed-on: https://code.wireshark.org/review/31825
    

Actions performed:

    from  0e1e4fd   Rpcap authentication replies now have an optional payload.
     add  cf61dd42  Check whether you've hit the end of the packet data before going further.


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