diff options
author | lgao4 <lgao4@6f19259b-4bc3-4df7-8a09-765794883524> | 2010-07-28 01:43:18 +0000 |
---|---|---|
committer | lgao4 <lgao4@6f19259b-4bc3-4df7-8a09-765794883524> | 2010-07-28 01:43:18 +0000 |
commit | 116b142d869c1a37893ab435045e38a7881dd18d (patch) | |
tree | f68bffec96bb6b413a77ad04600afed9d72545ab /MdePkg/Library | |
parent | ab58ce18aa8c979e35c6ac7750ae97f871145c50 (diff) | |
download | edk2-platforms-116b142d869c1a37893ab435045e38a7881dd18d.tar.xz |
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
Diffstat (limited to 'MdePkg/Library')
-rw-r--r-- | MdePkg/Library/DxeServicesLib/DxeServicesLib.c | 38 | ||||
-rw-r--r-- | MdePkg/Library/UefiPciSegmentLibPciRootBridgeIo/PciSegmentLib.c | 6 |
2 files changed, 4 insertions, 40 deletions
diff --git a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c index 9cf845d6fc..d8e6b5cc4e 100644 --- a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c +++ b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c @@ -122,7 +122,6 @@ InternalGetSectionFromFv ( EFI_STATUS Status;
EFI_FIRMWARE_VOLUME2_PROTOCOL *Fv;
UINT32 AuthenticationStatus;
- VOID* TempBuffer;
ASSERT (NameGuid != 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;
}
@@ -324,11 +312,7 @@ GetSectionFromAnyFvByFileType ( Done:
if (HandleBuffer != NULL) {
- //
- // HandleBuffer is allocated by gBS AllocatePool() service.
- // So, gBS FreePool() service is used to free HandleBuffer.
- //
- gBS->FreePool (HandleBuffer);
+ FreePool(HandleBuffer);
}
return Status;
@@ -451,12 +435,8 @@ GetSectionFromAnyFv ( Done:
- if (HandleBuffer != NULL) {
- //
- // HandleBuffer is allocated by gBS AllocatePool() service.
- // So, gBS FreePool() service is used to free HandleBuffer.
- //
- gBS->FreePool (HandleBuffer);
+ if (HandleBuffer != NULL) {
+ FreePool(HandleBuffer);
}
return Status;
@@ -635,7 +615,6 @@ GetFileBufferByFilePath ( EFI_FIRMWARE_VOLUME2_PROTOCOL *FwVol;
EFI_SECTION_TYPE SectionType;
UINT8 *ImageBuffer;
- UINT8 *TempBuffer;
UINTN ImageBufferSize;
EFI_FV_FILETYPE Type;
EFI_FV_FILE_ATTRIBUTES Attrib;
@@ -663,7 +642,6 @@ GetFileBufferByFilePath ( FileInfo = NULL;
FileHandle = NULL;
ImageBuffer = NULL;
- TempBuffer = NULL;
ImageBufferSize = 0;
*AuthenticationStatus = 0;
@@ -724,16 +702,6 @@ GetFileBufferByFilePath ( 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;
diff --git a/MdePkg/Library/UefiPciSegmentLibPciRootBridgeIo/PciSegmentLib.c b/MdePkg/Library/UefiPciSegmentLibPciRootBridgeIo/PciSegmentLib.c index d6527ef301..ccd9df34d0 100644 --- a/MdePkg/Library/UefiPciSegmentLibPciRootBridgeIo/PciSegmentLib.c +++ b/MdePkg/Library/UefiPciSegmentLibPciRootBridgeIo/PciSegmentLib.c @@ -96,11 +96,7 @@ PciSegmentLibConstructor ( ASSERT (Descriptors->Desc != ACPI_END_TAG_DESCRIPTOR);
}
- //
- // HandleBuffer is allocated by gBS AllocatePool() service.
- // So, gBS FreePool() service is used to free HandleBuffer.
- //
- gBS->FreePool (HandleBuffer);
+ FreePool(HandleBuffer);
return EFI_SUCCESS;
}
|