From 6bf123e83d18d4ac06c37ab6a58482d749d3b31a Mon Sep 17 00:00:00 2001 From: Dan Crowell Date: Thu, 16 Aug 2018 15:08:23 -0500 Subject: Remove double free from WOF lookup in HBRT The recent change to use a stored lid image instead of allocating space missed where we tried to free the data. Change-Id: I780a6576019e6904ac31b50fd02d955e7e651e11 CQ: SW442669 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64693 Reviewed-by: Prachi Gupta Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Reviewed-by: Daniel M. Crowell --- src/usr/fapi2/plat_wof_access.C | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'src/usr/fapi2') diff --git a/src/usr/fapi2/plat_wof_access.C b/src/usr/fapi2/plat_wof_access.C index f4a6313c8..bd58fb01a 100644 --- a/src/usr/fapi2/plat_wof_access.C +++ b/src/usr/fapi2/plat_wof_access.C @@ -631,11 +631,7 @@ fapi2::ReturnCode platParseWOFTables(uint8_t* o_wofData) // Free the wof tables memory if(l_pWofImage != nullptr) { -#ifdef __HOSTBOOT_RUNTIME - free(l_pWofImage); - l_pWofImage = nullptr; - -#else +#ifndef __HOSTBOOT_RUNTIME errlHndl_t l_tmpErr = nullptr; // Release the memory we may still have allocated and set the // permissions to prevent further access to it -- cgit v1.2.3