Wireshark-commits: [Wireshark-commits] master 59e6c9e: Fix construction of subset tvbuff.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 17 Jan 2016 20:33:41 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=59e6c9ed5ebce3e49eed268393495ac1f632aa5b
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

59e6c9e by Guy Harris (guy@xxxxxxxxxxxx):

    Fix construction of subset tvbuff.
    
    We should use both the captured length and original length, and we
    should use them as the captured length and original length of the
    tvbuff.
    
    Clean up indentation and returning process while we're at it.
    
    Change-Id: If7538268cd6ae557d9d5b85eefd57cf3e2e34a9e
    Reviewed-on: https://code.wireshark.org/review/13366
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  278b213   Call the "without FCS" version of the Ethernet dissector.
    adds  59e6c9e   Fix construction of subset tvbuff.


Summary of changes:
 epan/dissectors/packet-e100.c |  108 ++++++++++++++++++++++-------------------
 1 file changed, 57 insertions(+), 51 deletions(-)