Wireshark-commits: [Wireshark-commits] master be4d42c: Widen openflow variables and check for under
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, 6 Jul 2014 16:53:41 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=be4d42c6ab54169c8c027d3b1871a282fd9d0a9d
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

be4d42c by Evan Huus (eapache@xxxxxxxxx):

    Widen openflow variables and check for underflows
    
    As suggested by Alexis reviewing change Ie76c5a810af927b, this issue was
    scattered throughout the v4/v5 openflow dissectors. Hopefully I got all of them.
    Original issue found in bug 10259.
    
    Change-Id: Idf87deea34132aec5cd4e8f83264906a29b15568
    Reviewed-on: https://code.wireshark.org/review/2878
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  5886e45   iscsi: minor retval fix
    adds  be4d42c   Widen openflow variables and check for underflows


Summary of changes:
 epan/dissectors/packet-openflow_v4.c |   25 +++++++++++--------
 epan/dissectors/packet-openflow_v5.c |   45 ++++++++++++++++++++++------------
 2 files changed, 44 insertions(+), 26 deletions(-)