Fix security check.

[mirror_edk2.git] / MdeModulePkg / Universal / PCD / Dxe / Service.c

diff --git a/MdeModulePkg/Universal/PCD/Dxe/Service.c b/MdeModulePkg/Universal/PCD/Dxe/Service.c
index 71e92399f42642a1749633f42f1b20c329dd088b..f07c107412ff9c68b922988fcc0bd31c51e2d67f 100644 (file)
--- a/MdeModulePkg/Universal/PCD/Dxe/Service.c
+++ b/MdeModulePkg/Universal/PCD/Dxe/Service.c
@@ -123,8 +123,8 @@ GetWorker (
                               \r
       VariableHead = (VARIABLE_HEAD *) (PcdDb + Offset);\r
       \r
-      Guid = &(GuidTable[VariableHead->GuidTableIndex]);\r
-      Name = &(StringTable[VariableHead->StringIndex]);\r
+      Guid = GuidTable + VariableHead->GuidTableIndex;\r
+      Name = StringTable + VariableHead->StringIndex;\r
       VaraiableDefaultBuffer = (UINT8 *) PcdDb + VariableHead->DefaultValueOffset;\r
 \r
       Status = GetHiiVariable (Guid, Name, &Data, &DataSize);\r
@@ -154,7 +154,7 @@ GetWorker (
 \r
     case PCD_TYPE_STRING:\r
       StringTableIdx = (UINT16) *((UINT8 *) PcdDb + Offset);\r
-      RetPtr = (VOID *) &StringTable[StringTableIdx];\r
+      RetPtr = (VOID *) (StringTable + StringTableIdx);\r
       break;\r
 \r
     case PCD_TYPE_DATA:\r
@@ -774,7 +774,7 @@ SetWorker (
     \r
     case PCD_TYPE_STRING:\r
       if (SetPtrTypeSize (TmpTokenNumber, Size)) {\r
-        CopyMem (&StringTable[*((UINT16 *)InternalData)], Data, *Size);\r
+        CopyMem (StringTable + *((UINT16 *)InternalData), Data, *Size);\r
         Status = EFI_SUCCESS;\r
       } else {\r
         Status = EFI_INVALID_PARAMETER;\r
@@ -794,8 +794,8 @@ SetWorker (
                               \r
       VariableHead = (VARIABLE_HEAD *) (PcdDb + Offset);\r
       \r
-      Guid = &(GuidTable[VariableHead->GuidTableIndex]);\r
-      Name = &(StringTable[VariableHead->StringIndex]);\r
+      Guid = GuidTable + VariableHead->GuidTableIndex;\r
+      Name = StringTable + VariableHead->StringIndex;\r
       VariableOffset = VariableHead->Offset;\r
 \r
       Status = SetHiiVariable (Guid, Name, Data, *Size, VariableOffset);\r