Messages in this thread Patch in this message | | | From | Oleksandr Tyshchenko <> | Subject | Re: [PATCH] xen: move from strlcpy with unused retval to strscpy | Date | Tue, 23 Aug 2022 13:46:43 +0000 |
| |
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@mail.gmail.com/__;!!GF_29dbcQIUBPA!x8tXRXTdzbSmaWA3GH-Aj-ZF2HrMIAVOr_qerO6avg05AKhT0RI7MnqV64DykFFHfuDwoDYtT-i5bw7nGYSpneDqzzGfhYFHcm7LBQ$ [lore[.]kernel[.]org] > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Looks correct:
Reviewed-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@epam.com>
> --- > 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
| |