net: mdio_bus: replace deprecated strncpy with strscpy
authorJustin Stitt <justinstitt@google.com>
Thu, 7 Dec 2023 21:57:50 +0000 (21:57 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 10 Dec 2023 19:08:58 +0000 (19:08 +0000)
commit1674110c0dd44a6240f03fff9a05fd72917b3f7d
tree058827875588a2bfb7ccfc5b70e8fa822ff4f1ea
parentf8dd2412ba66128de4325c187c9504c6da511bc8
net: mdio_bus: replace deprecated strncpy with strscpy

strncpy() is deprecated for use on NUL-terminated destination strings
[1] and as such we should prefer more robust and less ambiguous string
interfaces.

We expect mdiodev->modalias to be NUL-terminated based on its usage with
strcmp():
|       return strcmp(mdiodev->modalias, drv->name) == 0;

Moreover, mdiodev->modalias is already zero-allocated:
|       mdiodev = kzalloc(sizeof(*mdiodev), GFP_KERNEL);
... which means the NUL-padding strncpy provides is not necessary.

Considering the above, a suitable replacement is `strscpy` [2] due to
the fact that it guarantees NUL-termination on the destination buffer
without unnecessarily NUL-padding.

Link: https://www.kernel.org/doc/html/latest/process/deprecated.html#strncpy-on-nul-terminated-strings
Link: https://manpages.debian.org/testing/linux-manual-4.8/strscpy.9.en.html
Link: https://github.com/KSPP/linux/issues/90
Cc: linux-hardening@vger.kernel.org
Signed-off-by: Justin Stitt <justinstitt@google.com>
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/mdio_bus.c