Fix memory leak issues.

Signed-off-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Reviewed-by: Li Elvin <elvin.li@intel.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@13288 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsMisc.c b/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsMisc.c
index 9993e4b19d4bdfcfc242c45af99ecd43d2f2bfc2..ada77d26ded416c8ad455a573f680bf5c29c7596 100644 (file)
--- a/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsMisc.c
+++ b/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsMisc.c
@@ -739,9 +739,11 @@ BdsLibVariableToOption (
   }\r
 \r
   FreePool (Variable);\r
+  FreePool (Option->Description);\r
+  FreePool (Option->DevicePath);\r
+  FreePool (Option->LoadOptions);\r
   FreePool (Option);\r
   return NULL;\r
-\r
 }\r
 \r
 /**\r

diff --git a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
index 2419b7c4a8dcc777e1e2d189a974aa01fccfd2a4..117569fa624aa6e333ef02c2b208e3ea6142d6f7 100644 (file)
--- a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
+++ b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
@@ -758,6 +758,7 @@ CheckModeSupported (
           //\r
           // If video device has been set to this mode, we do not need to SetMode again\r
           //\r
+          FreePool (Info);\r
           break;\r
         } else {\r
           Status = GraphicsOutput->SetMode (GraphicsOutput, ModeNumber);\r

diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/String.c b/MdeModulePkg/Universal/HiiDatabaseDxe/String.c
index 52d348c5720e50302d8a1096559fbd06895693ad..a76e5b64c93199f3144ac0615f6a54da5220df0d 100644 (file)
--- a/MdeModulePkg/Universal/HiiDatabaseDxe/String.c
+++ b/MdeModulePkg/Universal/HiiDatabaseDxe/String.c
@@ -2028,6 +2028,8 @@ HiiCompareLanguage (
       //\r
       // Return FALSE if any characters are different.\r
       //\r
+      FreePool (Lan1);\r
+      FreePool (Lan2);\r
       return FALSE;\r
     }\r
   }\r