]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Variable/RuntimeDxe/VariableParsing.c
MdeModulePkg: Apply uncrustify changes
[mirror_edk2.git] / MdeModulePkg / Universal / Variable / RuntimeDxe / VariableParsing.c
index f6d187543d08c4303d5809f6e62527660bb59525..39060ed405b8dc970034419abfacfb7c35f059d1 100644 (file)
@@ -22,8 +22,8 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 **/\r
 BOOLEAN\r
 IsValidVariableHeader (\r
-  IN  VARIABLE_HEADER       *Variable,\r
-  IN  VARIABLE_HEADER       *VariableStoreEnd\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  VARIABLE_HEADER  *VariableStoreEnd\r
   )\r
 {\r
   if ((Variable == NULL) || (Variable >= VariableStoreEnd) || (Variable->StartId != VARIABLE_DATA)) {\r
@@ -50,25 +50,25 @@ IsValidVariableHeader (
 **/\r
 VARIABLE_STORE_STATUS\r
 GetVariableStoreStatus (\r
-  IN VARIABLE_STORE_HEADER *VarStoreHeader\r
+  IN VARIABLE_STORE_HEADER  *VarStoreHeader\r
   )\r
 {\r
   if ((CompareGuid (&VarStoreHeader->Signature, &gEfiAuthenticatedVariableGuid) ||\r
        CompareGuid (&VarStoreHeader->Signature, &gEfiVariableGuid)) &&\r
-      VarStoreHeader->Format == VARIABLE_STORE_FORMATTED &&\r
-      VarStoreHeader->State == VARIABLE_STORE_HEALTHY\r
-      ) {\r
-\r
+      (VarStoreHeader->Format == VARIABLE_STORE_FORMATTED) &&\r
+      (VarStoreHeader->State == VARIABLE_STORE_HEALTHY)\r
+      )\r
+  {\r
     return EfiValid;\r
-  } else if (((UINT32 *)(&VarStoreHeader->Signature))[0] == 0xffffffff &&\r
-             ((UINT32 *)(&VarStoreHeader->Signature))[1] == 0xffffffff &&\r
-             ((UINT32 *)(&VarStoreHeader->Signature))[2] == 0xffffffff &&\r
-             ((UINT32 *)(&VarStoreHeader->Signature))[3] == 0xffffffff &&\r
-             VarStoreHeader->Size == 0xffffffff &&\r
-             VarStoreHeader->Format == 0xff &&\r
-             VarStoreHeader->State == 0xff\r
-          ) {\r
-\r
+  } else if ((((UINT32 *)(&VarStoreHeader->Signature))[0] == 0xffffffff) &&\r
+             (((UINT32 *)(&VarStoreHeader->Signature))[1] == 0xffffffff) &&\r
+             (((UINT32 *)(&VarStoreHeader->Signature))[2] == 0xffffffff) &&\r
+             (((UINT32 *)(&VarStoreHeader->Signature))[3] == 0xffffffff) &&\r
+             (VarStoreHeader->Size == 0xffffffff) &&\r
+             (VarStoreHeader->Format == 0xff) &&\r
+             (VarStoreHeader->State == 0xff)\r
+             )\r
+  {\r
     return EfiRaw;\r
   } else {\r
     return EfiInvalid;\r
@@ -86,10 +86,10 @@ GetVariableStoreStatus (
 **/\r
 UINTN\r
 GetVariableHeaderSize (\r
-  IN  BOOLEAN   AuthFormat\r
+  IN  BOOLEAN  AuthFormat\r
   )\r
 {\r
-  UINTN Value;\r
+  UINTN  Value;\r
 \r
   if (AuthFormat) {\r
     Value = sizeof (AUTHENTICATED_VARIABLE_HEADER);\r
@@ -113,29 +113,33 @@ GetVariableHeaderSize (
 **/\r
 UINTN\r
 NameSizeOfVariable (\r
-  IN  VARIABLE_HEADER   *Variable,\r
-  IN  BOOLEAN           AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  AUTHENTICATED_VARIABLE_HEADER *AuthVariable;\r
+  AUTHENTICATED_VARIABLE_HEADER  *AuthVariable;\r
 \r
-  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *) Variable;\r
+  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *)Variable;\r
   if (AuthFormat) {\r
-    if (AuthVariable->State == (UINT8) (-1) ||\r
-       AuthVariable->DataSize == (UINT32) (-1) ||\r
-       AuthVariable->NameSize == (UINT32) (-1) ||\r
-       AuthVariable->Attributes == (UINT32) (-1)) {\r
+    if ((AuthVariable->State == (UINT8)(-1)) ||\r
+        (AuthVariable->DataSize == (UINT32)(-1)) ||\r
+        (AuthVariable->NameSize == (UINT32)(-1)) ||\r
+        (AuthVariable->Attributes == (UINT32)(-1)))\r
+    {\r
       return 0;\r
     }\r
-    return (UINTN) AuthVariable->NameSize;\r
+\r
+    return (UINTN)AuthVariable->NameSize;\r
   } else {\r
-    if (Variable->State == (UINT8) (-1) ||\r
-        Variable->DataSize == (UINT32) (-1) ||\r
-        Variable->NameSize == (UINT32) (-1) ||\r
-        Variable->Attributes == (UINT32) (-1)) {\r
+    if ((Variable->State == (UINT8)(-1)) ||\r
+        (Variable->DataSize == (UINT32)(-1)) ||\r
+        (Variable->NameSize == (UINT32)(-1)) ||\r
+        (Variable->Attributes == (UINT32)(-1)))\r
+    {\r
       return 0;\r
     }\r
-    return (UINTN) Variable->NameSize;\r
+\r
+    return (UINTN)Variable->NameSize;\r
   }\r
 }\r
 \r
@@ -150,18 +154,18 @@ NameSizeOfVariable (
 **/\r
 VOID\r
 SetNameSizeOfVariable (\r
-  IN VARIABLE_HEADER    *Variable,\r
-  IN UINTN              NameSize,\r
-  IN BOOLEAN            AuthFormat\r
+  IN VARIABLE_HEADER  *Variable,\r
+  IN UINTN            NameSize,\r
+  IN BOOLEAN          AuthFormat\r
   )\r
 {\r
-  AUTHENTICATED_VARIABLE_HEADER *AuthVariable;\r
+  AUTHENTICATED_VARIABLE_HEADER  *AuthVariable;\r
 \r
-  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *) Variable;\r
+  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *)Variable;\r
   if (AuthFormat) {\r
-    AuthVariable->NameSize = (UINT32) NameSize;\r
+    AuthVariable->NameSize = (UINT32)NameSize;\r
   } else {\r
-    Variable->NameSize = (UINT32) NameSize;\r
+    Variable->NameSize = (UINT32)NameSize;\r
   }\r
 }\r
 \r
@@ -178,29 +182,33 @@ SetNameSizeOfVariable (
 **/\r
 UINTN\r
 DataSizeOfVariable (\r
-  IN  VARIABLE_HEADER   *Variable,\r
-  IN  BOOLEAN           AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  AUTHENTICATED_VARIABLE_HEADER *AuthVariable;\r
+  AUTHENTICATED_VARIABLE_HEADER  *AuthVariable;\r
 \r
-  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *) Variable;\r
+  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *)Variable;\r
   if (AuthFormat) {\r
-    if (AuthVariable->State == (UINT8) (-1) ||\r
-       AuthVariable->DataSize == (UINT32) (-1) ||\r
-       AuthVariable->NameSize == (UINT32) (-1) ||\r
-       AuthVariable->Attributes == (UINT32) (-1)) {\r
+    if ((AuthVariable->State == (UINT8)(-1)) ||\r
+        (AuthVariable->DataSize == (UINT32)(-1)) ||\r
+        (AuthVariable->NameSize == (UINT32)(-1)) ||\r
+        (AuthVariable->Attributes == (UINT32)(-1)))\r
+    {\r
       return 0;\r
     }\r
-    return (UINTN) AuthVariable->DataSize;\r
+\r
+    return (UINTN)AuthVariable->DataSize;\r
   } else {\r
-    if (Variable->State == (UINT8) (-1) ||\r
-        Variable->DataSize == (UINT32) (-1) ||\r
-        Variable->NameSize == (UINT32) (-1) ||\r
-        Variable->Attributes == (UINT32) (-1)) {\r
+    if ((Variable->State == (UINT8)(-1)) ||\r
+        (Variable->DataSize == (UINT32)(-1)) ||\r
+        (Variable->NameSize == (UINT32)(-1)) ||\r
+        (Variable->Attributes == (UINT32)(-1)))\r
+    {\r
       return 0;\r
     }\r
-    return (UINTN) Variable->DataSize;\r
+\r
+    return (UINTN)Variable->DataSize;\r
   }\r
 }\r
 \r
@@ -215,18 +223,18 @@ DataSizeOfVariable (
 **/\r
 VOID\r
 SetDataSizeOfVariable (\r
-  IN  VARIABLE_HEADER   *Variable,\r
-  IN  UINTN             DataSize,\r
-  IN  BOOLEAN           AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  UINTN            DataSize,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  AUTHENTICATED_VARIABLE_HEADER *AuthVariable;\r
+  AUTHENTICATED_VARIABLE_HEADER  *AuthVariable;\r
 \r
-  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *) Variable;\r
+  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *)Variable;\r
   if (AuthFormat) {\r
-    AuthVariable->DataSize = (UINT32) DataSize;\r
+    AuthVariable->DataSize = (UINT32)DataSize;\r
   } else {\r
-    Variable->DataSize = (UINT32) DataSize;\r
+    Variable->DataSize = (UINT32)DataSize;\r
   }\r
 }\r
 \r
@@ -243,11 +251,11 @@ SetDataSizeOfVariable (
 **/\r
 CHAR16 *\r
 GetVariableNamePtr (\r
-  IN  VARIABLE_HEADER   *Variable,\r
-  IN  BOOLEAN           AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  return (CHAR16 *) ((UINTN) Variable + GetVariableHeaderSize (AuthFormat));\r
+  return (CHAR16 *)((UINTN)Variable + GetVariableHeaderSize (AuthFormat));\r
 }\r
 \r
 /**\r
@@ -262,13 +270,13 @@ GetVariableNamePtr (
 **/\r
 EFI_GUID *\r
 GetVendorGuidPtr (\r
-  IN  VARIABLE_HEADER    *Variable,\r
-  IN  BOOLEAN            AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  AUTHENTICATED_VARIABLE_HEADER *AuthVariable;\r
+  AUTHENTICATED_VARIABLE_HEADER  *AuthVariable;\r
 \r
-  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *) Variable;\r
+  AuthVariable = (AUTHENTICATED_VARIABLE_HEADER *)Variable;\r
   if (AuthFormat) {\r
     return &AuthVariable->VendorGuid;\r
   } else {\r
@@ -289,20 +297,20 @@ GetVendorGuidPtr (
 **/\r
 UINT8 *\r
 GetVariableDataPtr (\r
-  IN  VARIABLE_HEADER    *Variable,\r
-  IN  BOOLEAN            AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  UINTN Value;\r
+  UINTN  Value;\r
 \r
   //\r
   // Be careful about pad size for alignment.\r
   //\r
-  Value =  (UINTN) GetVariableNamePtr (Variable, AuthFormat);\r
+  Value  =  (UINTN)GetVariableNamePtr (Variable, AuthFormat);\r
   Value += NameSizeOfVariable (Variable, AuthFormat);\r
   Value += GET_PAD_SIZE (NameSizeOfVariable (Variable, AuthFormat));\r
 \r
-  return (UINT8 *) Value;\r
+  return (UINT8 *)Value;\r
 }\r
 \r
 /**\r
@@ -317,16 +325,16 @@ GetVariableDataPtr (
 **/\r
 UINTN\r
 GetVariableDataOffset (\r
-  IN  VARIABLE_HEADER   *Variable,\r
-  IN  BOOLEAN           AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  UINTN Value;\r
+  UINTN  Value;\r
 \r
   //\r
   // Be careful about pad size for alignment\r
   //\r
-  Value = GetVariableHeaderSize (AuthFormat);\r
+  Value  = GetVariableHeaderSize (AuthFormat);\r
   Value += NameSizeOfVariable (Variable, AuthFormat);\r
   Value += GET_PAD_SIZE (NameSizeOfVariable (Variable, AuthFormat));\r
 \r
@@ -346,20 +354,20 @@ GetVariableDataOffset (
 **/\r
 VARIABLE_HEADER *\r
 GetNextVariablePtr (\r
-  IN  VARIABLE_HEADER   *Variable,\r
-  IN  BOOLEAN           AuthFormat\r
+  IN  VARIABLE_HEADER  *Variable,\r
+  IN  BOOLEAN          AuthFormat\r
   )\r
 {\r
-  UINTN Value;\r
+  UINTN  Value;\r
 \r
-  Value =  (UINTN) GetVariableDataPtr (Variable, AuthFormat);\r
+  Value  =  (UINTN)GetVariableDataPtr (Variable, AuthFormat);\r
   Value += DataSizeOfVariable (Variable, AuthFormat);\r
   Value += GET_PAD_SIZE (DataSizeOfVariable (Variable, AuthFormat));\r
 \r
   //\r
   // Be careful about pad size for alignment.\r
   //\r
-  return (VARIABLE_HEADER *) HEADER_ALIGN (Value);\r
+  return (VARIABLE_HEADER *)HEADER_ALIGN (Value);\r
 }\r
 \r
 /**\r
@@ -373,13 +381,13 @@ GetNextVariablePtr (
 **/\r
 VARIABLE_HEADER *\r
 GetStartPointer (\r
-  IN VARIABLE_STORE_HEADER       *VarStoreHeader\r
+  IN VARIABLE_STORE_HEADER  *VarStoreHeader\r
   )\r
 {\r
   //\r
   // The start of variable store.\r
   //\r
-  return (VARIABLE_HEADER *) HEADER_ALIGN (VarStoreHeader + 1);\r
+  return (VARIABLE_HEADER *)HEADER_ALIGN (VarStoreHeader + 1);\r
 }\r
 \r
 /**\r
@@ -396,13 +404,13 @@ GetStartPointer (
 **/\r
 VARIABLE_HEADER *\r
 GetEndPointer (\r
-  IN VARIABLE_STORE_HEADER       *VarStoreHeader\r
+  IN VARIABLE_STORE_HEADER  *VarStoreHeader\r
   )\r
 {\r
   //\r
   // The end of variable store\r
   //\r
-  return (VARIABLE_HEADER *) HEADER_ALIGN ((UINTN) VarStoreHeader + VarStoreHeader->Size);\r
+  return (VARIABLE_HEADER *)HEADER_ALIGN ((UINTN)VarStoreHeader + VarStoreHeader->Size);\r
 }\r
 \r
 /**\r
@@ -418,23 +426,23 @@ GetEndPointer (
 **/\r
 BOOLEAN\r
 VariableCompareTimeStampInternal (\r
-  IN EFI_TIME               *FirstTime,\r
-  IN EFI_TIME               *SecondTime\r
+  IN EFI_TIME  *FirstTime,\r
+  IN EFI_TIME  *SecondTime\r
   )\r
 {\r
   if (FirstTime->Year != SecondTime->Year) {\r
-    return (BOOLEAN) (FirstTime->Year < SecondTime->Year);\r
+    return (BOOLEAN)(FirstTime->Year < SecondTime->Year);\r
   } else if (FirstTime->Month != SecondTime->Month) {\r
-    return (BOOLEAN) (FirstTime->Month < SecondTime->Month);\r
+    return (BOOLEAN)(FirstTime->Month < SecondTime->Month);\r
   } else if (FirstTime->Day != SecondTime->Day) {\r
-    return (BOOLEAN) (FirstTime->Day < SecondTime->Day);\r
+    return (BOOLEAN)(FirstTime->Day < SecondTime->Day);\r
   } else if (FirstTime->Hour != SecondTime->Hour) {\r
-    return (BOOLEAN) (FirstTime->Hour < SecondTime->Hour);\r
+    return (BOOLEAN)(FirstTime->Hour < SecondTime->Hour);\r
   } else if (FirstTime->Minute != SecondTime->Minute) {\r
-    return (BOOLEAN) (FirstTime->Minute < SecondTime->Minute);\r
+    return (BOOLEAN)(FirstTime->Minute < SecondTime->Minute);\r
   }\r
 \r
-  return (BOOLEAN) (FirstTime->Second <= SecondTime->Second);\r
+  return (BOOLEAN)(FirstTime->Second <= SecondTime->Second);\r
 }\r
 \r
 /**\r
@@ -460,39 +468,41 @@ FindVariableEx (
   IN     BOOLEAN                 AuthFormat\r
   )\r
 {\r
-  VARIABLE_HEADER                *InDeletedVariable;\r
-  VOID                           *Point;\r
+  VARIABLE_HEADER  *InDeletedVariable;\r
+  VOID             *Point;\r
 \r
   PtrTrack->InDeletedTransitionPtr = NULL;\r
 \r
   //\r
   // Find the variable by walk through HOB, volatile and non-volatile variable store.\r
   //\r
-  InDeletedVariable  = NULL;\r
+  InDeletedVariable = NULL;\r
 \r
   for ( PtrTrack->CurrPtr = PtrTrack->StartPtr\r
-      ; IsValidVariableHeader (PtrTrack->CurrPtr, PtrTrack->EndPtr)\r
-      ; PtrTrack->CurrPtr = GetNextVariablePtr (PtrTrack->CurrPtr, AuthFormat)\r
-      ) {\r
-    if (PtrTrack->CurrPtr->State == VAR_ADDED ||\r
-        PtrTrack->CurrPtr->State == (VAR_IN_DELETED_TRANSITION & VAR_ADDED)\r
-       ) {\r
+        ; IsValidVariableHeader (PtrTrack->CurrPtr, PtrTrack->EndPtr)\r
+        ; PtrTrack->CurrPtr = GetNextVariablePtr (PtrTrack->CurrPtr, AuthFormat)\r
+        )\r
+  {\r
+    if ((PtrTrack->CurrPtr->State == VAR_ADDED) ||\r
+        (PtrTrack->CurrPtr->State == (VAR_IN_DELETED_TRANSITION & VAR_ADDED))\r
+        )\r
+    {\r
       if (IgnoreRtCheck || !AtRuntime () || ((PtrTrack->CurrPtr->Attributes & EFI_VARIABLE_RUNTIME_ACCESS) != 0)) {\r
         if (VariableName[0] == 0) {\r
           if (PtrTrack->CurrPtr->State == (VAR_IN_DELETED_TRANSITION & VAR_ADDED)) {\r
-            InDeletedVariable   = PtrTrack->CurrPtr;\r
+            InDeletedVariable = PtrTrack->CurrPtr;\r
           } else {\r
             PtrTrack->InDeletedTransitionPtr = InDeletedVariable;\r
             return EFI_SUCCESS;\r
           }\r
         } else {\r
           if (CompareGuid (VendorGuid, GetVendorGuidPtr (PtrTrack->CurrPtr, AuthFormat))) {\r
-            Point = (VOID *) GetVariableNamePtr (PtrTrack->CurrPtr, AuthFormat);\r
+            Point = (VOID *)GetVariableNamePtr (PtrTrack->CurrPtr, AuthFormat);\r
 \r
             ASSERT (NameSizeOfVariable (PtrTrack->CurrPtr, AuthFormat) != 0);\r
             if (CompareMem (VariableName, Point, NameSizeOfVariable (PtrTrack->CurrPtr, AuthFormat)) == 0) {\r
               if (PtrTrack->CurrPtr->State == (VAR_IN_DELETED_TRANSITION & VAR_ADDED)) {\r
-                InDeletedVariable     = PtrTrack->CurrPtr;\r
+                InDeletedVariable = PtrTrack->CurrPtr;\r
               } else {\r
                 PtrTrack->InDeletedTransitionPtr = InDeletedVariable;\r
                 return EFI_SUCCESS;\r
@@ -532,11 +542,11 @@ FindVariableEx (
 EFI_STATUS\r
 EFIAPI\r
 VariableServiceGetNextVariableInternal (\r
-  IN  CHAR16                *VariableName,\r
-  IN  EFI_GUID              *VendorGuid,\r
-  IN  VARIABLE_STORE_HEADER **VariableStoreList,\r
-  OUT VARIABLE_HEADER       **VariablePtr,\r
-  IN  BOOLEAN               AuthFormat\r
+  IN  CHAR16                 *VariableName,\r
+  IN  EFI_GUID               *VendorGuid,\r
+  IN  VARIABLE_STORE_HEADER  **VariableStoreList,\r
+  OUT VARIABLE_HEADER        **VariablePtr,\r
+  IN  BOOLEAN                AuthFormat\r
   )\r
 {\r
   EFI_STATUS              Status;\r
@@ -554,14 +564,14 @@ VariableServiceGetNextVariableInternal (
   ZeroMem (&Variable, sizeof (Variable));\r
 \r
   // Check if the variable exists in the given variable store list\r
-  for (StoreType = (VARIABLE_STORE_TYPE) 0; StoreType < VariableStoreTypeMax; StoreType++) {\r
+  for (StoreType = (VARIABLE_STORE_TYPE)0; StoreType < VariableStoreTypeMax; StoreType++) {\r
     if (VariableStoreList[StoreType] == NULL) {\r
       continue;\r
     }\r
 \r
     Variable.StartPtr = GetStartPointer (VariableStoreList[StoreType]);\r
-    Variable.EndPtr   = GetEndPointer   (VariableStoreList[StoreType]);\r
-    Variable.Volatile = (BOOLEAN) (StoreType == VariableStoreTypeVolatile);\r
+    Variable.EndPtr   = GetEndPointer (VariableStoreList[StoreType]);\r
+    Variable.Volatile = (BOOLEAN)(StoreType == VariableStoreTypeVolatile);\r
 \r
     Status = FindVariableEx (VariableName, VendorGuid, FALSE, &Variable, AuthFormat);\r
     if (!EFI_ERROR (Status)) {\r
@@ -569,7 +579,7 @@ VariableServiceGetNextVariableInternal (
     }\r
   }\r
 \r
-  if (Variable.CurrPtr == NULL || EFI_ERROR (Status)) {\r
+  if ((Variable.CurrPtr == NULL) || EFI_ERROR (Status)) {\r
     //\r
     // For VariableName is an empty string, FindVariableEx() will try to find and return\r
     // the first qualified variable, and if FindVariableEx() returns error (EFI_NOT_FOUND)\r
@@ -583,6 +593,7 @@ VariableServiceGetNextVariableInternal (
       //\r
       Status = EFI_INVALID_PARAMETER;\r
     }\r
+\r
     goto Done;\r
   }\r
 \r
@@ -601,11 +612,12 @@ VariableServiceGetNextVariableInternal (
       //\r
       // Find current storage index\r
       //\r
-      for (StoreType = (VARIABLE_STORE_TYPE) 0; StoreType < VariableStoreTypeMax; StoreType++) {\r
+      for (StoreType = (VARIABLE_STORE_TYPE)0; StoreType < VariableStoreTypeMax; StoreType++) {\r
         if ((VariableStoreList[StoreType] != NULL) && (Variable.StartPtr == GetStartPointer (VariableStoreList[StoreType]))) {\r
           break;\r
         }\r
       }\r
+\r
       ASSERT (StoreType < VariableStoreTypeMax);\r
       //\r
       // Switch to next storage\r
@@ -615,6 +627,7 @@ VariableServiceGetNextVariableInternal (
           break;\r
         }\r
       }\r
+\r
       //\r
       // Capture the case that\r
       // 1. current storage is the last one, or\r
@@ -624,15 +637,16 @@ VariableServiceGetNextVariableInternal (
         Status = EFI_NOT_FOUND;\r
         goto Done;\r
       }\r
+\r
       Variable.StartPtr = GetStartPointer (VariableStoreList[StoreType]);\r
-      Variable.EndPtr   = GetEndPointer   (VariableStoreList[StoreType]);\r
+      Variable.EndPtr   = GetEndPointer (VariableStoreList[StoreType]);\r
       Variable.CurrPtr  = Variable.StartPtr;\r
     }\r
 \r
     //\r
     // Variable is found\r
     //\r
-    if (Variable.CurrPtr->State == VAR_ADDED || Variable.CurrPtr->State == (VAR_IN_DELETED_TRANSITION & VAR_ADDED)) {\r
+    if ((Variable.CurrPtr->State == VAR_ADDED) || (Variable.CurrPtr->State == (VAR_IN_DELETED_TRANSITION & VAR_ADDED))) {\r
       if (!AtRuntime () || ((Variable.CurrPtr->Attributes & EFI_VARIABLE_RUNTIME_ACCESS) != 0)) {\r
         if (Variable.CurrPtr->State == (VAR_IN_DELETED_TRANSITION & VAR_ADDED)) {\r
           //\r
@@ -641,15 +655,15 @@ VariableServiceGetNextVariableInternal (
           // don't return it.\r
           //\r
           VariablePtrTrack.StartPtr = Variable.StartPtr;\r
-          VariablePtrTrack.EndPtr = Variable.EndPtr;\r
-          Status = FindVariableEx (\r
-                     GetVariableNamePtr (Variable.CurrPtr, AuthFormat),\r
-                     GetVendorGuidPtr (Variable.CurrPtr, AuthFormat),\r
-                     FALSE,\r
-                     &VariablePtrTrack,\r
-                     AuthFormat\r
-                     );\r
-          if (!EFI_ERROR (Status) && VariablePtrTrack.CurrPtr->State == VAR_ADDED) {\r
+          VariablePtrTrack.EndPtr   = Variable.EndPtr;\r
+          Status                    = FindVariableEx (\r
+                                        GetVariableNamePtr (Variable.CurrPtr, AuthFormat),\r
+                                        GetVendorGuidPtr (Variable.CurrPtr, AuthFormat),\r
+                                        FALSE,\r
+                                        &VariablePtrTrack,\r
+                                        AuthFormat\r
+                                        );\r
+          if (!EFI_ERROR (Status) && (VariablePtrTrack.CurrPtr->State == VAR_ADDED)) {\r
             Variable.CurrPtr = GetNextVariablePtr (Variable.CurrPtr, AuthFormat);\r
             continue;\r
           }\r
@@ -660,16 +674,17 @@ VariableServiceGetNextVariableInternal (
         //\r
         if ((VariableStoreList[VariableStoreTypeHob] != NULL) && (VariableStoreList[VariableStoreTypeNv] != NULL) &&\r
             (Variable.StartPtr == GetStartPointer (VariableStoreList[VariableStoreTypeNv]))\r
-           ) {\r
+            )\r
+        {\r
           VariableInHob.StartPtr = GetStartPointer (VariableStoreList[VariableStoreTypeHob]);\r
-          VariableInHob.EndPtr   = GetEndPointer   (VariableStoreList[VariableStoreTypeHob]);\r
-          Status = FindVariableEx (\r
-                     GetVariableNamePtr (Variable.CurrPtr, AuthFormat),\r
-                     GetVendorGuidPtr (Variable.CurrPtr, AuthFormat),\r
-                     FALSE,\r
-                     &VariableInHob,\r
-                     AuthFormat\r
-                     );\r
+          VariableInHob.EndPtr   = GetEndPointer (VariableStoreList[VariableStoreTypeHob]);\r
+          Status                 = FindVariableEx (\r
+                                     GetVariableNamePtr (Variable.CurrPtr, AuthFormat),\r
+                                     GetVendorGuidPtr (Variable.CurrPtr, AuthFormat),\r
+                                     FALSE,\r
+                                     &VariableInHob,\r
+                                     AuthFormat\r
+                                     );\r
           if (!EFI_ERROR (Status)) {\r
             Variable.CurrPtr = GetNextVariablePtr (Variable.CurrPtr, AuthFormat);\r
             continue;\r
@@ -677,7 +692,7 @@ VariableServiceGetNextVariableInternal (
         }\r
 \r
         *VariablePtr = Variable.CurrPtr;\r
-        Status = EFI_SUCCESS;\r
+        Status       = EFI_SUCCESS;\r
         goto Done;\r
       }\r
     }\r
@@ -722,12 +737,13 @@ UpdateVariableInfo (
   IN OUT VARIABLE_INFO_ENTRY  **VariableInfo\r
   )\r
 {\r
-  VARIABLE_INFO_ENTRY   *Entry;\r
+  VARIABLE_INFO_ENTRY  *Entry;\r
 \r
   if (FeaturePcdGet (PcdVariableCollectStatistics)) {\r
-    if (VariableName == NULL || VendorGuid == NULL || VariableInfo == NULL) {\r
+    if ((VariableName == NULL) || (VendorGuid == NULL) || (VariableInfo == NULL)) {\r
       return;\r
     }\r
+\r
     if (AtRuntime ()) {\r
       // Don't collect statistics at runtime.\r
       return;\r
@@ -744,23 +760,25 @@ UpdateVariableInfo (
       CopyGuid (&(*VariableInfo)->VendorGuid, VendorGuid);\r
       (*VariableInfo)->Name = AllocateZeroPool (StrSize (VariableName));\r
       ASSERT ((*VariableInfo)->Name != NULL);\r
-      StrCpyS ((*VariableInfo)->Name, StrSize(VariableName)/sizeof(CHAR16), VariableName);\r
+      StrCpyS ((*VariableInfo)->Name, StrSize (VariableName)/sizeof (CHAR16), VariableName);\r
       (*VariableInfo)->Volatile = Volatile;\r
     }\r
 \r
-\r
     for (Entry = (*VariableInfo); Entry != NULL; Entry = Entry->Next) {\r
       if (CompareGuid (VendorGuid, &Entry->VendorGuid)) {\r
         if (StrCmp (VariableName, Entry->Name) == 0) {\r
           if (Read) {\r
             Entry->ReadCount++;\r
           }\r
+\r
           if (Write) {\r
             Entry->WriteCount++;\r
           }\r
+\r
           if (Delete) {\r
             Entry->DeleteCount++;\r
           }\r
+\r
           if (Cache) {\r
             Entry->CacheCount++;\r
           }\r
@@ -780,7 +798,7 @@ UpdateVariableInfo (
         CopyGuid (&Entry->Next->VendorGuid, VendorGuid);\r
         Entry->Next->Name = AllocateZeroPool (StrSize (VariableName));\r
         ASSERT (Entry->Next->Name != NULL);\r
-        StrCpyS (Entry->Next->Name, StrSize(VariableName)/sizeof(CHAR16), VariableName);\r
+        StrCpyS (Entry->Next->Name, StrSize (VariableName)/sizeof (CHAR16), VariableName);\r
         Entry->Next->Volatile = Volatile;\r
       }\r
     }\r