meta-flow: Correctly byteswap skb_priority/skb_mark for mf_value.
authorBen Pfaff <blp@nicira.com>
Tue, 18 Dec 2012 23:54:29 +0000 (15:54 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 19 Dec 2012 00:46:20 +0000 (16:46 -0800)
Fixes the following sparse warnings:

meta-flow.c:947:21: warning: incorrect type in assignment (different base types)
meta-flow.c:947:21:    expected restricted __be32 [usertype] be32
meta-flow.c:947:21:    got unsigned int const [unsigned] [usertype] skb_priority
meta-flow.c:951:21: warning: incorrect type in assignment (different base types)
meta-flow.c:951:21:    expected restricted __be32 [usertype] be32
meta-flow.c:951:21:    got unsigned int const [unsigned] [usertype] skb_mark

Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/meta-flow.c

index dd643ee..314ffc6 100644 (file)
@@ -944,11 +944,11 @@ mf_get_value(const struct mf_field *mf, const struct flow *flow,
         break;
 
     case MFF_SKB_PRIORITY:
-        value->be32 = flow->skb_priority;
+        value->be32 = htonl(flow->skb_priority);
         break;
 
     case MFF_SKB_MARK:
-        value->be32 = flow->skb_mark;
+        value->be32 = htonl(flow->skb_mark);
         break;
 
     CASE_MFF_REGS: