From 676df92c2c0c5bdeb0f8e27349f5dd467928ce09 Mon Sep 17 00:00:00 2001 From: qwang12 Date: Thu, 30 Oct 2008 07:32:46 +0000 Subject: Remove SafeFreePool from MemoryAllocationLib as this API's name is misleading. Its implementation only check if a pointer is NULL. If a garbage pointer is passed in, the gBS->FreePool will still ASSERT in debug build and return error code. It is recommended that module writer should keep track how a pointer is allocated and free it after use. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@6306 6f19259b-4bc3-4df7-8a09-765794883524 --- MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatform.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'MdeModulePkg/Universal/Console/ConPlatformDxe') diff --git a/MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatform.c b/MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatform.c index c2540795cd..2743dd8d04 100644 --- a/MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatform.c +++ b/MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatform.c @@ -795,7 +795,9 @@ ConPlatformMatchDevicePaths ( TempDevicePath1, DevicePathInst ); - SafeFreePool (TempDevicePath1); + if (TempDevicePath1 != NULL) { + FreePool (TempDevicePath1); + } TempDevicePath1 = TempDevicePath2; } } -- cgit v1.2.3