xfrm: policy: remove set but not used variable 'priority'
authorYueHaibing <yuehaibing@huawei.com>
Wed, 19 Dec 2018 01:12:20 +0000 (01:12 +0000)
committerSteffen Klassert <steffen.klassert@secunet.com>
Wed, 19 Dec 2018 11:24:43 +0000 (12:24 +0100)
Fixes gcc '-Wunused-but-set-variable' warning:

net/xfrm/xfrm_policy.c: In function 'xfrm_policy_lookup_bytype':
net/xfrm/xfrm_policy.c:2079:6: warning:
 variable 'priority' set but not used [-Wunused-but-set-variable]

It not used since commit 6be3b0db6db8 ("xfrm: policy: add inexact policy
search tree infrastructure")

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Acked-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
net/xfrm/xfrm_policy.c

index be04091eb7db4a0f028e7828a1c36667a96fc6b0..537b7eec623c353753980d094320d728d644690f 100644 (file)
@@ -2076,7 +2076,6 @@ static struct xfrm_policy *xfrm_policy_lookup_bytype(struct net *net, u8 type,
        struct xfrm_policy *pol, *ret;
        struct hlist_head *chain;
        unsigned int sequence;
-       u32 priority;
        int err;
 
        daddr = xfrm_flowi_daddr(fl, family);
@@ -2091,7 +2090,6 @@ static struct xfrm_policy *xfrm_policy_lookup_bytype(struct net *net, u8 type,
                chain = policy_hash_direct(net, daddr, saddr, family, dir);
        } while (read_seqcount_retry(&xfrm_policy_hash_generation, sequence));
 
-       priority = ~0U;
        ret = NULL;
        hlist_for_each_entry_rcu(pol, chain, bydst) {
                err = xfrm_policy_match(pol, fl, type, family, dir, if_id);
@@ -2104,7 +2102,6 @@ static struct xfrm_policy *xfrm_policy_lookup_bytype(struct net *net, u8 type,
                        }
                } else {
                        ret = pol;
-                       priority = ret->priority;
                        break;
                }
        }