On 2/4/25 17:00, Jerome Forissier wrote:
net/lwip/wget.c/mbedtls_hardware_poll() is calling dm_rng_read() but dependency is not recorded anywhere that's why depend on DM_RNG when WGET_HTTPS is used. Suggested-by: Michal Simek <michal.si...@amd.com> Co-developed-by: Ilias Apalodimas <ilias.apalodi...@linaro.org> Signed-off-by: Ilias Apalodimas <ilias.apalodi...@linaro.org> Signed-off-by: Jerome Forissier <jerome.foriss...@linaro.org> --- cmd/Kconfig | 1 + net/lwip/wget.c | 11 ++++++++--- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/cmd/Kconfig b/cmd/Kconfig index e62817bd977..e3fbb7538ee 100644 --- a/cmd/Kconfig +++ b/cmd/Kconfig @@ -2154,6 +2154,7 @@ config WGET_HTTPS depends on CMD_WGET depends on PROT_TCP_LWIP depends on MBEDTLS_LIB + depends on DM_RNG select SHA256 select RSA select ASYMMETRIC_KEY_TYPE diff --git a/net/lwip/wget.c b/net/lwip/wget.c index b76f6c0f1d9..de688641dd7 100644 --- a/net/lwip/wget.c +++ b/net/lwip/wget.c @@ -433,10 +433,15 @@ bool wget_validate_uri(char *uri)if (!strncmp(uri, "http://";, strlen("http://";))) {prefix_len = strlen("http://";); - } else if (!strncmp(uri, "https://";, strlen("https://";))) { - prefix_len = strlen("https://";); + } else if (CONFIG_IS_ENABLED(WGET_HTTPS)) { + if (!strncmp(uri, "https://";, strlen("https://";))) { + prefix_len = strlen("https://";); + } else { + log_err("only http(s):// is supported\n"); + return false; + } } else { - log_err("only http(s):// is supported\n"); + log_err("only http:// is supported\n"); return false; }
Works for me. Tested-by: Michal Simek <michal.si...@amd.com> Thanks, Michal
