Wireshark-commits: [Wireshark-commits] master 1ea82e4: Don't crash for ERSPAN packets in Linux cook
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: Thu, 22 Mar 2018 21:16:51 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1ea82e4ff7a81a3719cc8d6b4cd95fada3abb79b
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

1ea82e4 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't crash for ERSPAN packets in Linux cooked captures.
    
    Sadly, "cooked" means the GRE header isn't available; the extra data
    pointer is null, so we can't dereference it.
    
    Bug: 14548
    Change-Id: I51ae67dcc144b7f5ab3c82dd9adf09b342b29ced
    Reviewed-on: https://code.wireshark.org/review/26595
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  c55d157   Add Arista Vendor Specific Protocol (Ethertype) dissector
    adds  1ea82e4   Don't crash for ERSPAN packets in Linux cooked captures.


Summary of changes:
 epan/dissectors/packet-cisco-erspan.c | 17 ++++++++++++++---
 epan/dissectors/packet-sll.c          |  8 ++++++++
 2 files changed, 22 insertions(+), 3 deletions(-)