diff options
| author | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2012-06-30 11:39:19 +0900 |
|---|---|---|
| committer | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2012-06-30 11:46:25 +0900 |
| commit | 23bd7d88e1986cff132716ebc76dc6d4300dcefa (patch) | |
| tree | 906d8d3afafe65e633cc5acd7cf829d66b7f4f2c | |
| parent | f4df6174b2621dc8206c78028ad09232939bc78c (diff) | |
of1.2: add unittest workaround to OFPPortStatus parser
Add unittest workaround to OFPPortStatus parser. Another Option is
defining something like OFP_PORT_STATUS_PACK_STR0, 'B'. I don't care
much. Let's just do as we do with OF1.0.
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
| -rw-r--r-- | ryu/ofproto/ofproto_v1_2_parser.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ryu/ofproto/ofproto_v1_2_parser.py b/ryu/ofproto/ofproto_v1_2_parser.py index bcf48b62..acbf5c7f 100644 --- a/ryu/ofproto/ofproto_v1_2_parser.py +++ b/ryu/ofproto/ofproto_v1_2_parser.py @@ -286,9 +286,9 @@ class OFPPortStatus(MsgBase): def parser(cls, datapath, version, msg_type, msg_len, xid, buf): msg = super(OFPPortStatus, cls).parser(datapath, version, msg_type, msg_len, xid, buf) - (msg.reason,) = struct.unpack_from( + msg.reason = struct.unpack_from( ofproto_v1_2.OFP_PORT_STATUS_PACK_STR, msg.buf, - ofproto_v1_2.OFP_HEADER_SIZE) + ofproto_v1_2.OFP_HEADER_SIZE)[0] msg.desc = OFPPort.parser(msg.buf, ofproto_v1_2.OFP_PORT_STATUS_DESC_OFFSET) return msg |