Wireshark-commits: [Wireshark-commits] master 44d1601: Don't try checksumming if we didn't capture

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, 2 May 2015 09:57:32 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=44d16019a27f1164d16dff7fb0f5f6bd8ed89af2
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

44d1601 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't try checksumming if we didn't capture all the data to be checksummed.
    
    Check the captured length as well as the reported length, as is done in
    other checksumming code paths.
    
    Change-Id: I55371d99a28ffd1325e7266037cced580077d80b
    Reviewed-on: https://code.wireshark.org/review/8272
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  27e480c   Handle PIMvN, for N > 2, by just adding the checksum and subtree and bailing.
    adds  44d1601   Don't try checksumming if we didn't capture all the data to be checksummed.


Summary of changes:
 epan/dissectors/packet-pim.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)