Reviewed-by: Qiu Shumin <[email protected]>

-----Original Message-----
From: Cinnamon Shia [mailto:[email protected]] 
Sent: Monday, March 28, 2016 11:03 AM
To: [email protected]
Cc: Carsey, Jaben; Qiu, Shumin; Zeng, Star; [email protected]; Cinnamon Shia
Subject: [PATCH 1/2] PerformancePkg/Dp_App: Fix a memory leak issue in Dp.

The allocated memory of ParamPackage is not freed.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Cinnamon Shia <[email protected]>
---
 PerformancePkg/Dp_App/Dp.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/PerformancePkg/Dp_App/Dp.c b/PerformancePkg/Dp_App/Dp.c index 
e36a032..b24a0de 100644
--- a/PerformancePkg/Dp_App/Dp.c
+++ b/PerformancePkg/Dp_App/Dp.c
@@ -526,6 +526,9 @@ Done:
   SafeFreePool (StringPtr);
   SafeFreePool (mPrintTokenBuffer);
 
+  if (ParamPackage != NULL) {
+    ShellCommandLineFreeVarList (ParamPackage);  }
   if (CustomCumulativeData != NULL) {
     SafeFreePool (CustomCumulativeData->Name);
   }
--
2.7.1.windows.2

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to