Hi Babu, If the call to InitializeHeader() fails then the buffer will still not be de-allcoated properly. Please add a Done: label and a goto Done; so that the buffer will still be freed in the case of an error.
Thanks, Nate -----Original Message----- From: Pabba, PhanindraX Babu <[email protected]> Sent: Thursday, February 22, 2024 10:01 PM To: [email protected] Cc: Pabba, PhanindraX Babu <[email protected]>; Chiu, Chasel <[email protected]>; Desimone, Nathaniel L <[email protected]>; Liming Gao <[email protected]>; Dong, Eric <[email protected]> Subject: [PATCH] MinPlatformPkg: Add missed call to FreePool From: PhanindraX Babu Pabba <[email protected]> Adding missed out call to FreePool API to free the allocated memory. Cc: Chasel Chiu <[email protected]> Cc: Nate DeSimone <[email protected]> Cc: Liming Gao <[email protected]> Cc: Eric Dong <[email protected]> Signed-off-by: PhanindraX Babu Pabba <[email protected]> --- Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c | 2 +- Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c b/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c index 2a833ec9..f637e1c9 100644 --- a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c +++ b/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c @@ -1002,7 +1002,7 @@ InstallMcfgFromScratch ( McfgTable->Header.Length, &TableHandle );-+ FreePool (McfgTable); return Status; } diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.c b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.c index 9bc22245..f5b1a71c 100644 --- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.c +++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.c @@ -195,6 +195,7 @@ PublishAcpiTablesFromFv ( // Increment the instance // Instance++;+ FreePool (CurrentTable); CurrentTable = NULL; } }-- 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117095): https://edk2.groups.io/g/devel/message/117095 Mute This Topic: https://groups.io/mt/104524859/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
