Skip to content

Commit

Permalink
httpboot: Convert HTTP status codes to EFI status codes
Browse files Browse the repository at this point in the history
This allows the caller to make a more informed decision about how to
handle the error. In particular, by mapping HTTP errors to
EFI_INVALID_PARAMETER and EFI_NOT_FOUND, shim will try to fetch the
default second stage image like it does when loading images from disk.

Note that this also changes the default to return EFI_HTTP_ERROR instead
of EFI_ABORTED. This should not change any behavior as EFI_ABORTED
wasn't being handled before, but letting the caller decide what to do
with an unknown HTTP error is more appropriate.

Signed-off-by: Dan Nicholson <dbn@endlessos.org>
  • Loading branch information
dbnicholson committed Oct 4, 2024
1 parent 4fa0495 commit bd3a507
Showing 1 changed file with 46 additions and 1 deletion.
47 changes: 46 additions & 1 deletion httpboot.c
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,51 @@ convert_http_status_code (EFI_HTTP_STATUS_CODE status_code)
return 0;
}

/* Convert an HTTP status code to an EFI status code. */
static EFI_STATUS
efi_status_from_http_status(EFI_HTTP_STATUS_CODE status_code)
{
switch (status_code) {
case HTTP_STATUS_400_BAD_REQUEST:
case HTTP_STATUS_411_LENGTH_REQUIRED:
case HTTP_STATUS_413_REQUEST_ENTITY_TOO_LARGE:
case HTTP_STATUS_414_REQUEST_URI_TOO_LARGE:
case HTTP_STATUS_415_UNSUPPORTED_MEDIA_TYPE:
case HTTP_STATUS_416_REQUESTED_RANGE_NOT_SATISFIED:
case HTTP_STATUS_417_EXPECTATION_FAILED:
return EFI_INVALID_PARAMETER;
case HTTP_STATUS_401_UNAUTHORIZED:
case HTTP_STATUS_402_PAYMENT_REQUIRED:
case HTTP_STATUS_403_FORBIDDEN:
case HTTP_STATUS_407_PROXY_AUTHENTICATION_REQUIRED:
return EFI_ACCESS_DENIED;
case HTTP_STATUS_404_NOT_FOUND:
case HTTP_STATUS_410_GONE:
return EFI_NOT_FOUND;
case HTTP_STATUS_405_METHOD_NOT_ALLOWED:
case HTTP_STATUS_501_NOT_IMPLEMENTED:
return EFI_UNSUPPORTED;
case HTTP_STATUS_406_NOT_ACCEPTABLE:
return EFI_NO_MEDIA;
case HTTP_STATUS_408_REQUEST_TIME_OUT:
case HTTP_STATUS_504_GATEWAY_TIME_OUT:
return EFI_TIMEOUT;
case HTTP_STATUS_409_CONFLICT:
case HTTP_STATUS_412_PRECONDITION_FAILED:
return EFI_MEDIA_CHANGED;
case HTTP_STATUS_500_INTERNAL_SERVER_ERROR:
case HTTP_STATUS_502_BAD_GATEWAY:
return EFI_DEVICE_ERROR;
case HTTP_STATUS_503_SERVICE_UNAVAILABLE:
return EFI_NOT_READY;
case HTTP_STATUS_505_HTTP_VERSION_NOT_SUPPORTED:
return EFI_INCOMPATIBLE_VERSION;
default:
/* Use a generic HTTP error for anything else. */
return EFI_HTTP_ERROR;
}
}

static EFI_DEVICE_PATH *devpath;
static EFI_MAC_ADDRESS mac_addr;
static IPv4_DEVICE_PATH ip4_node;
Expand Down Expand Up @@ -565,7 +610,7 @@ receive_http_response(EFI_HTTP_PROTOCOL *http, VOID **buffer, UINT64 *buf_size)
if (http_status != HTTP_STATUS_200_OK) {
perror(L"HTTP Status Code: %d\n",
convert_http_status_code(http_status));
efi_status = EFI_ABORTED;
efi_status = efi_status_from_http_status(http_status);
goto error;
}

Expand Down

0 comments on commit bd3a507

Please sign in to comment.