From 70734c1b5a0dd39c651b8c1043b79f735d4b890b Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Tue, 18 Dec 2012 15:54:29 -0800 Subject: [PATCH] meta-flow: Correctly byteswap skb_priority/skb_mark for mf_value. 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 --- lib/meta-flow.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/meta-flow.c b/lib/meta-flow.c index dd643ee40..314ffc667 100644 --- a/lib/meta-flow.c +++ b/lib/meta-flow.c @@ -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: -- 2.20.1