diff options
author | Dandan Bi <dandan.bi@intel.com> | 2016-05-27 16:00:28 +0800 |
---|---|---|
committer | Star Zeng <star.zeng@intel.com> | 2016-11-25 11:00:23 +0800 |
commit | 384df7b0e02f855e6e272b4f179ae41fcfe6ccc1 (patch) | |
tree | bd924ec4848b818758b398f0deeac367e6e97cdd | |
parent | 0c01c6713506249cb53c436965d036871736e19e (diff) | |
download | edk2-384df7b0e02f855e6e272b4f179ae41fcfe6ccc1.zip edk2-384df7b0e02f855e6e272b4f179ae41fcfe6ccc1.tar.gz edk2-384df7b0e02f855e6e272b4f179ae41fcfe6ccc1.tar.bz2 |
MdeModulePkg: Clear the buffer after using it
Cc: Liming Gao <liming.gao@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
(cherry picked from commit a275df8f9a4d2a43772ea5d1765b541a1191ad3f)
-rw-r--r-- | MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c | 3 | ||||
-rw-r--r-- | MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c | 1 |
2 files changed, 3 insertions, 1 deletions
diff --git a/MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c b/MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c index b8337f2..c5a206c 100644 --- a/MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c +++ b/MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c @@ -918,7 +918,8 @@ PasswordProcess ( Status = EFI_INVALID_PARAMETER;
}
-
+ ZeroMem (TempString, (Maximum + 1) * sizeof (CHAR16));
+ ZeroMem (StringPtr, (Maximum + 1) * sizeof (CHAR16));
FreePool (TempString);
FreePool (StringPtr);
diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c index 356cd9c..e65a6b9 100644 --- a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c +++ b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c @@ -1615,6 +1615,7 @@ ProcessUserInput ( DeleteString(Statement->HiiValue.Value.string, gCurrentSelection->FormSet->HiiHandle);
Statement->HiiValue.Value.string = UserInput->InputValue.Value.string;
CopyMem (Statement->BufferValue, UserInput->InputValue.Buffer, (UINTN) UserInput->InputValue.BufferLen);
+ ZeroMem (UserInput->InputValue.Buffer, (UINTN) UserInput->InputValue.BufferLen);
FreePool (UserInput->InputValue.Buffer);
//
// Two password match, send it to Configuration Driver
|