Commit 360eb5da authored by Nicolas Dichtel's avatar Nicolas Dichtel Committed by David S. Miller

ipmr: fix sparse warning when testing origin or group

mfc_mcastgrp and mfc_origin are __be32, thus we need to convert INADDR_ANY.
Because INADDR_ANY is 0, this patch just fix sparse warnings.
Reported-by: default avatarFengguang Wu <fengguang.wu@intel.com>
Signed-off-by: default avatarNicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 63b203b4
...@@ -832,12 +832,12 @@ static struct mfc_cache *ipmr_cache_find(struct mr_table *mrt, ...@@ -832,12 +832,12 @@ static struct mfc_cache *ipmr_cache_find(struct mr_table *mrt,
static struct mfc_cache *ipmr_cache_find_any_parent(struct mr_table *mrt, static struct mfc_cache *ipmr_cache_find_any_parent(struct mr_table *mrt,
int vifi) int vifi)
{ {
int line = MFC_HASH(INADDR_ANY, INADDR_ANY); int line = MFC_HASH(htonl(INADDR_ANY), htonl(INADDR_ANY));
struct mfc_cache *c; struct mfc_cache *c;
list_for_each_entry_rcu(c, &mrt->mfc_cache_array[line], list) list_for_each_entry_rcu(c, &mrt->mfc_cache_array[line], list)
if (c->mfc_origin == INADDR_ANY && if (c->mfc_origin == htonl(INADDR_ANY) &&
c->mfc_mcastgrp == INADDR_ANY && c->mfc_mcastgrp == htonl(INADDR_ANY) &&
c->mfc_un.res.ttls[vifi] < 255) c->mfc_un.res.ttls[vifi] < 255)
return c; return c;
...@@ -848,14 +848,14 @@ static struct mfc_cache *ipmr_cache_find_any_parent(struct mr_table *mrt, ...@@ -848,14 +848,14 @@ static struct mfc_cache *ipmr_cache_find_any_parent(struct mr_table *mrt,
static struct mfc_cache *ipmr_cache_find_any(struct mr_table *mrt, static struct mfc_cache *ipmr_cache_find_any(struct mr_table *mrt,
__be32 mcastgrp, int vifi) __be32 mcastgrp, int vifi)
{ {
int line = MFC_HASH(mcastgrp, INADDR_ANY); int line = MFC_HASH(mcastgrp, htonl(INADDR_ANY));
struct mfc_cache *c, *proxy; struct mfc_cache *c, *proxy;
if (mcastgrp == INADDR_ANY) if (mcastgrp == htonl(INADDR_ANY))
goto skip; goto skip;
list_for_each_entry_rcu(c, &mrt->mfc_cache_array[line], list) list_for_each_entry_rcu(c, &mrt->mfc_cache_array[line], list)
if (c->mfc_origin == INADDR_ANY && if (c->mfc_origin == htonl(INADDR_ANY) &&
c->mfc_mcastgrp == mcastgrp) { c->mfc_mcastgrp == mcastgrp) {
if (c->mfc_un.res.ttls[vifi] < 255) if (c->mfc_un.res.ttls[vifi] < 255)
return c; return c;
...@@ -1148,7 +1148,7 @@ static int ipmr_mfc_add(struct net *net, struct mr_table *mrt, ...@@ -1148,7 +1148,7 @@ static int ipmr_mfc_add(struct net *net, struct mr_table *mrt,
return 0; return 0;
} }
if (mfc->mfcc_mcastgrp.s_addr != INADDR_ANY && if (mfc->mfcc_mcastgrp.s_addr != htonl(INADDR_ANY) &&
!ipv4_is_multicast(mfc->mfcc_mcastgrp.s_addr)) !ipv4_is_multicast(mfc->mfcc_mcastgrp.s_addr))
return -EINVAL; return -EINVAL;
...@@ -1807,7 +1807,7 @@ static int ip_mr_forward(struct net *net, struct mr_table *mrt, ...@@ -1807,7 +1807,7 @@ static int ip_mr_forward(struct net *net, struct mr_table *mrt,
cache->mfc_un.res.pkt++; cache->mfc_un.res.pkt++;
cache->mfc_un.res.bytes += skb->len; cache->mfc_un.res.bytes += skb->len;
if (cache->mfc_origin == INADDR_ANY && true_vifi >= 0) { if (cache->mfc_origin == htonl(INADDR_ANY) && true_vifi >= 0) {
struct mfc_cache *cache_proxy; struct mfc_cache *cache_proxy;
/* For an (*,G) entry, we only check that the incomming /* For an (*,G) entry, we only check that the incomming
...@@ -1863,8 +1863,8 @@ static int ip_mr_forward(struct net *net, struct mr_table *mrt, ...@@ -1863,8 +1863,8 @@ static int ip_mr_forward(struct net *net, struct mr_table *mrt,
/* /*
* Forward the frame * Forward the frame
*/ */
if (cache->mfc_origin == INADDR_ANY && if (cache->mfc_origin == htonl(INADDR_ANY) &&
cache->mfc_mcastgrp == INADDR_ANY) { cache->mfc_mcastgrp == htonl(INADDR_ANY)) {
if (true_vifi >= 0 && if (true_vifi >= 0 &&
true_vifi != cache->mfc_parent && true_vifi != cache->mfc_parent &&
ip_hdr(skb)->ttl > ip_hdr(skb)->ttl >
...@@ -1881,7 +1881,8 @@ static int ip_mr_forward(struct net *net, struct mr_table *mrt, ...@@ -1881,7 +1881,8 @@ static int ip_mr_forward(struct net *net, struct mr_table *mrt,
for (ct = cache->mfc_un.res.maxvif - 1; for (ct = cache->mfc_un.res.maxvif - 1;
ct >= cache->mfc_un.res.minvif; ct--) { ct >= cache->mfc_un.res.minvif; ct--) {
/* For (*,G) entry, don't forward to the incoming interface */ /* For (*,G) entry, don't forward to the incoming interface */
if ((cache->mfc_origin != INADDR_ANY || ct != true_vifi) && if ((cache->mfc_origin != htonl(INADDR_ANY) ||
ct != true_vifi) &&
ip_hdr(skb)->ttl > cache->mfc_un.res.ttls[ct]) { ip_hdr(skb)->ttl > cache->mfc_un.res.ttls[ct]) {
if (psend != -1) { if (psend != -1) {
struct sk_buff *skb2 = skb_clone(skb, GFP_ATOMIC); struct sk_buff *skb2 = skb_clone(skb, GFP_ATOMIC);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment