Wireshark-commits: [Wireshark-commits] master 498b64c: Check whether you've hit the end of the pack
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 31 Jan 2019 07:40:39 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=498b64c95c13980505e495bad33d911c26c52b8a
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

498b64c 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>
    

Actions performed:

    from  2a675c0   Use tvb_reported_length_remaining() to get the remaining data length.
     add  498b64c   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(-)