[PATCH v1 1/2] MdeModulePkg: BdsDxe: Uninitialized local variable.


Paul
 

From: Paul Grimes <paul.grimes@...>

In function BdsFormalizeConsoleVariable(), VariableSize is used
prior to being initialized. This change initializes the variable.

Cc: Zhiguang Liu <zhiguang.liu@...>
Cc: Ray Ni <ray.ni@...>
Cc: Gua Guo <gua.guo@...>
Signed-off-by: Paul Grimes <paul.grimes@...>
---
MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/MdeModulePkg/Universal/BdsDxe/BdsEntry.c b/MdeModulePkg/Universal/BdsDxe/BdsEntry.c
index 766dde3aaeeb..455fc344f0be 100644
--- a/MdeModulePkg/Universal/BdsDxe/BdsEntry.c
+++ b/MdeModulePkg/Universal/BdsDxe/BdsEntry.c
@@ -518,6 +518,8 @@ BdsFormalizeConsoleVariable (
UINTN VariableSize;
EFI_STATUS Status;

+ VariableSize = 0;
+
GetEfiGlobalVariable2 (VariableName, (VOID **)&DevicePath, &VariableSize);
if ((DevicePath != NULL) && !IsDevicePathValid (DevicePath, VariableSize)) {
Status = gRT->SetVariable (
--
2.24.0.windows.2