If ryu tries to parse segmented OF packets by the packet library,
the process becomes infinite loop, bacause Ryu tries to parse
OpenFlow header for the segmented packets,
which don't have OpenFlow header, they only have body.
This commit fixes this by stopping parsing OpenFlow header
when the result is invalid.

Satoshi Fujimoto (2):
  test/packet: Remove openflow_invalid_version test
  packet: Avoid freezing when receiving segmented OF packets

 ryu/lib/packet/openflow.py                               |   2 ++
 ryu/ofproto/ofproto_parser.py                            |   3 ++-
 ryu/tests/packet_data/pcap/openflow_invalid_version.pcap | Bin 186 -> 0 bytes
 ryu/tests/unit/packet/test_openflow.py                   |   1 -
 4 files changed, 4 insertions(+), 2 deletions(-)
 delete mode 100644 ryu/tests/packet_data/pcap/openflow_invalid_version.pcap

-- 
2.7.4


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to