Messages in this thread | | | Subject | Re: [PATCH net-next] net: ipv4: Remove unneed BUG() function | From | David Ahern <> | Date | Mon, 7 Jun 2021 10:20:54 -0600 |
| |
On 6/7/21 8:39 AM, Zheng Yongjun wrote: > When 'nla_parse_nested_deprecated' failed, it's no need to > BUG() here, return -EINVAL is ok. > > Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com> > --- > net/ipv4/devinet.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c > index 2e35f68da40a..1c6429c353a9 100644 > --- a/net/ipv4/devinet.c > +++ b/net/ipv4/devinet.c > @@ -1989,7 +1989,7 @@ static int inet_set_link_af(struct net_device *dev, const struct nlattr *nla, > return -EAFNOSUPPORT; > > if (nla_parse_nested_deprecated(tb, IFLA_INET_MAX, nla, NULL, NULL) < 0)
Avoid assumptions on the failure reason:
int err;
err = nla_parse_nested_deprecated(); if (err < 0) return err;
> - BUG(); > + return -EINVAL; > > if (tb[IFLA_INET_CONF]) { > nla_for_each_nested(a, tb[IFLA_INET_CONF], rem) >
seems like this patch and a similar fix for the IPv6 version of set_link_af should go to net rather than net-next.
| |