On Tue, Mar 17, 2020 at 07:56:14PM +0100, Olaf Hering wrote:
> A http transfer will hang if an unhandled error is returned.
> The error branch returns the value zero, which is not expected by the caller.
>
> Signed-off-by: Olaf Hering <o...@aepfle.de>
> ---
> grub-core/net/http.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/grub-core/net/http.c b/grub-core/net/http.c
> index b616cf40b..9d92a4905 100644
> --- a/grub-core/net/http.c
> +++ b/grub-core/net/http.c
> @@ -125,7 +125,7 @@ parse_line (grub_file_t file, http_data_t data, char
> *ptr, grub_size_t len)
> valid answers like 403 will trigger this very generic message. */
> data->errmsg = grub_xasprintf (_("unsupported HTTP error %d: %s"),
> code, ptr);
> - return GRUB_ERR_NONE;
> + return GRUB_ERR_FILE_READ_ERROR;
Should not we do the same for 404, file not found, a few lines above?
Daniel
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel
