SecurityPkg: AuthVariableLib: Cache UserPhysicalPresent in AuthVariableLib

[mirror_edk2.git] / SecurityPkg / Library / AuthVariableLib / AuthVariableLib.c

/** @file\r
  Implement authentication services for the authenticated variables.\r
\r
  Caution: This module requires additional review when modified.\r
  This driver will have external input - variable data. It may be input in SMM mode.\r
  This external input must be validated carefully to avoid security issue like\r
  buffer overflow, integer overflow.\r
  Variable attribute should also be checked to avoid authentication bypass.\r
     The whole SMM authentication variable design relies on the integrity of flash part and SMM.\r
  which is assumed to be protected by platform.  All variable code and metadata in flash/SMM Memory\r
  may not be modified without authorization. If platform fails to protect these resources,\r
  the authentication service provided in this driver will be broken, and the behavior is undefined.\r
\r
Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.<BR>\r
This program and the accompanying materials\r
are licensed and made available under the terms and conditions of the BSD License\r
which accompanies this distribution.  The full text of the license may be found at\r
http://opensource.org/licenses/bsd-license.php\r
\r
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
\r
**/\r
\r
#include "AuthServiceInternal.h"\r
\r
///\r
/// Global database array for scratch\r
///\r
UINT8    *mPubKeyStore;\r
UINT32   mPubKeyNumber;\r
UINT32   mMaxKeyNumber;\r
UINT32   mMaxKeyDbSize;\r
UINT8    *mCertDbStore;\r
UINT32   mMaxCertDbSize;\r
UINT32   mPlatformMode;\r
UINT8    mVendorKeyState;\r
BOOLEAN  mUserPhysicalPresent;\r
\r
EFI_GUID mSignatureSupport[] = {EFI_CERT_SHA1_GUID, EFI_CERT_SHA256_GUID, EFI_CERT_RSA2048_GUID, EFI_CERT_X509_GUID};\r
\r
//\r
// Hash context pointer\r
//\r
VOID  *mHashCtx = NULL;\r
\r
VARIABLE_ENTRY_PROPERTY mAuthVarEntry[] = {\r
  {\r
    &gEfiSecureBootEnableDisableGuid,\r
    EFI_SECURE_BOOT_ENABLE_NAME,\r
    {\r
      VAR_CHECK_VARIABLE_PROPERTY_REVISION,\r
      0,\r
      VARIABLE_ATTRIBUTE_NV_BS,\r
      sizeof (UINT8),\r
      sizeof (UINT8)\r
    }\r
  },\r
  {\r
    &gEfiCustomModeEnableGuid,\r
    EFI_CUSTOM_MODE_NAME,\r
    {\r
      VAR_CHECK_VARIABLE_PROPERTY_REVISION,\r
      0,\r
      VARIABLE_ATTRIBUTE_NV_BS,\r
      sizeof (UINT8),\r
      sizeof (UINT8)\r
    }\r
  },\r
  {\r
    &gEfiVendorKeysNvGuid,\r
    EFI_VENDOR_KEYS_NV_VARIABLE_NAME,\r
    {\r
      VAR_CHECK_VARIABLE_PROPERTY_REVISION,\r
      VAR_CHECK_VARIABLE_PROPERTY_READ_ONLY,\r
      VARIABLE_ATTRIBUTE_NV_BS_RT_AT,\r
      sizeof (UINT8),\r
      sizeof (UINT8)\r
    }\r
  },\r
  {\r
    &gEfiAuthenticatedVariableGuid,\r
    AUTHVAR_KEYDB_NAME,\r
    {\r
      VAR_CHECK_VARIABLE_PROPERTY_REVISION,\r
      VAR_CHECK_VARIABLE_PROPERTY_READ_ONLY,\r
      VARIABLE_ATTRIBUTE_NV_BS_RT_AW,\r
      sizeof (UINT8),\r
      MAX_UINTN\r
    }\r
  },\r
  {\r
    &gEfiCertDbGuid,\r
    EFI_CERT_DB_NAME,\r
    {\r
      VAR_CHECK_VARIABLE_PROPERTY_REVISION,\r
      VAR_CHECK_VARIABLE_PROPERTY_READ_ONLY,\r
      VARIABLE_ATTRIBUTE_NV_BS_RT_AT,\r
      sizeof (UINT32),\r
      MAX_UINTN\r
    }\r
  },\r
  {\r
    &gEfiCertDbGuid,\r
    EFI_CERT_DB_VOLATILE_NAME,\r
    {\r
      VAR_CHECK_VARIABLE_PROPERTY_REVISION,\r
      VAR_CHECK_VARIABLE_PROPERTY_READ_ONLY,\r
      VARIABLE_ATTRIBUTE_BS_RT_AT,\r
      sizeof (UINT32),\r
      MAX_UINTN\r
    }\r
  },\r
};\r
\r
VOID **mAuthVarAddressPointer[10];\r
\r
AUTH_VAR_LIB_CONTEXT_IN *mAuthVarLibContextIn = NULL;\r
\r
/**\r
  Initialization for authenticated varibale services.\r
  If this initialization returns error status, other APIs will not work\r
  and expect to be not called then.\r
\r
  @param[in]  AuthVarLibContextIn   Pointer to input auth variable lib context.\r
  @param[out] AuthVarLibContextOut  Pointer to output auth variable lib context.\r
\r
  @retval EFI_SUCCESS               Function successfully executed.\r
  @retval EFI_INVALID_PARAMETER     If AuthVarLibContextIn == NULL or AuthVarLibContextOut == NULL.\r
  @retval EFI_OUT_OF_RESOURCES      Fail to allocate enough resource.\r
  @retval EFI_UNSUPPORTED           Unsupported to process authenticated variable.\r
\r
**/\r
EFI_STATUS\r
EFIAPI\r
AuthVariableLibInitialize (\r
  IN  AUTH_VAR_LIB_CONTEXT_IN   *AuthVarLibContextIn,\r
  OUT AUTH_VAR_LIB_CONTEXT_OUT  *AuthVarLibContextOut\r
  )\r
{\r
  EFI_STATUS            Status;\r
  UINT8                 VarValue;\r
  UINT32                VarAttr;\r
  UINT8                 *Data;\r
  UINTN                 DataSize;\r
  UINTN                 CtxSize;\r
  UINT8                 SecureBootMode;\r
  UINT8                 SecureBootEnable;\r
  UINT8                 CustomMode;\r
  UINT32                ListSize;\r
\r
  if ((AuthVarLibContextIn == NULL) || (AuthVarLibContextOut == NULL)) {\r
    return EFI_INVALID_PARAMETER;\r
  }\r
\r
  mAuthVarLibContextIn = AuthVarLibContextIn;\r
\r
  //\r
  // Initialize hash context.\r
  //\r
  CtxSize   = Sha256GetContextSize ();\r
  mHashCtx  = AllocateRuntimePool (CtxSize);\r
  if (mHashCtx == NULL) {\r
    return EFI_OUT_OF_RESOURCES;\r
  }\r
\r
  //\r
  // Reserve runtime buffer for public key database. The size excludes variable header and name size.\r
  //\r
  mMaxKeyDbSize = (UINT32) (mAuthVarLibContextIn->MaxAuthVariableSize - sizeof (AUTHVAR_KEYDB_NAME));\r
  mMaxKeyNumber = mMaxKeyDbSize / sizeof (AUTHVAR_KEY_DB_DATA);\r
  mPubKeyStore  = AllocateRuntimePool (mMaxKeyDbSize);\r
  if (mPubKeyStore == NULL) {\r
    return EFI_OUT_OF_RESOURCES;\r
  }\r
\r
  //\r
  // Reserve runtime buffer for certificate database. The size excludes variable header and name size.\r
  // Use EFI_CERT_DB_VOLATILE_NAME size since it is longer.\r
  //\r
  mMaxCertDbSize = (UINT32) (mAuthVarLibContextIn->MaxAuthVariableSize - sizeof (EFI_CERT_DB_VOLATILE_NAME));\r
  mCertDbStore   = AllocateRuntimePool (mMaxCertDbSize);\r
  if (mCertDbStore == NULL) {\r
    return EFI_OUT_OF_RESOURCES;\r
  }\r
\r
  //\r
  // Check "AuthVarKeyDatabase" variable's existence.\r
  // If it doesn't exist, create a new one with initial value of 0 and EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS set.\r
  //\r
  Status = AuthServiceInternalFindVariable (\r
             AUTHVAR_KEYDB_NAME,\r
             &gEfiAuthenticatedVariableGuid,\r
             (VOID **) &Data,\r
             &DataSize\r
             );\r
  if (EFI_ERROR (Status)) {\r
    VarAttr       = EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_RUNTIME_ACCESS | EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS;\r
    VarValue      = 0;\r
    mPubKeyNumber = 0;\r
    Status        = AuthServiceInternalUpdateVariable (\r
                      AUTHVAR_KEYDB_NAME,\r
                      &gEfiAuthenticatedVariableGuid,\r
                      &VarValue,\r
                      sizeof(UINT8),\r
                      VarAttr\r
                      );\r
    if (EFI_ERROR (Status)) {\r
      return Status;\r
    }\r
  } else {\r
    //\r
    // Load database in global variable for cache.\r
    //\r
    ASSERT ((DataSize != 0) && (Data != NULL));\r
    //\r
    // "AuthVarKeyDatabase" is an internal variable. Its DataSize is always ensured not to exceed mPubKeyStore buffer size(See definition before)\r
    //  Therefore, there is no memory overflow in underlying CopyMem.\r
    //\r
    CopyMem (mPubKeyStore, (UINT8 *) Data, DataSize);\r
    mPubKeyNumber = (UINT32) (DataSize / sizeof (AUTHVAR_KEY_DB_DATA));\r
  }\r
\r
  Status = AuthServiceInternalFindVariable (EFI_PLATFORM_KEY_NAME, &gEfiGlobalVariableGuid, (VOID **) &Data, &DataSize);\r
  if (EFI_ERROR (Status)) {\r
    DEBUG ((EFI_D_INFO, "Variable %s does not exist.\n", EFI_PLATFORM_KEY_NAME));\r
  } else {\r
    DEBUG ((EFI_D_INFO, "Variable %s exists.\n", EFI_PLATFORM_KEY_NAME));\r
  }\r
\r
  //\r
  // Create "SetupMode" variable with BS+RT attribute set.\r
  //\r
  if (EFI_ERROR (Status)) {\r
    mPlatformMode = SETUP_MODE;\r
  } else {\r
    mPlatformMode = USER_MODE;\r
  }\r
  Status = AuthServiceInternalUpdateVariable (\r
             EFI_SETUP_MODE_NAME,\r
             &gEfiGlobalVariableGuid,\r
             &mPlatformMode,\r
             sizeof(UINT8),\r
             EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS\r
             );\r
  if (EFI_ERROR (Status)) {\r
    return Status;\r
  }\r
\r
  //\r
  // Create "SignatureSupport" variable with BS+RT attribute set.\r
  //\r
  Status  = AuthServiceInternalUpdateVariable (\r
              EFI_SIGNATURE_SUPPORT_NAME,\r
              &gEfiGlobalVariableGuid,\r
              mSignatureSupport,\r
              sizeof(mSignatureSupport),\r
              EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS\r
              );\r
  if (EFI_ERROR (Status)) {\r
    return Status;\r
  }\r
\r
  //\r
  // If "SecureBootEnable" variable exists, then update "SecureBoot" variable.\r
  // If "SecureBootEnable" variable is SECURE_BOOT_ENABLE and in USER_MODE, Set "SecureBoot" variable to SECURE_BOOT_MODE_ENABLE.\r
  // If "SecureBootEnable" variable is SECURE_BOOT_DISABLE, Set "SecureBoot" variable to SECURE_BOOT_MODE_DISABLE.\r
  //\r
  SecureBootEnable = SECURE_BOOT_DISABLE;\r
  Status = AuthServiceInternalFindVariable (EFI_SECURE_BOOT_ENABLE_NAME, &gEfiSecureBootEnableDisableGuid, (VOID **) &Data, &DataSize);\r
  if (!EFI_ERROR (Status)) {\r
    if (mPlatformMode == USER_MODE){\r
      SecureBootEnable = *(UINT8 *) Data;\r
    }\r
  } else if (mPlatformMode == USER_MODE) {\r
    //\r
    // "SecureBootEnable" not exist, initialize it in USER_MODE.\r
    //\r
    SecureBootEnable = SECURE_BOOT_ENABLE;\r
    Status = AuthServiceInternalUpdateVariable (\r
               EFI_SECURE_BOOT_ENABLE_NAME,\r
               &gEfiSecureBootEnableDisableGuid,\r
               &SecureBootEnable,\r
               sizeof (UINT8),\r
               EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_BOOTSERVICE_ACCESS\r
               );\r
    if (EFI_ERROR (Status)) {\r
      return Status;\r
    }\r
  }\r
\r
  //\r
  // Create "SecureBoot" variable with BS+RT attribute set.\r
  //\r
  if (SecureBootEnable == SECURE_BOOT_ENABLE && mPlatformMode == USER_MODE) {\r
    SecureBootMode = SECURE_BOOT_MODE_ENABLE;\r
  } else {\r
    SecureBootMode = SECURE_BOOT_MODE_DISABLE;\r
  }\r
  Status = AuthServiceInternalUpdateVariable (\r
             EFI_SECURE_BOOT_MODE_NAME,\r
             &gEfiGlobalVariableGuid,\r
             &SecureBootMode,\r
             sizeof (UINT8),\r
             EFI_VARIABLE_RUNTIME_ACCESS | EFI_VARIABLE_BOOTSERVICE_ACCESS\r
             );\r
  if (EFI_ERROR (Status)) {\r
    return Status;\r
  }\r
\r
  DEBUG ((EFI_D_INFO, "Variable %s is %x\n", EFI_SETUP_MODE_NAME, mPlatformMode));\r
  DEBUG ((EFI_D_INFO, "Variable %s is %x\n", EFI_SECURE_BOOT_MODE_NAME, SecureBootMode));\r
  DEBUG ((EFI_D_INFO, "Variable %s is %x\n", EFI_SECURE_BOOT_ENABLE_NAME, SecureBootEnable));\r
\r
  //\r
  // Initialize "CustomMode" in STANDARD_SECURE_BOOT_MODE state.\r
  //\r
  CustomMode = STANDARD_SECURE_BOOT_MODE;\r
  Status = AuthServiceInternalUpdateVariable (\r
             EFI_CUSTOM_MODE_NAME,\r
             &gEfiCustomModeEnableGuid,\r
             &CustomMode,\r
             sizeof (UINT8),\r
             EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_BOOTSERVICE_ACCESS\r
             );\r
  if (EFI_ERROR (Status)) {\r
    return Status;\r
  }\r
\r
  DEBUG ((EFI_D_INFO, "Variable %s is %x\n", EFI_CUSTOM_MODE_NAME, CustomMode));\r
\r
  //\r
  // Check "certdb" variable's existence.\r
  // If it doesn't exist, then create a new one with\r
  // EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACCESS set.\r
  //\r
  Status = AuthServiceInternalFindVariable (\r
             EFI_CERT_DB_NAME,\r
             &gEfiCertDbGuid,\r
             (VOID **) &Data,\r
             &DataSize\r
             );\r
  if (EFI_ERROR (Status)) {\r
    VarAttr  = EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_RUNTIME_ACCESS | EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACCESS;\r
    ListSize = sizeof (UINT32);\r
    Status   = AuthServiceInternalUpdateVariable (\r
                 EFI_CERT_DB_NAME,\r
                 &gEfiCertDbGuid,\r
                 &ListSize,\r
                 sizeof (UINT32),\r
                 VarAttr\r
                 );\r
    if (EFI_ERROR (Status)) {\r
      return Status;\r
    }\r
  } else {\r
    //\r
    // Clean up Certs to make certDB & Time based auth variable consistent\r
    //\r
    Status = CleanCertsFromDb();\r
    if (EFI_ERROR (Status)) {\r
      DEBUG ((EFI_D_ERROR, "Clean up CertDB fail! Status %x\n", Status));\r
      return Status;\r
    }\r
  }\r
\r
  //\r
  // Create "certdbv" variable with RT+BS+AT set.\r
  //\r
  VarAttr  = EFI_VARIABLE_RUNTIME_ACCESS | EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACCESS;\r
  ListSize = sizeof (UINT32);\r
  Status   = AuthServiceInternalUpdateVariable (\r
               EFI_CERT_DB_VOLATILE_NAME,\r
               &gEfiCertDbGuid,\r
               &ListSize,\r
               sizeof (UINT32),\r
               VarAttr\r
               );\r
  if (EFI_ERROR (Status)) {\r
    return Status;\r
  }\r
\r
  //\r
  // Check "VendorKeysNv" variable's existence and create "VendorKeys" variable accordingly.\r
  //\r
  Status = AuthServiceInternalFindVariable (EFI_VENDOR_KEYS_NV_VARIABLE_NAME, &gEfiVendorKeysNvGuid, (VOID **) &Data, &DataSize);\r
  if (!EFI_ERROR (Status)) {\r
    mVendorKeyState = *(UINT8 *)Data;\r
  } else {\r
    //\r
    // "VendorKeysNv" not exist, initialize it in VENDOR_KEYS_VALID state.\r
    //\r
    mVendorKeyState = VENDOR_KEYS_VALID;\r
    Status = AuthServiceInternalUpdateVariable (\r
               EFI_VENDOR_KEYS_NV_VARIABLE_NAME,\r
               &gEfiVendorKeysNvGuid,\r
               &mVendorKeyState,\r
               sizeof (UINT8),\r
               EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACCESS\r
               );\r
    if (EFI_ERROR (Status)) {\r
      return Status;\r
    }\r
  }\r
\r
  //\r
  // Create "VendorKeys" variable with BS+RT attribute set.\r
  //\r
  Status = AuthServiceInternalUpdateVariable (\r
             EFI_VENDOR_KEYS_VARIABLE_NAME,\r
             &gEfiGlobalVariableGuid,\r
             &mVendorKeyState,\r
             sizeof (UINT8),\r
             EFI_VARIABLE_RUNTIME_ACCESS | EFI_VARIABLE_BOOTSERVICE_ACCESS\r
             );\r
  if (EFI_ERROR (Status)) {\r
    return Status;\r
  }\r
\r
  DEBUG ((EFI_D_INFO, "Variable %s is %x\n", EFI_VENDOR_KEYS_VARIABLE_NAME, mVendorKeyState));\r
\r
  AuthVarLibContextOut->StructVersion = AUTH_VAR_LIB_CONTEXT_OUT_STRUCT_VERSION;\r
  AuthVarLibContextOut->StructSize = sizeof (AUTH_VAR_LIB_CONTEXT_OUT);\r
  AuthVarLibContextOut->AuthVarEntry = mAuthVarEntry;\r
  AuthVarLibContextOut->AuthVarEntryCount = sizeof (mAuthVarEntry) / sizeof (mAuthVarEntry[0]);\r
  mAuthVarAddressPointer[0] = (VOID **) &mPubKeyStore;\r
  mAuthVarAddressPointer[1] = (VOID **) &mCertDbStore;\r
  mAuthVarAddressPointer[2] = (VOID **) &mHashCtx;\r
  mAuthVarAddressPointer[3] = (VOID **) &mAuthVarLibContextIn;\r
  mAuthVarAddressPointer[4] = (VOID **) &(mAuthVarLibContextIn->FindVariable),\r
  mAuthVarAddressPointer[5] = (VOID **) &(mAuthVarLibContextIn->FindNextVariable),\r
  mAuthVarAddressPointer[6] = (VOID **) &(mAuthVarLibContextIn->UpdateVariable),\r
  mAuthVarAddressPointer[7] = (VOID **) &(mAuthVarLibContextIn->GetScratchBuffer),\r
  mAuthVarAddressPointer[8] = (VOID **) &(mAuthVarLibContextIn->CheckRemainingSpaceForConsistency),\r
  mAuthVarAddressPointer[9] = (VOID **) &(mAuthVarLibContextIn->AtRuntime),\r
  AuthVarLibContextOut->AddressPointer = mAuthVarAddressPointer;\r
  AuthVarLibContextOut->AddressPointerCount = sizeof (mAuthVarAddressPointer) / sizeof (mAuthVarAddressPointer[0]);\r
\r
  //\r
  // Cache UserPhysicalPresent State. \r
  // Platform should report PhysicalPresent before this point\r
  //\r
  mUserPhysicalPresent = UserPhysicalPresent();\r
\r
  return Status;\r
}\r
\r
/**\r
  Process variable with EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS/EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACCESS set.\r
\r
  @param[in] VariableName           Name of the variable.\r
  @param[in] VendorGuid             Variable vendor GUID.\r
  @param[in] Data                   Data pointer.\r
  @param[in] DataSize               Size of Data.\r
  @param[in] Attributes             Attribute value of the variable.\r
\r
  @retval EFI_SUCCESS               The firmware has successfully stored the variable and its data as\r
                                    defined by the Attributes.\r
  @retval EFI_INVALID_PARAMETER     Invalid parameter.\r
  @retval EFI_WRITE_PROTECTED       Variable is write-protected.\r
  @retval EFI_OUT_OF_RESOURCES      There is not enough resource.\r
  @retval EFI_SECURITY_VIOLATION    The variable is with EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS\r
                                    or EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACESS\r
                                    set, but the AuthInfo does NOT pass the validation\r
                                    check carried out by the firmware.\r
  @retval EFI_UNSUPPORTED           Unsupported to process authenticated variable.\r
\r
**/\r
EFI_STATUS\r
EFIAPI\r
AuthVariableLibProcessVariable (\r
  IN CHAR16         *VariableName,\r
  IN EFI_GUID       *VendorGuid,\r
  IN VOID           *Data,\r
  IN UINTN          DataSize,\r
  IN UINT32         Attributes\r
  )\r
{\r
  EFI_STATUS        Status;\r
\r
  if (CompareGuid (VendorGuid, &gEfiGlobalVariableGuid) && (StrCmp (VariableName, EFI_PLATFORM_KEY_NAME) == 0)){\r
    Status = ProcessVarWithPk (VariableName, VendorGuid, Data, DataSize, Attributes, TRUE);\r
  } else if (CompareGuid (VendorGuid, &gEfiGlobalVariableGuid) && (StrCmp (VariableName, EFI_KEY_EXCHANGE_KEY_NAME) == 0)) {\r
    Status = ProcessVarWithPk (VariableName, VendorGuid, Data, DataSize, Attributes, FALSE);\r
  } else if (CompareGuid (VendorGuid, &gEfiImageSecurityDatabaseGuid) &&\r
             ((StrCmp (VariableName, EFI_IMAGE_SECURITY_DATABASE)  == 0) ||\r
              (StrCmp (VariableName, EFI_IMAGE_SECURITY_DATABASE1) == 0) ||\r
              (StrCmp (VariableName, EFI_IMAGE_SECURITY_DATABASE2) == 0)\r
             )) {\r
    Status = ProcessVarWithPk (VariableName, VendorGuid, Data, DataSize, Attributes, FALSE);\r
    if (EFI_ERROR (Status)) {\r
      Status = ProcessVarWithKek (VariableName, VendorGuid, Data, DataSize, Attributes);\r
    }\r
  } else {\r
    Status = ProcessVariable (VariableName, VendorGuid, Data, DataSize, Attributes);\r
  }\r
\r
  return Status;\r
}\r