From 7275993dc64481b1c21d7d70af434bfaafe86e81 Mon Sep 17 00:00:00 2001 From: Nickle Wang Date: Fri, 21 Jul 2023 21:35:51 +0800 Subject: RedfishPkg/RedfishRestExDxe: return HTTP status code to caller. Return unsupported HTTP status code to caller so caller can handle HTTP error status code. Current implementation only return EFI error to caller. Without knowing the HTTP status code, caller has trouble to handle HTTP request failure. Signed-off-by: Nickle Wang Cc: Abner Chang Cc: Igor Kulchytskyy Cc: Nick Ramirez Cc: Mike Maslenkin Reviewed-by: Igor Kulchytskyy Reviewed-by: Abner Chang Acked-by: Mike Maslenkin --- .../RedfishRestExDxe/RedfishRestExProtocol.c | 27 +++++++++++----------- 1 file changed, 14 insertions(+), 13 deletions(-) (limited to 'RedfishPkg') diff --git a/RedfishPkg/RedfishRestExDxe/RedfishRestExProtocol.c b/RedfishPkg/RedfishRestExDxe/RedfishRestExProtocol.c index 9097361..5bcdade 100644 --- a/RedfishPkg/RedfishRestExDxe/RedfishRestExProtocol.c +++ b/RedfishPkg/RedfishRestExDxe/RedfishRestExProtocol.c @@ -29,7 +29,8 @@ EFI_REST_EX_PROTOCOL mRedfishRestExProtocol = { response when the data is retrieved from the service. RequestMessage contains the HTTP request to the REST resource identified by RequestMessage.Request.Url. The ResponseMessage is the returned HTTP response for that request, including any HTTP - status. + status. It's caller's responsibility to free this ResponseMessage using FreePool(). + RestConfigFreeHttpMessage() in RedfishLib is an example to release ResponseMessage structure. @param[in] This Pointer to EFI_REST_EX_PROTOCOL instance for a particular REST service. @@ -320,6 +321,18 @@ ReSendRequest:; DEBUG ((DEBUG_ERROR, "This HTTP Status is not handled!\n")); DumpHttpStatusCode (DEBUG_REDFISH_NETWORK, ResponseData->Response.StatusCode); Status = EFI_UNSUPPORTED; + + // + // Deliver status code back to caller so caller can handle it. + // + ResponseMessage->Data.Response = AllocateZeroPool (sizeof (EFI_HTTP_RESPONSE_DATA)); + if (ResponseMessage->Data.Response == NULL) { + Status = EFI_OUT_OF_RESOURCES; + goto ON_EXIT; + } + + ResponseMessage->Data.Response->StatusCode = ResponseData->Response.StatusCode; + goto ON_EXIT; } @@ -443,18 +456,6 @@ ON_EXIT: FreePool (ResponseData); } - if (EFI_ERROR (Status)) { - if (ResponseMessage->Data.Response != NULL) { - FreePool (ResponseMessage->Data.Response); - ResponseMessage->Data.Response = NULL; - } - - if (ResponseMessage->Body != NULL) { - FreePool (ResponseMessage->Body); - ResponseMessage->Body = NULL; - } - } - return Status; } -- cgit v1.1