summaryrefslogtreecommitdiff
path: root/net/hsr/hsr_forward.c
diff options
context:
space:
mode:
authorWang Hai <wanghai38@huawei.com>2020-07-18 15:53:38 +0300
committerDavid S. Miller <davem@davemloft.net>2020-07-21 04:33:32 +0300
commit2b96692bcfcd2be7c9aa55dc13440213cd54c654 (patch)
tree2897555f2ba464f17b71173cb7eaca97ad1a91fe /net/hsr/hsr_forward.c
parent405e30e23cb37c1a1cd521f41db89d2d93bd1e3b (diff)
downloadlinux-2b96692bcfcd2be7c9aa55dc13440213cd54c654.tar.xz
net: hsr: remove redundant null check
Because kfree_skb already checked NULL skb parameter, so the additional checks are unnecessary, just remove them. Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: Wang Hai <wanghai38@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/hsr/hsr_forward.c')
-rw-r--r--net/hsr/hsr_forward.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/net/hsr/hsr_forward.c b/net/hsr/hsr_forward.c
index ed13760463de..92c8ad75200b 100644
--- a/net/hsr/hsr_forward.c
+++ b/net/hsr/hsr_forward.c
@@ -367,10 +367,8 @@ void hsr_forward_skb(struct sk_buff *skb, struct hsr_port *port)
port->dev->stats.tx_bytes += skb->len;
}
- if (frame.skb_hsr)
- kfree_skb(frame.skb_hsr);
- if (frame.skb_std)
- kfree_skb(frame.skb_std);
+ kfree_skb(frame.skb_hsr);
+ kfree_skb(frame.skb_std);
return;
out_drop: