Wireshark-commits: [Wireshark-commits] master bf8ca6a: Replace a tvb_new_subset_length_caplen() cal

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, 28 Feb 2019 18:42:07 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bf8ca6a605424e6ee23b66ac9a45ef4818268b6d
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

bf8ca6a by Guy Harris (guy@xxxxxxxxxxxx):

    Replace a tvb_new_subset_length_caplen() call with tvb_new_subset_remaining().
    
    It's simpler, and is not incorrectly using the *captured* length to set
    the *reported* length.
    
    Change-Id: If4b7f1c431f4c39dcc568698358667a1b4fc1a12
    Reviewed-on: https://code.wireshark.org/review/32268
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  c6781b6   Remove FCS length checks from code that's handed a tvbuff without an FCS.
     add  bf8ca6a   Replace a tvb_new_subset_length_caplen() call with tvb_new_subset_remaining().


Summary of changes:
 epan/dissectors/packet-ieee802154.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)