net: core: rtnetlink: use BUG_ON instead of if condition followed by BUG
Subsystems (2)
PATCH 1/1 net: core: rtnetlink: use BUG_ON instead of if condition followed by BUG
Use BUG_ON instead of if condition followed by BUG in do_setlink.
This issue was detected with the help of Coccinelle.
Signed-off-by Gustavo A. R. Silva <[email protected]>
---
This code was tested by compilation only.
net/core/rtnetlink.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
net/core/rtnetlink.c +1 -2
@@ -2274,8 +2274,7 @@ static int do_setlink(const struct sk_buff *skb,rcu_read_lock (); -if (!(af_ops =rtnl_af_lookup (nla_type (af)))) -BUG (); +BUG_ON (!(af_ops =rtnl_af_lookup (nla_type (af)))); err = af_ops->set_link_af (dev, af);if (err <0 ) {
Signature · 1 line
2.7.4
From: "Gustavo A. R. Silva" <[email protected]>
Date: Fri, 20 Oct 2017 19:43:11 -0500
Quoted 5 lines
Use
BUG_ONinstead of if condition followed by BUG indo_setlink.This issue was detected with the help of Coccinelle.
Signed-off-by Gustavo A. R. Silva <[email protected]>
Applied.