Hi,

On 2015年12月10日 19:09, Yura Yurochko wrote:
> ---
>  ryu/controller/ofp_handler.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ryu/controller/ofp_handler.py b/ryu/controller/ofp_handler.py
> index cc64b1f..d60154f 100644
> --- a/ryu/controller/ofp_handler.py
> +++ b/ryu/controller/ofp_handler.py
> @@ -251,7 +251,7 @@ class OFPHandler(ryu.base.app_manager.RyuApp):
>      def error_msg_handler(self, ev):
>          msg = ev.msg
>          ofp = msg.datapath.ofproto
> -        (version, msg_type, msg_len, xid) = ofproto_parser.header(msg.data)

This code parses OpenFlow Header using the data field of OFPT_ERROR messages,
which contains part of the failed request.
So I think the existing codes are not invalid.


> +        (version, msg_type, msg_len, xid) = ofproto_parser.header(msg.buf)

If msg.buf is specified, (version, msg_type, msg_len, xid) will always indicate
the OFPT_ERROR messages.


>          self.logger.debug('EventOFPErrorMsg received.')
>          self.logger.debug(
>              'version=%s, msg_type=%s, msg_len=%s, xid=%s', hex(msg.version),
> 


Thanks,
Iwase

------------------------------------------------------------------------------
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to