From 356e635c15556b50ecfba55f4e0e78718da9f788 Mon Sep 17 00:00:00 2001 From: eric_tian Date: Fri, 16 May 2008 07:14:25 +0000 Subject: [PATCH] [Description] Memory may leak because the handles in IScsiUpdateDeviceList is not freed if the "iSCSIDeviceList" already exists [Impaction] add Free() func to free memory properly. [Reference Info] The Handles in IScsiUpdateDeviceList is not freed if the "iSCSIDeviceList" already exists git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@5211 6f19259b-4bc3-4df7-8a09-765794883524 --- MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfig.c | 1 + 1 file changed, 1 insertion(+) diff --git a/MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfig.c b/MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfig.c index e196ffda3d..0f496a02c1 100644 --- a/MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfig.c +++ b/MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfig.c @@ -205,6 +205,7 @@ IScsiUpdateDeviceList ( ); gBS->FreePool (DeviceList); + gBS->FreePool (Handles); return Status; }