crypto: chtls - remove set but not used variable 'csk'
authorYueHaibing <yuehaibing@huawei.com>
Fri, 5 Oct 2018 06:43:27 +0000 (06:43 +0000)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 12 Oct 2018 06:20:44 +0000 (14:20 +0800)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/crypto/chelsio/chtls/chtls_cm.c: In function 'chtls_disconnect':
drivers/crypto/chelsio/chtls/chtls_cm.c:408:21: warning:
 variable 'csk' set but not used [-Wunused-but-set-variable]

drivers/crypto/chelsio/chtls/chtls_cm.c: In function 'chtls_recv_sock':
drivers/crypto/chelsio/chtls/chtls_cm.c:1016:23: warning:
 variable 'tcph' set but not used [-Wunused-but-set-variable]

'csk' and 'tcph' are never used since introduce
in commit cc35c88ae4db ("crypto : chtls - CPL handler definition")

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/chelsio/chtls/chtls_cm.c

index fd2ae1e..20209e2 100644 (file)
@@ -405,12 +405,10 @@ static int wait_for_states(struct sock *sk, unsigned int states)
 
 int chtls_disconnect(struct sock *sk, int flags)
 {
-       struct chtls_sock *csk;
        struct tcp_sock *tp;
        int err;
 
        tp = tcp_sk(sk);
-       csk = rcu_dereference_sk_user_data(sk);
        chtls_purge_recv_queue(sk);
        chtls_purge_receive_queue(sk);
        chtls_purge_write_queue(sk);
@@ -1013,7 +1011,6 @@ static struct sock *chtls_recv_sock(struct sock *lsk,
                                    const struct cpl_pass_accept_req *req,
                                    struct chtls_dev *cdev)
 {
-       const struct tcphdr *tcph;
        struct inet_sock *newinet;
        const struct iphdr *iph;
        struct net_device *ndev;
@@ -1035,7 +1032,6 @@ static struct sock *chtls_recv_sock(struct sock *lsk,
        if (!dst)
                goto free_sk;
 
-       tcph = (struct tcphdr *)(iph + 1);
        n = dst_neigh_lookup(dst, &iph->saddr);
        if (!n)
                goto free_sk;