ryu crashes when it gets OFPT_STATS_REPLY including OFP actions. ofp_flow_stats can include ofp_action_header(s) so parser_stats_body_array method can't use the entry_size argument (the minimum size, 88 bytes). It needs to use the size that body_cls's parser returns.
Signed-off-by: FUJITA Tomonori <[email protected]> --- ryu/ofproto/ofproto_v1_0_parser.py | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_0_parser.py b/ryu/ofproto/ofproto_v1_0_parser.py index ea164bd..4b9b63d 100644 --- a/ryu/ofproto/ofproto_v1_0_parser.py +++ b/ryu/ofproto/ofproto_v1_0_parser.py @@ -1117,10 +1117,10 @@ class OFPStatsReply(MsgBase): def parser_stats_body_array(cls, buf, msg_len, offset, entry_size): body_cls = cls.cls_stats_body_cls body = [] - while offset + entry_size <= msg_len: + while offset < msg_len: entry = body_cls.parser(buf, offset) body.append(entry) - offset += entry_size + offset += entry.length return body @classmethod -- 1.7.4.4 ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
