Re: [PATCH v2 08/14] net-lwip: add wget command

2024-06-06 Thread Maxim Uvarov
чт, 6 июн. 2024 г. в 12:56, Jerome Forissier : > > > > On 5/28/24 15:39, Maxim Uvarov wrote: > > пт, 24 мая 2024 г. в 19:22, Jerome Forissier : > >> > >> Add support for the wget command with NET_LWIP. > >> > >> About the small change in cmd/efidebug.c: when the wget command based > >> on the lwIP

Re: [PATCH v2 08/14] net-lwip: add wget command

2024-06-06 Thread Jerome Forissier
On 6/6/24 12:16, Maxim Uvarov wrote: > чт, 6 июн. 2024 г. в 12:56, Jerome Forissier : [...] diff --git a/net-lwip/wget.c b/net-lwip/wget.c new file mode 100644 index 000..25b75040806 --- /dev/null +++ b/net-lwip/wget.c @@ -0,0 +1,180 @@ +// SPDX-Licen

Re: [PATCH v2 08/14] net-lwip: add wget command

2024-06-06 Thread Jerome Forissier
On 5/28/24 15:39, Maxim Uvarov wrote: > пт, 24 мая 2024 г. в 19:22, Jerome Forissier : >> >> Add support for the wget command with NET_LWIP. >> >> About the small change in cmd/efidebug.c: when the wget command based >> on the lwIP stack is used the wget command has a built-in URL >> validation

Re: [PATCH v2 08/14] net-lwip: add wget command

2024-06-06 Thread Ilias Apalodimas
Hi Jerome > if (!label) > return CMD_RET_FAILURE; > > - if (!wget_validate_uri(argv[3])) { > - printf("ERROR: invalid URI\n"); > - return CMD_RET_FAILURE; > + if (IS_ENABLED(CONFIG_CMD_WGET)) { efi_boot_add_uri() is only called if CONFIG_EFI_HTT

Re: [PATCH v2 08/14] net-lwip: add wget command

2024-05-28 Thread Maxim Uvarov
пт, 24 мая 2024 г. в 19:22, Jerome Forissier : > > Add support for the wget command with NET_LWIP. > > About the small change in cmd/efidebug.c: when the wget command based > on the lwIP stack is used the wget command has a built-in URL > validation function since it needs to parse it anyways (in p

[PATCH v2 08/14] net-lwip: add wget command

2024-05-24 Thread Jerome Forissier
Add support for the wget command with NET_LWIP. About the small change in cmd/efidebug.c: when the wget command based on the lwIP stack is used the wget command has a built-in URL validation function since it needs to parse it anyways (in parse_url()). Therefore wget_validate_uri() doesn't exist.