Wireshark-commits: [Wireshark-commits] master-2.0 9f6e607: Call the "no FCS present" version 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: Wed, 11 May 2016 20:38:22 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9f6e6071cb6ec05d40516473b015dddfd0d9538d
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

9f6e607 by Guy Harris (guy@xxxxxxxxxxxx):

    Call the "no FCS present" version of the Ethernet dissector.
    
    The I-D doesn't explicity indicate that the FCS isn't present in the
    tunneled Ethernet frames, but it is missing from the captures attached
    to bug 10282.
    
    Update the I-D number while we're at it.
    
    Bug: 9933
    Change-Id: I6e78b20cb858e6c8c4d7380b17b88382db013d56
    Reviewed-on: https://code.wireshark.org/review/13350
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit fe82323df04e5fb2aaa3875d69e4e1ffe122b1ef)
    Reviewed-on: https://code.wireshark.org/review/15384
    

Actions performed:

    from  f3878c6   Have the "maybe an FCS" version of the Ethernet dissector take a data argument.
    adds  9f6e607   Call the "no FCS present" version of the Ethernet dissector.


Summary of changes:
 epan/dissectors/packet-stt.c |   10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)