On 19.08.22 00:01, Wolfram Sang wrote: Hello Wolfram > 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://urldefense.com/v3/__https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw@xxxxxxxxxxxxxx/__;!!GF_29dbcQIUBPA!x8tXRXTdzbSmaWA3GH-Aj-ZF2HrMIAVOr_qerO6avg05AKhT0RI7MnqV64DykFFHfuDwoDYtT-i5bw7nGYSpneDqzzGfhYFHcm7LBQ$ [lore[.]kernel[.]org] > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Looks correct: Reviewed-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx> > --- > drivers/xen/xen-scsiback.c | 2 +- > drivers/xen/xenbus/xenbus_probe_frontend.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/xen/xen-scsiback.c b/drivers/xen/xen-scsiback.c > index 7a0c93acc2c5..d3dcda344989 100644 > --- a/drivers/xen/xen-scsiback.c > +++ b/drivers/xen/xen-scsiback.c > @@ -1121,7 +1121,7 @@ static void scsiback_do_1lun_hotplug(struct vscsibk_info *info, int op, > "%s: writing %s", __func__, state); > return; > } > - strlcpy(phy, val, VSCSI_NAMELEN); > + strscpy(phy, val, VSCSI_NAMELEN); > kfree(val); > > /* virtual SCSI device */ > diff --git a/drivers/xen/xenbus/xenbus_probe_frontend.c b/drivers/xen/xenbus/xenbus_probe_frontend.c > index 07b010a68fcf..f44d5a64351e 100644 > --- a/drivers/xen/xenbus/xenbus_probe_frontend.c > +++ b/drivers/xen/xenbus/xenbus_probe_frontend.c > @@ -40,7 +40,7 @@ static int frontend_bus_id(char bus_id[XEN_BUS_ID_SIZE], const char *nodename) > return -EINVAL; > } > > - strlcpy(bus_id, nodename + 1, XEN_BUS_ID_SIZE); > + strscpy(bus_id, nodename + 1, XEN_BUS_ID_SIZE); > if (!strchr(bus_id, '/')) { > pr_warn("bus_id %s no slash\n", bus_id); > return -EINVAL; -- Regards, Oleksandr Tyshchenko
