[PATCH] MdeModulePkg/PiSmmCore: Fix potentially uninitialized local variable

[PATCH] MdeModulePkg/PiSmmCore: Fix potentially uninitialized local variable

[Hao Wu]

Cc: Jiewen Yao <jiewen.yao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a.wu@intel.com>
---
 MdeModulePkg/Core/PiSmmCore/Pool.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/MdeModulePkg/Core/PiSmmCore/Pool.c b/MdeModulePkg/Core/PiSmmCore/Pool.c
index f734b3f72d..43ce869d1e 100644
--- a/MdeModulePkg/Core/PiSmmCore/Pool.c
+++ b/MdeModulePkg/Core/PiSmmCore/Pool.c
@@ -1,7 +1,7 @@
 /** @file
   SMM Memory pool management functions.
 
-  Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved.<BR>
   This program and the accompanying materials are licensed and made available 
   under the terms and conditions of the BSD License which accompanies this 
   distribution.  The full text of the license may be found at        
@@ -136,6 +136,7 @@ InternalAllocPoolByIndex (
   EFI_PHYSICAL_ADDRESS  Address;
   SMM_POOL_TYPE         SmmPoolType;
 
+  Address     = 0;
   SmmPoolType = UefiMemoryTypeToSmmPoolType(PoolType);
 
   ASSERT (PoolIndex <= MAX_POOL_INDEX);
@@ -227,6 +228,8 @@ SmmInternalAllocatePool (
   EFI_PHYSICAL_ADDRESS  Address;
   UINTN                 PoolIndex;
 
+  Address = 0;
+
   if (PoolType != EfiRuntimeServicesCode &&
       PoolType != EfiRuntimeServicesData) {
     return EFI_INVALID_PARAMETER;
-- 
2.12.0.windows.1

Re: [PATCH] MdeModulePkg/PiSmmCore: Fix potentially uninitialized local variable

[Yao, Jiewen]

Reviewed-by: jiewen.yao@intel.com

> -----Original Message-----
> From: Wu, Hao A
> Sent: Monday, April 10, 2017 2:15 PM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
> Subject: [PATCH] MdeModulePkg/PiSmmCore: Fix potentially uninitialized local
> variable
> 
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Hao Wu <hao.a.wu@intel.com>
> ---
>  MdeModulePkg/Core/PiSmmCore/Pool.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/MdeModulePkg/Core/PiSmmCore/Pool.c
> b/MdeModulePkg/Core/PiSmmCore/Pool.c
> index f734b3f72d..43ce869d1e 100644
> --- a/MdeModulePkg/Core/PiSmmCore/Pool.c
> +++ b/MdeModulePkg/Core/PiSmmCore/Pool.c
> @@ -1,7 +1,7 @@
>  /** @file
>    SMM Memory pool management functions.
> 
> -  Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved.<BR>
> +  Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved.<BR>
>    This program and the accompanying materials are licensed and made available
>    under the terms and conditions of the BSD License which accompanies this
>    distribution.  The full text of the license may be found at
> @@ -136,6 +136,7 @@ InternalAllocPoolByIndex (
>    EFI_PHYSICAL_ADDRESS  Address;
>    SMM_POOL_TYPE         SmmPoolType;
> 
> +  Address     = 0;
>    SmmPoolType = UefiMemoryTypeToSmmPoolType(PoolType);
> 
>    ASSERT (PoolIndex <= MAX_POOL_INDEX);
> @@ -227,6 +228,8 @@ SmmInternalAllocatePool (
>    EFI_PHYSICAL_ADDRESS  Address;
>    UINTN                 PoolIndex;
> 
> +  Address = 0;
> +
>    if (PoolType != EfiRuntimeServicesCode &&
>        PoolType != EfiRuntimeServicesData) {
>      return EFI_INVALID_PARAMETER;
> --
> 2.12.0.windows.1