Messages in this thread | | | Subject | Re: [PATCH] usb: move from strlcpy with unused retval to strscpy | From | Shuah Khan <> | Date | Thu, 18 Aug 2022 15:17:40 -0600 |
| |
On 8/18/22 3:01 PM, Wolfram Sang wrote: > 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. >
Please elaborate and summarize why this change makses sense in the commit log? It will be easier for reviewers - saves time checking the link.
> Link: https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw@mail.gmail.com/ > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > --- > drivers/usb/atm/usbatm.c | 2 +- > drivers/usb/core/devio.c | 2 +- > drivers/usb/gadget/function/f_fs.c | 2 +- > drivers/usb/gadget/function/f_uvc.c | 2 +- > drivers/usb/gadget/function/u_ether.c | 8 ++++---- > drivers/usb/gadget/function/uvc_v4l2.c | 6 +++--- > drivers/usb/gadget/udc/omap_udc.c | 2 +- > drivers/usb/misc/usb251xb.c | 6 +++--- > drivers/usb/storage/onetouch.c | 2 +- > drivers/usb/typec/tcpm/fusb302.c | 2 +- > drivers/usb/usbip/stub_main.c | 2 +-
I don't have any problems with the change itself. I would ike to see the commit log clealrly state why this change is good. With that for usbip change:
Acked-by: Shuah Khan <skhan@linuxfoundation.org>
thanks, -- Shuah
| |