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;
        }
 
-- 
2.43.0

Reply via email to