# Re: \[PATCH 19/20\] usbip: usbip\_host: Manual replacement of the deprecated strlcpy() with return values

- From: Sergei Shtylyov \<hidden\>
- Date: 2021-02-28 09:05:02 
- List: linux-usb
- Page: https://ratatoskr.run/linux-usb/2021/02/6280818
- Thread: https://ratatoskr.run/linux-usb/2021/02/6275721/t.md

```text
Hello!

On 22.02.2021 18:12, Romain Perier wrote:

> The strlcpy() reads the entire source buffer first, it is dangerous if
> the source buffer lenght is unbounded or possibility non NULL-terminated.

   Length. Possibly?

> It can lead to linear read overflows, crashes, etc...
>
> As recommended in the deprecated interfaces [1], it should be replaced
> by strscpy.
>
> This commit replaces all calls to strlcpy that handle the return values
> by the corresponding strscpy calls with new handling of the return
> values (as it is quite different between the two functions).
>
> [1] https://www.kernel.org/doc/html/latest/process/deprecated.html#strlcpy
>
> Signed-off-by: Romain Perier <romain.perier@gmail.com>
[...]

MBR, Sergei
```