[ovs-dev] [PreQoS 01/13] ofproto: Fix byte order for OFP_VLAN_NONE to dl_vlan in struct flow.

Ben Pfaff blp at nicira.com
Thu May 27 20:33:00 UTC 2010


This is not a bug, since OFP_VLAN_NONE is all-1-bits, but it is still
best to get it correct.
---
 ofproto/ofproto.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 7cca951..6780250 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -2467,7 +2467,7 @@ do_xlate_actions(const union ofp_action *in, size_t n_in,
 
         case OFPAT_STRIP_VLAN:
             odp_actions_add(ctx->out, ODPAT_STRIP_VLAN);
-            ctx->flow.dl_vlan = OFP_VLAN_NONE;
+            ctx->flow.dl_vlan = htons(OFP_VLAN_NONE);
             ctx->flow.dl_vlan_pcp = 0;
             break;
 
-- 
1.7.1





More information about the dev mailing list