]> git.dujemihanovic.xyz Git - u-boot.git/commit
net: lwip: wget: fix legacy syntax
authorJerome Forissier <jerome.forissier@linaro.org>
Tue, 26 Nov 2024 14:45:06 +0000 (15:45 +0100)
committerTom Rini <trini@konsulko.com>
Sat, 30 Nov 2024 17:52:47 +0000 (11:52 -0600)
commita37064cb2e482739908e76d6a53914301c8bb8ee
treebfe8f68f4e11ff45a579b66d280cfa27b3ed8099
parent65b38a519b0fcae6a76f48116a5f999400d7294c
net: lwip: wget: fix legacy syntax

Commit 356011f7ac25 ("lwip: fix code style issues") has inadvertently
broken the support for the legacy syntax:

 => wget 192.168.0.16:test.bin
 invalid uri, no file path
 Invalid URL. Use http(s)://

The reason is two calls to strncpy() were replaced by strlcpy() without
paying attention to the fact that they are not equivalent in the present
case. Since we are using a character counter (n) and since we do not
depend on having a properly null-terminated string at each step in the
parsing, strlcpy() is not justified and strncpy() is the right tool for
the job. So use it again.

Reported-by: Tom Rini <trini@konsulko.com>
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
net/lwip/wget.c