MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the
mZeroGuid defined in VariableAuthenticated/RuntimeDxe is redundant.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a...@intel.com>
Reviewed-by: Liming Gao <liming....@intel.com>
Reviewed-by: Jaben Carsey <jaben.car...@intel.com>
---
 SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.c   | 4 ++--
 SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.inf | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.c 
b/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.c
index 1eb169a..14e66e1 100644
--- a/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.c
+++ b/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.c
@@ -36,13 +36,13 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER 
EXPRESS OR IMPLIED.
 
 #include <Guid/AuthenticatedVariableFormat.h>
 #include <Guid/SmmVariableCommon.h>
+#include <Guid/ZeroGuid.h>
 #include "Variable.h"
 
 extern VARIABLE_INFO_ENTRY                           *gVariableInfo;
 EFI_HANDLE                                           mSmmVariableHandle      = 
NULL;
 EFI_HANDLE                                           mVariableHandle         = 
NULL;
 BOOLEAN                                              mAtRuntime              = 
FALSE;
-EFI_GUID                                             mZeroGuid               = 
{0, 0, 0, {0, 0, 0, 0, 0, 0, 0, 0}};
 UINT8                                                *mVariableBufferPayload = 
NULL;
 UINTN                                                
mVariableBufferPayloadSize;
 extern BOOLEAN                                       mEndOfDxe;
@@ -374,7 +374,7 @@ SmmVariableGetStatistics (
 
   CopyGuid (&VendorGuid, &InfoEntry->VendorGuid);
 
-  if (CompareGuid (&VendorGuid, &mZeroGuid)) {
+  if (CompareGuid (&VendorGuid, &gZeroGuid)) {
     //
     // Return the first variable info
     //
diff --git a/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.inf 
b/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.inf
index 00181db..ccee08c 100644
--- a/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.inf
+++ b/SecurityPkg/VariableAuthenticated/RuntimeDxe/VariableSmm.inf
@@ -140,6 +140,7 @@
   gEfiHardwareErrorVariableGuid            ## SOMETIMES_CONSUMES     ## 
Variable:L"HwErrRec####"
   gEdkiiFaultTolerantWriteGuid             ## SOMETIMES_CONSUMES     ## HOB
   gEdkiiVarErrorFlagGuid                   ## CONSUMES               ## GUID
+  gZeroGuid                                ## SOMETIMES_CONSUMES     ## GUID
 
 [Pcd]
   gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize       ## 
CONSUMES
-- 
1.9.5.msysgit.0


------------------------------------------------------------------------------
Monitor 25 network devices or servers for free with OpManager!
OpManager is web-based network management software that monitors 
network devices and physical & virtual servers, alerts via email & sms 
for fault. Monitor 25 devices for free with no restriction. Download now
http://ad.doubleclick.net/ddm/clk/292181274;119417398;o
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to