[edk2-devel] [PATCH 1/2] RedfishPkg: fix config handler driver bug
Chang, Abner via groups.io
abner.chang=amd.com at groups.io
Sat Feb 4 03:51:29 UTC 2023
[AMD Official Use Only - General]
Reviewed-by: Abner Chang <abner.chang at amd.com>
> -----Original Message-----
> From: Nickle Wang <nicklew at nvidia.com>
> Sent: Wednesday, February 1, 2023 11:05 AM
> To: devel at edk2.groups.io
> Cc: Chang, Abner <Abner.Chang at amd.com>; Igor Kulchytskyy
> <igork at ami.com>; Nick Ramirez <nramirez at nvidia.com>
> Subject: [PATCH 1/2] RedfishPkg: fix config handler driver bug
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Bug fix:
> - function stack fault
> - config handler driver requires the dependency of Redfish Credential
> Protocol
> - incorrect caller id installation when failed to perform init()
>
> Signed-off-by: Nickle Wang <nicklew at nvidia.com>
> Cc: Abner Chang <abner.chang at amd.com>
> Cc: Igor Kulchytskyy <igork at ami.com>
> Cc: Nick Ramirez <nramirez at nvidia.com>
> ---
> .../RedfishConfigHandler/RedfishConfigHandlerDriver.inf | 4 ++--
> RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerCommon.c | 4 +++-
> 2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerDriver.inf
> b/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerDriver.inf
> index def91c7531..b167c6e1ee 100644
> --- a/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerDriver.inf
> +++ b/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerDriver.inf
> @@ -4,6 +4,7 @@
> #
> # Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> # (C)
> Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
> +# Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> #
> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -56,5 +57,4 @@
> gEfiEndOfDxeEventGroupGuid ## CONSUMES ## Event
>
> [Depex]
> - TRUE
> -
> + gEdkIIRedfishCredentialProtocolGuid
> diff --git a/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerCommon.c
> b/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerCommon.c
> index c3a7efd796..45276793c7 100644
> --- a/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerCommon.c
> +++ b/RedfishPkg/RedfishConfigHandler/RedfishConfigHandlerCommon.c
> @@ -2,6 +2,7 @@
> The common code of EDKII Redfish Configuration Handler driver.
>
> (C) Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
> + Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -225,7 +226,7 @@ RedfishConfigHandlerInitialization (
> UINTN NumberOfHandles;
> EDKII_REDFISH_CONFIG_HANDLER_PROTOCOL *ConfigHandler;
> UINTN Index;
> - UINT32 Id;
> + UINT32 *Id;
>
> Status = gBS->LocateHandleBuffer (
> ByProtocol,
> @@ -257,6 +258,7 @@ RedfishConfigHandlerInitialization (
> Status = ConfigHandler->Init (ConfigHandler,
> &gRedfishConfigData.RedfishServiceInfo);
> if (EFI_ERROR (Status) && (Status != EFI_ALREADY_STARTED)) {
> DEBUG ((DEBUG_ERROR, "ERROR: Failed to init Redfish config handler
> %p.\n", ConfigHandler));
> + continue;
> }
>
> //
> --
> 2.39.1.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#99628): https://edk2.groups.io/g/devel/message/99628
Mute This Topic: https://groups.io/mt/96668712/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list