l2tp: move from strlcpy with unused retval to strscpy
Follow the advice of the below link and prefer 'strscpy' in this subsystem. Conversion is 1:1 because the return value is not used. Generated by a coccinelle script. Link: https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw@mail.gmail.com/ Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Link: https://lore.kernel.org/r/20220818210222.8515-1-wsa+renesas@sang-engineering.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
7574cc5837
commit
a5afe5305d
|
@ -254,7 +254,7 @@ static int l2tp_eth_create(struct net *net, struct l2tp_tunnel *tunnel,
|
|||
int rc;
|
||||
|
||||
if (cfg->ifname) {
|
||||
strlcpy(name, cfg->ifname, IFNAMSIZ);
|
||||
strscpy(name, cfg->ifname, IFNAMSIZ);
|
||||
name_assign_type = NET_NAME_USER;
|
||||
} else {
|
||||
strcpy(name, L2TP_ETH_DEV_NAME);
|
||||
|
@ -314,7 +314,7 @@ static int l2tp_eth_create(struct net *net, struct l2tp_tunnel *tunnel,
|
|||
return rc;
|
||||
}
|
||||
|
||||
strlcpy(session->ifname, dev->name, IFNAMSIZ);
|
||||
strscpy(session->ifname, dev->name, IFNAMSIZ);
|
||||
rcu_assign_pointer(spriv->dev, dev);
|
||||
|
||||
rtnl_unlock();
|
||||
|
|
Loading…
Reference in New Issue