On Sat, Sep 07, 2013 at 03:36:21PM +0300, Alexandru Copot wrote: > Signed-off-by: Alexandru Copot <alex.miha...@gmail.com> > Cc: Daniel Baluta <dbal...@ixiacom.com>
"sparse" reported that the byte order conversions were reversed, so I folded this in: diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c index 36151dd..6900e99 100644 --- a/ofproto/ofproto.c +++ b/ofproto/ofproto.c @@ -4133,13 +4133,13 @@ handle_nxt_get_async_request(struct ofconn *ofconn, const struct ofp_header *oh) buf = ofpraw_alloc_reply(OFPRAW_OFPT13_GET_ASYNC_REPLY, oh, 0); msg = ofpbuf_put_zeros(buf, sizeof *msg); - msg->packet_in_mask[0] = ntohl(master[OAM_PACKET_IN]); - msg->port_status_mask[0] = ntohl(master[OAM_PORT_STATUS]); - msg->flow_removed_mask[0] = ntohl(master[OAM_FLOW_REMOVED]); + msg->packet_in_mask[0] = htonl(master[OAM_PACKET_IN]); + msg->port_status_mask[0] = htonl(master[OAM_PORT_STATUS]); + msg->flow_removed_mask[0] = htonl(master[OAM_FLOW_REMOVED]); - msg->packet_in_mask[1] = ntohl(slave[OAM_PACKET_IN]); - msg->port_status_mask[1] = ntohl(slave[OAM_PORT_STATUS]); - msg->flow_removed_mask[1] = ntohl(slave[OAM_FLOW_REMOVED]); + msg->packet_in_mask[1] = htonl(slave[OAM_PACKET_IN]); + msg->port_status_mask[1] = htonl(slave[OAM_PORT_STATUS]); + msg->flow_removed_mask[1] = htonl(slave[OAM_FLOW_REMOVED]); ofconn_send_reply(ofconn, buf); _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev