Skip to content

Commit d1892e4

Browse files
tklauserdavem330
authored andcommitted
rtnl: simplify error return path in rtnl_create_link()
There is only one possible error path which reaches the err label, so return ERR_PTR(-ENOMEM) directly if alloc_netdev_mqs() fails. This also allows to omit the err variable. Signed-off-by: Tobias Klauser <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 017b29c commit d1892e4

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

net/core/rtnetlink.c

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2358,7 +2358,6 @@ struct net_device *rtnl_create_link(struct net *net,
23582358
const char *ifname, unsigned char name_assign_type,
23592359
const struct rtnl_link_ops *ops, struct nlattr *tb[])
23602360
{
2361-
int err;
23622361
struct net_device *dev;
23632362
unsigned int num_tx_queues = 1;
23642363
unsigned int num_rx_queues = 1;
@@ -2373,11 +2372,10 @@ struct net_device *rtnl_create_link(struct net *net,
23732372
else if (ops->get_num_rx_queues)
23742373
num_rx_queues = ops->get_num_rx_queues();
23752374

2376-
err = -ENOMEM;
23772375
dev = alloc_netdev_mqs(ops->priv_size, ifname, name_assign_type,
23782376
ops->setup, num_tx_queues, num_rx_queues);
23792377
if (!dev)
2380-
goto err;
2378+
return ERR_PTR(-ENOMEM);
23812379

23822380
dev_net_set(dev, net);
23832381
dev->rtnl_link_ops = ops;
@@ -2403,9 +2401,6 @@ struct net_device *rtnl_create_link(struct net *net,
24032401
dev_set_group(dev, nla_get_u32(tb[IFLA_GROUP]));
24042402

24052403
return dev;
2406-
2407-
err:
2408-
return ERR_PTR(err);
24092404
}
24102405
EXPORT_SYMBOL(rtnl_create_link);
24112406

0 commit comments

Comments
 (0)