net: tap: remove redundant null pointer check before kfree_skb

kfree_skb has taken the null pointer into account. hence it is safe
to remove the redundant null pointer check before kfree_skb.

Signed-off-by: zhong jiang <zhongjiang@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/tap.c b/drivers/net/tap.c
index a4ab4a791fe7e3fe47988eb30ec85712f8bde664..f03004f37eca600b25120b892400f1045f9576d6 100644 (file)
--- a/drivers/net/tap.c
+++ b/drivers/net/tap.c
@@ -830,8 +830,7 @@ static ssize_t tap_do_read(struct tap_queue *q,
        ssize_t ret = 0;
 
        if (!iov_iter_count(to)) {
-               if (skb)
-                       kfree_skb(skb);
+               kfree_skb(skb);
                return 0;
        }
 
@@ -1236,8 +1235,7 @@ static int tap_recvmsg(struct socket *sock, struct msghdr *m,
        struct sk_buff *skb = m->msg_control;
        int ret;
        if (flags & ~(MSG_DONTWAIT|MSG_TRUNC)) {
-               if (skb)
-                       kfree_skb(skb);
+               kfree_skb(skb);
                return -EINVAL;
        }
        ret = tap_do_read(q, &m->msg_iter, flags & MSG_DONTWAIT, skb);