* [PATCH] net: Remove redundant BUG_ON() check in phonet_pernet
@ 2020-01-03 9:20 Xu Wang
2020-01-03 20:26 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Xu Wang @ 2020-01-03 9:20 UTC (permalink / raw)
To: davem; +Cc: netdev, linux-kernel
Passing NULL to phonet_pernet causes a crash via BUG_ON.
Dereferencing net in net_generic() also has the same effect.
This patch removes the redundant BUG_ON check on the same parameter.
Signed-off-by: Xu Wang <vulab@iscas.ac.cn>
---
net/phonet/pn_dev.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/net/phonet/pn_dev.c b/net/phonet/pn_dev.c
index 49bd76a..ac0fae0 100644
--- a/net/phonet/pn_dev.c
+++ b/net/phonet/pn_dev.c
@@ -35,8 +35,6 @@ static unsigned int phonet_net_id __read_mostly;
static struct phonet_net *phonet_pernet(struct net *net)
{
- BUG_ON(!net);
-
return net_generic(net, phonet_net_id);
}
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] net: Remove redundant BUG_ON() check in phonet_pernet
2020-01-03 9:20 [PATCH] net: Remove redundant BUG_ON() check in phonet_pernet Xu Wang
@ 2020-01-03 20:26 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2020-01-03 20:26 UTC (permalink / raw)
To: vulab; +Cc: netdev, linux-kernel
From: Xu Wang <vulab@iscas.ac.cn>
Date: Fri, 3 Jan 2020 09:20:40 +0000
> Passing NULL to phonet_pernet causes a crash via BUG_ON.
> Dereferencing net in net_generic() also has the same effect.
> This patch removes the redundant BUG_ON check on the same parameter.
>
> Signed-off-by: Xu Wang <vulab@iscas.ac.cn>
Applied.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-01-03 20:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-03 9:20 [PATCH] net: Remove redundant BUG_ON() check in phonet_pernet Xu Wang
2020-01-03 20:26 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).