leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

net: dsa: b53: remove redundant null check on dev

Browse Source 
The pointer dev can never be null, the null check is redundant
and can be removed. Cleans up a static analysis warning that
pointer priv is dereferencing dev before dev is being null
checked.

Addresses-Coverity: ("Dereference before null check")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Colin Ian King
2021-06-15 10:05:16 +01:00
committed by David S. Miller
parent 848ca9182a
commit 11b57faf95
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
drivers/net/dsa/b53/b53_srab.c
View File

@@ -632,7 +632,6 @@ static int b53_srab_remove(struct platform_device *pdev)
struct b53_srab_priv *priv = dev->priv; struct b53_srab_priv *priv = dev->priv;
b53_srab_intr_set(priv, false); b53_srab_intr_set(priv, false);
if (dev)
b53_switch_remove(dev); b53_switch_remove(dev);
return 0; return 0;