Roll back change r10695, because new update on r10700 and r10701 handles this issue.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@10704 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
lgao4 2010-07-28 01:43:18 +00:00
parent ab58ce18aa
commit 116b142d86
2 changed files with 4 additions and 40 deletions

View File

@ -122,7 +122,6 @@ InternalGetSectionFromFv (
EFI_STATUS Status; EFI_STATUS Status;
EFI_FIRMWARE_VOLUME2_PROTOCOL *Fv; EFI_FIRMWARE_VOLUME2_PROTOCOL *Fv;
UINT32 AuthenticationStatus; UINT32 AuthenticationStatus;
VOID* TempBuffer;
ASSERT (NameGuid != NULL); ASSERT (NameGuid != NULL);
ASSERT (Buffer != NULL); ASSERT (Buffer != NULL);
@ -171,17 +170,6 @@ InternalGetSectionFromFv (
); );
} }
if (!EFI_ERROR (Status)) {
//
// The found buffer by FV protocol is allocated by gBS AllocatePool() service.
// Copy the found buffer to the allocated buffer by AllocatePool().
// So, the returned buffer can be freed by FreePool().
//
TempBuffer = AllocateCopyPool (*Size, *Buffer);
gBS->FreePool (*Buffer);
*Buffer = TempBuffer;
}
return Status; return Status;
} }
@ -324,11 +312,7 @@ GetSectionFromAnyFvByFileType (
Done: Done:
if (HandleBuffer != NULL) { if (HandleBuffer != NULL) {
// FreePool(HandleBuffer);
// HandleBuffer is allocated by gBS AllocatePool() service.
// So, gBS FreePool() service is used to free HandleBuffer.
//
gBS->FreePool (HandleBuffer);
} }
return Status; return Status;
@ -452,11 +436,7 @@ GetSectionFromAnyFv (
Done: Done:
if (HandleBuffer != NULL) { if (HandleBuffer != NULL) {
// FreePool(HandleBuffer);
// HandleBuffer is allocated by gBS AllocatePool() service.
// So, gBS FreePool() service is used to free HandleBuffer.
//
gBS->FreePool (HandleBuffer);
} }
return Status; return Status;
@ -635,7 +615,6 @@ GetFileBufferByFilePath (
EFI_FIRMWARE_VOLUME2_PROTOCOL *FwVol; EFI_FIRMWARE_VOLUME2_PROTOCOL *FwVol;
EFI_SECTION_TYPE SectionType; EFI_SECTION_TYPE SectionType;
UINT8 *ImageBuffer; UINT8 *ImageBuffer;
UINT8 *TempBuffer;
UINTN ImageBufferSize; UINTN ImageBufferSize;
EFI_FV_FILETYPE Type; EFI_FV_FILETYPE Type;
EFI_FV_FILE_ATTRIBUTES Attrib; EFI_FV_FILE_ATTRIBUTES Attrib;
@ -663,7 +642,6 @@ GetFileBufferByFilePath (
FileInfo = NULL; FileInfo = NULL;
FileHandle = NULL; FileHandle = NULL;
ImageBuffer = NULL; ImageBuffer = NULL;
TempBuffer = NULL;
ImageBufferSize = 0; ImageBufferSize = 0;
*AuthenticationStatus = 0; *AuthenticationStatus = 0;
@ -724,16 +702,6 @@ GetFileBufferByFilePath (
AuthenticationStatus AuthenticationStatus
); );
} }
if (!EFI_ERROR (Status)) {
//
// The found buffer by FV protocol is allocated by gBS AllocatePool() service.
// Copy the found buffer to the allocated buffer by AllocatePool().
// Then, this returned buffer can be freed by FreePool().
//
TempBuffer = AllocateCopyPool (ImageBufferSize, ImageBuffer);
gBS->FreePool (ImageBuffer);
ImageBuffer = TempBuffer;
}
} }
} }
goto Finish; goto Finish;

View File

@ -96,11 +96,7 @@ PciSegmentLibConstructor (
ASSERT (Descriptors->Desc != ACPI_END_TAG_DESCRIPTOR); ASSERT (Descriptors->Desc != ACPI_END_TAG_DESCRIPTOR);
} }
// FreePool(HandleBuffer);
// HandleBuffer is allocated by gBS AllocatePool() service.
// So, gBS FreePool() service is used to free HandleBuffer.
//
gBS->FreePool (HandleBuffer);
return EFI_SUCCESS; return EFI_SUCCESS;
} }