Wireshark-commits: [Wireshark-commits] master-1.12 07713f6: OpenFlow: fix flow_monitor_request diss

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, 27 Apr 2016 01:02:51 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=07713f626d4e766415ba0168569abb8b4505e209
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

07713f6 by IWASE Yusuke (iwase.yusuke0@xxxxxxxxx):

    OpenFlow: fix flow_monitor_request dissector
    
    This patch fixes:
     - offset calculation
     - monitor_id field description
    
    Change-Id: I9568b8897ca9ebbc6203db6640b88e1b1676448d
    Reviewed-on: https://code.wireshark.org/review/15128
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit e899c3eb610b4f546dfced6c41ae4e166b9c729e)
    Reviewed-on: https://code.wireshark.org/review/15130
    

Actions performed:

    from  6bcf422   Widen openflow variables and check for underflows
    adds  07713f6   OpenFlow: fix flow_monitor_request dissector


Summary of changes:
 epan/dissectors/packet-openflow_v5.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)