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

Reply via email to