diff options
author | Yajun Deng <yajun.deng@linux.dev> | 2021-08-05 19:55:27 +0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2021-08-05 13:27:50 +0100 |
commit | 1160dfa178eb848327e9dec39960a735f4dc1685 (patch) | |
tree | b9e62879f43f63602da7a68c20d8e58a5acf79f3 /net/llc | |
parent | a85b99ab6abb583e051d266c2138647daa5c0ecc (diff) | |
download | linux-1160dfa178eb848327e9dec39960a735f4dc1685.tar.gz linux-1160dfa178eb848327e9dec39960a735f4dc1685.tar.bz2 linux-1160dfa178eb848327e9dec39960a735f4dc1685.zip |
net: Remove redundant if statements
The 'if (dev)' statement already move into dev_{put , hold}, so remove
redundant if statements.
Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/llc')
-rw-r--r-- | net/llc/af_llc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/llc/af_llc.c b/net/llc/af_llc.c index ac5cadd02cfa..3086f4a6ae68 100644 --- a/net/llc/af_llc.c +++ b/net/llc/af_llc.c @@ -224,8 +224,7 @@ static int llc_ui_release(struct socket *sock) } else { release_sock(sk); } - if (llc->dev) - dev_put(llc->dev); + dev_put(llc->dev); sock_put(sk); llc_sk_free(sk); out: @@ -363,8 +362,7 @@ static int llc_ui_bind(struct socket *sock, struct sockaddr *uaddr, int addrlen) } else llc->dev = dev_getbyhwaddr_rcu(&init_net, addr->sllc_arphrd, addr->sllc_mac); - if (llc->dev) - dev_hold(llc->dev); + dev_hold(llc->dev); rcu_read_unlock(); if (!llc->dev) goto out; |