Fixed error messages
authorlhauch <lhauch@7335b38e-4728-0410-8992-fb3ffe349368>
Fri, 18 Jan 2008 19:33:12 +0000 (19:33 +0000)
committerlhauch <lhauch@7335b38e-4728-0410-8992-fb3ffe349368>
Fri, 18 Jan 2008 19:33:12 +0000 (19:33 +0000)
git-svn-id: https://buildtools.tianocore.org/svn/buildtools/trunk/BaseTools@955 7335b38e-4728-0410-8992-fb3ffe349368

Source/C/GenFv/GenFv.c
Source/C/GenFv/GenFvInternalLib.c

index c528340..ea7bbf9 100644 (file)
@@ -192,7 +192,7 @@ Returns:
   SetUtilityName (UTILITY_NAME);\r
 \r
   if (argc == 1) {\r
-    Error (NULL, 0, 1001, "Missing options", "No input options");\r
+    Error (NULL, 0, 1001, "Missing options", "No input options specified.");\r
     Usage ();\r
     return STATUS_ERROR;\r
   }\r
@@ -291,7 +291,7 @@ Returns:
         return STATUS_ERROR;\r
       }\r
       if (LogLevel > 9) {\r
-        Error (NULL, 0, 1003, "Invalid option value", "Debug Level range is 0~9, currnt input level is %d", LogLevel);\r
+        Error (NULL, 0, 1003, "Invalid option value", "Debug Level range is 0-9, current input level is %d", LogLevel);\r
         return STATUS_ERROR;\r
       }\r
       SetPrintLevel (LogLevel);\r
@@ -320,7 +320,7 @@ Returns:
   VerboseMsg ("the input file name is %s", InfFileName);\r
 \r
   if (!DumpCapsule && OutFileName == NULL) {\r
-    Error (NULL, 0, 1001, "Missing option", "Output file name");\r
+    Error (NULL, 0, 1001, "Missing option", "Output File");\r
     return STATUS_ERROR;\r
   }\r
   if (OutFileName != NULL) {\r
index b934f09..6a3673e 100644 (file)
@@ -219,7 +219,7 @@ Returns:
   } else {\r
     memcpy (&FvInfo->FvGuid, &mEfiFirmwareFileSystem2Guid, sizeof (EFI_GUID));\r
   }\r
-  DebugMsg (NULL, 0, 9, "Fv File Guid", "%08X-%04X-%04X-%02X%02X-%02X%02X%02X%02X%02X%02X", \r
+  DebugMsg (NULL, 0, 9, "FV File Guid", "%08X-%04X-%04X-%02X%02X-%02X%02X%02X%02X%02X%02X", \r
                 FvInfo->FvGuid.Data1,\r
                 FvInfo->FvGuid.Data2,\r
                 FvInfo->FvGuid.Data3,\r
@@ -291,7 +291,7 @@ Returns:
       }\r
 \r
       FvInfo->FvBlocks[Index].Length = (UINT32) Value64;\r
-      DebugMsg (NULL, 0, 9, "Fv Block Size", "%s = %s", EFI_BLOCK_SIZE_STRING, Value);\r
+      DebugMsg (NULL, 0, 9, "FV Block Size", "%s = %s", EFI_BLOCK_SIZE_STRING, Value);\r
     } else {\r
       //\r
       // If there is no blocks size, but there is the number of block, then we have a mismatched pair\r
@@ -299,7 +299,7 @@ Returns:
       //\r
       Status = FindToken (InfFile, OPTIONS_SECTION_STRING, EFI_NUM_BLOCKS_STRING, Index, Value);\r
       if (!EFI_ERROR (Status)) {\r
-        Error (NULL, 0, 2000, "Invalid parameter", "%s and %s must specify both", EFI_NUM_BLOCKS_STRING, EFI_BLOCK_SIZE_STRING);\r
+        Error (NULL, 0, 2000, "Invalid parameter", "both %s and %s must be specified.", EFI_NUM_BLOCKS_STRING, EFI_BLOCK_SIZE_STRING);\r
         return EFI_ABORTED;\r
       } else {\r
         //\r
@@ -325,12 +325,12 @@ Returns:
       }\r
 \r
       FvInfo->FvBlocks[Index].NumBlocks = (UINT32) Value64;\r
-      DebugMsg (NULL, 0, 9, "Fv Block Number", "%s = %s", EFI_NUM_BLOCKS_STRING, Value);\r
+      DebugMsg (NULL, 0, 9, "FV Block Number", "%s = %s", EFI_NUM_BLOCKS_STRING, Value);\r
     }\r
   }\r
 \r
   if (Index == 0) {\r
-    Error (NULL, 0, 2001, "Missing required argument", "block size");\r
+    Error (NULL, 0, 2001, "Missing required argument", "block size.");\r
     return EFI_ABORTED;\r
   }\r
 \r
@@ -348,14 +348,14 @@ Returns:
       // Add the file\r
       //\r
       strcpy (FvInfo->FvFiles[Index], Value);\r
-      DebugMsg (NULL, 0, 9, "Fv component file", "the %dth name is %s", Index, Value);\r
+      DebugMsg (NULL, 0, 9, "FV component file", "the %dth name is %s", Index, Value);\r
     } else {\r
       break;\r
     }\r
   }\r
 \r
   if (Index == 0) {\r
-    Warning (NULL, 0, 0, "Fv components are not specified.", NULL);\r
+    Warning (NULL, 0, 0, "FV components are not specified.", NULL);\r
   }\r
   //\r
   // Compute size for easy access later\r
@@ -723,7 +723,7 @@ Returns:
   //\r
   FileBuffer = malloc (FileSize);\r
   if (FileBuffer == NULL) {\r
-    Error (NULL, 0, 4001, "Resouce", "memory cannot be allocated");\r
+    Error (NULL, 0, 4001, "Resouce", "memory cannot be allocated!");\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
@@ -749,7 +749,7 @@ Returns:
   Status = VerifyFfsFile (FileBuffer);\r
   if (EFI_ERROR (Status)) {\r
     free (FileBuffer);\r
-    Error (NULL, 0, 3000, "Invalid", "%s is FFS file", FvInfo->FvFiles[Index]);\r
+    Error (NULL, 0, 3000, "Invalid", "%s is a FFS file.", FvInfo->FvFiles[Index]);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -758,7 +758,7 @@ Returns:
   //\r
   if (FileSize > (UINTN) ((UINTN) *VtfFileImage - (UINTN) FvImage->CurrentFilePointer)) {\r
     free (FileBuffer);\r
-    Error (NULL, 0, 4002, "Resource", "Fv space is full not to add %s file", FvInfo->FvFiles[Index]);\r
+    Error (NULL, 0, 4002, "Resource", "FV space is full, not enough room to add file %s.", FvInfo->FvFiles[Index]);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
@@ -767,7 +767,7 @@ Returns:
   //\r
   for (Index1 = 0; Index1 < Index; Index1 ++) {\r
     if (CompareGuid ((EFI_GUID *) FileBuffer, &mFileGuidArray [Index1]) == 0) {\r
-      Error (NULL, 0, 2000, "Invalid parameter", "the %dth file and %dth file has the same File Guid", Index1, Index);\r
+      Error (NULL, 0, 2000, "Invalid parameter", "the %dth file and %dth file have the same file GUID.", Index1, Index);\r
       return EFI_INVALID_PARAMETER;\r
     }\r
   }\r
@@ -805,7 +805,7 @@ Returns:
       // Sanity check. The file MUST align appropriately\r
       //\r
       if (((UINTN) *VtfFileImage + sizeof (EFI_FFS_FILE_HEADER) - (UINTN) FvImage->FileImage) % (1 << CurrentFileAlignment)) {\r
-        Error (NULL, 0, 3000, "Invalid", "VTF file does not align on %d-byte boundary", 1 << CurrentFileAlignment);\r
+        Error (NULL, 0, 3000, "Invalid", "VTF file cannot be aligned on a %d-byte boundary.", 1 << CurrentFileAlignment);\r
         free (FileBuffer);\r
         return EFI_ABORTED;\r
       }\r
@@ -825,7 +825,7 @@ Returns:
       //\r
       // Already found a VTF file.\r
       //\r
-      Error (NULL, 0, 3000, "Invalid", "multiple VTF files are illegal in a single FV");\r
+      Error (NULL, 0, 3000, "Invalid", "multiple VTF files are not permitted within a single FV.");\r
       free (FileBuffer);\r
       return EFI_ABORTED;\r
     }\r
@@ -836,7 +836,7 @@ Returns:
   //\r
   Status = AddPadFile (FvImage, 1 << CurrentFileAlignment);\r
   if (EFI_ERROR (Status)) {\r
-    Error (NULL, 0, 4002, "Resource", "Fv space is full not to add pad file for data align property");\r
+    Error (NULL, 0, 4002, "Resource", "FV space is full, could not add pad file for data alignment property.");\r
     free (FileBuffer);\r
     return EFI_ABORTED;\r
   }\r
@@ -855,7 +855,7 @@ Returns:
     memcpy (FvImage->CurrentFilePointer, FileBuffer, FileSize);\r
     FvImage->CurrentFilePointer += FileSize;\r
   } else {\r
-    Error (NULL, 0, 4002, "Resource", "Fv space is full not to add %s file", FvInfo->FvFiles[Index]);\r
+    Error (NULL, 0, 4002, "Resource", "FV space is full, cannot add file %s.", FvInfo->FvFiles[Index]);\r
     free (FileBuffer);\r
     return EFI_ABORTED;\r
   }\r
@@ -1032,7 +1032,7 @@ Returns:
   //\r
   Status = GetFileByType (EFI_FV_FILETYPE_SECURITY_CORE, 1, &SecCoreFile);\r
   if (EFI_ERROR (Status) || SecCoreFile == NULL) {\r
-    Error (NULL, 0, 3000, "Invaild", "could not find the Sec core in the FV");\r
+    Error (NULL, 0, 3000, "Invalid", "could not find the SEC core file in the FV.");\r
     return EFI_ABORTED;\r
   }\r
   //\r
@@ -1044,7 +1044,7 @@ Returns:
   }\r
 \r
   if (EFI_ERROR (Status)) {\r
-    Error (NULL, 0, 3000, "Invaild", "could not find PE32 section in SEC core file");\r
+    Error (NULL, 0, 3000, "Invalid", "could not find a PE32 section in the SEC core file.");\r
     return EFI_ABORTED;\r
   }\r
 \r
@@ -1056,7 +1056,7 @@ Returns:
             );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    Error (NULL, 0, 3000, "Invaild", "could not get PE32 entry point for SEC core");\r
+    Error (NULL, 0, 3000, "Invalid", "could not get the PE32 entry point for the SEC core.");\r
     return EFI_ABORTED;\r
   }\r
   //\r
@@ -1072,7 +1072,7 @@ Returns:
   //\r
   Status = GetFileByType (EFI_FV_FILETYPE_PEI_CORE, 1, &PeiCoreFile);\r
   if (EFI_ERROR (Status) || PeiCoreFile == NULL) {\r
-    Error (NULL, 0, 3000, "Invaild", "could not find the PEI core in the FV");\r
+    Error (NULL, 0, 3000, "Invalid", "could not find the PEI core in the FV.");\r
     return EFI_ABORTED;\r
   }\r
   //\r
@@ -1084,7 +1084,7 @@ Returns:
   }\r
 \r
   if (EFI_ERROR (Status)) {\r
-    Error (NULL, 0, 3000, "Invaild", "could not find PE32 or TE section in PEI core file");\r
+    Error (NULL, 0, 3000, "Invalid", "could not find either a PE32 or a TE section in PEI core file.");\r
     return EFI_ABORTED;\r
   }\r
 \r
@@ -1096,7 +1096,7 @@ Returns:
             );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    Error (NULL, 0, 3000, "Invaild", "could not get PE32 entry point for PEI core");\r
+    Error (NULL, 0, 3000, "Invalid", "could not get the PE32 entry point for the PEI core.");\r
     return EFI_ABORTED;\r
   }\r
   //\r
@@ -1120,7 +1120,7 @@ Returns:
     // Check if address is aligned on a 16 byte boundary\r
     //\r
     if (PeiCorePhysicalAddress & 0xF) {\r
-      Error (NULL, 0, 3000, "Invaild",\r
+      Error (NULL, 0, 3000, "Invalid",\r
         "PEI_CORE entry point is not aligned on a 16 byte boundary, address specified is %Xh.",\r
         PeiCorePhysicalAddress\r
         );\r
@@ -1150,7 +1150,7 @@ Returns:
     // Check if address is aligned on a 16 byte boundary\r
     //\r
     if (SecCorePhysicalAddress & 0xF) {\r
-      Error (NULL, 0, 3000, "Invaild",\r
+      Error (NULL, 0, 3000, "Invalid",\r
         "SALE_ENTRY entry point is not aligned on a 16 byte boundary, address specified is %Xh.",\r
         SecCorePhysicalAddress\r
         );\r
@@ -1180,7 +1180,7 @@ Returns:
     \r
     Ia32SecEntryOffset   = SecCorePhysicalAddress - (FV_IMAGES_TOP_ADDRESS - IA32_SEC_CORE_ENTRY_OFFSET + 2);\r
     if (Ia32SecEntryOffset <= -65536) {\r
-      Error (NULL, 0, 3000, "Invaild", "The SEC EXE file size is too big");\r
+      Error (NULL, 0, 3000, "Invalid", "The SEC EXE file size is too large, it must be less than 64K.");\r
       return STATUS_ERROR;\r
     }\r
     \r
@@ -1191,7 +1191,7 @@ Returns:
     //\r
     Ia32ResetAddressPtr   = (UINT32 *) ((UINTN) FvImage->Eof - 4);\r
     *Ia32ResetAddressPtr  = (UINT32) (FvInfo->BaseAddress);\r
-    DebugMsg (NULL, 0, 9, "update BFV base address in the top FV image", "BFV base address = 0x%X", FvInfo->BaseAddress);\r
+    DebugMsg (NULL, 0, 9, "update BFV base address in the top FV image", "BFV base address = 0x%X.", FvInfo->BaseAddress);\r
 \r
     //\r
     // Update the Startup AP in the FVH header block ZeroVector region.\r
@@ -1211,7 +1211,7 @@ Returns:
       //\r
       Status = FindApResetVectorPosition (FvImage, &BytePointer);\r
       if (EFI_ERROR (Status)) {\r
-        Error (NULL, 0, 3000, "Invaild", "Can't find the appropriate space in FvImage to add Ap reset vector!");\r
+        Error (NULL, 0, 3000, "Invalid", "Cannot find the appropriate location in FvImage to add Ap reset vector!");\r
         return EFI_ABORTED;\r
       }\r
     }\r
@@ -1248,7 +1248,7 @@ Returns:
     Ia32ResetAddressPtr   = (UINT32 *) ((UINTN) FvImage->Eof - 8);\r
     *Ia32ResetAddressPtr  = IpiVector;\r
   } else {\r
-    Error (NULL, 0, 3000, "Invaild", "machine type=0x%X in PEI core", (UINT32) MachineType);\r
+    Error (NULL, 0, 3000, "Invalid", "machine type=0x%X in PEI core.", (UINT32) MachineType);\r
     return EFI_ABORTED;\r
   }\r
 \r
@@ -1337,7 +1337,7 @@ Returns:
     // Verify DOS header is expected\r
     //\r
     if (DosHeader->e_magic != EFI_IMAGE_DOS_SIGNATURE) {\r
-      Error (NULL, 0, 3000, "Invaild", "Unknown magic number in the DOS header, 0x%04X.", DosHeader->e_magic);\r
+      Error (NULL, 0, 3000, "Invalid", "Unknown magic number in the DOS header, 0x%04X.", DosHeader->e_magic);\r
       return EFI_UNSUPPORTED;\r
     }\r
     //\r
@@ -1349,7 +1349,7 @@ Returns:
     // Verify NT header is expected\r
     //\r
     if (NtHeader->Signature != EFI_IMAGE_NT_SIGNATURE) {\r
-      Error (NULL, 0, 3000, "Invaild", "Unrecognized image signature 0x%08X.", NtHeader->Signature);\r
+      Error (NULL, 0, 3000, "Invalid", "Unrecognized image signature 0x%08X.", NtHeader->Signature);\r
       return EFI_UNSUPPORTED;\r
     }\r
     //\r
@@ -1364,7 +1364,7 @@ Returns:
   // Verify machine type is supported\r
   //\r
   if (*MachineType != EFI_IMAGE_MACHINE_IA32 && *MachineType != EFI_IMAGE_MACHINE_IA64 && *MachineType != EFI_IMAGE_MACHINE_X64 && *MachineType != EFI_IMAGE_MACHINE_EBC) {\r
-    Error (NULL, 0, 3000, "Invaild", "Unrecognized machine type in the PE32 file.");\r
+    Error (NULL, 0, 3000, "Invalid", "Unrecognized machine type in the PE32 file.");\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -1467,7 +1467,7 @@ Returns:
     strcpy (FvMapName, FvFileName);\r
     strcat (FvMapName, ".map");\r
   }\r
-  VerboseMsg ("Fv Map file name is %s", FvMapName);\r
+  VerboseMsg ("FV Map file name is %s", FvMapName);\r
 \r
   FvMapFile = fopen (FvMapName, "w");\r
   if (FvMapFile == NULL) {\r
@@ -1496,7 +1496,7 @@ Returns:
   if (EFI_ERROR (Status)) {\r
     return Status;    \r
   }\r
-  VerboseMsg ("the generated Fv image size is %d bytes", FvInfo.Size);\r
+  VerboseMsg ("the generated FV image size is %d bytes", FvInfo.Size);\r
   \r
   //\r
   // support fv image and empty fv image\r
@@ -1615,7 +1615,7 @@ Returns:
     //\r
     Status = PadFvImage (&FvImageMemoryFile, VtfFileImage);\r
     if (EFI_ERROR (Status)) {\r
-      Error (NULL, 0, 4002, "Resource", "Fv space is full not to add pad file between the last file and the VTF file.");\r
+      Error (NULL, 0, 4002, "Resource", "FV space is full, cannot add pad file between the last file and the VTF file.");\r
       goto Finish;\r
     }\r
     //\r
@@ -2036,7 +2036,7 @@ Returns:
     ImageContext.ImageRead  = (PE_COFF_LOADER_READ_FILE) FfsRebaseImageRead;\r
     Status                  = PeCoffLoaderGetImageInfo (&ImageContext);\r
     if (EFI_ERROR (Status)) {\r
-      Error (NULL, 0, 3000, "Invalid", "GetImageInfo() call failed on rebase %s", FileName);\r
+      Error (NULL, 0, 3000, "Invalid", "GetImageInfo() call failed on rebase %s.", FileName);\r
       return Status;\r
     }\r
 \r
@@ -2067,7 +2067,7 @@ Returns:
           //\r
           // Xip module has the same section alignment and file alignment.\r
           //\r
-          Error (NULL, 0, 3000, "Invalid", "Section-Alignment and File-Alignment does not match : %s", FileName);\r
+          Error (NULL, 0, 3000, "Invalid", "Section-Alignment and File-Alignment do not match : %s.", FileName);\r
           return EFI_ABORTED;\r
         }\r
         //\r
@@ -2083,7 +2083,7 @@ Returns:
             Cptr --;\r
           }\r
           if (*Cptr != '.') {\r
-            Error (NULL, 0, 3000, "Invalid", "The file %s has not .reloc section", FileName);\r
+            Error (NULL, 0, 3000, "Invalid", "The file %s has no .reloc section.", FileName);\r
             return EFI_ABORTED;\r
           } else {\r
             *(Cptr + 1) = 'e';\r
@@ -2093,7 +2093,7 @@ Returns:
           }\r
           PeFile = fopen (PeFileName, "rb");\r
           if (PeFile == NULL) {\r
-            Error (NULL, 0, 3000, "Invalid", "The file %s has not .reloc section", FileName);\r
+            Error (NULL, 0, 3000, "Invalid", "The file %s has no .reloc section.", FileName);\r
             return EFI_ABORTED;\r
           }\r
           //\r
@@ -2102,7 +2102,7 @@ Returns:
           PeFileSize = _filelength (fileno (PeFile));\r
           PeFileBuffer = (UINT8 *) malloc (PeFileSize);\r
           if (PeFileBuffer == NULL) {\r
-            Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated when rebase %s", FileName);\r
+            Error (NULL, 0, 4001, "Resource", "memory cannot be allocated on rebase of %s", FileName);\r
             return EFI_OUT_OF_RESOURCES;\r
           }\r
           //\r
@@ -2119,7 +2119,7 @@ Returns:
           ImageContext.Handle = PeFileBuffer;\r
           Status              = PeCoffLoaderGetImageInfo (&ImageContext);\r
           if (EFI_ERROR (Status)) {\r
-            Error (NULL, 0, 3000, "Invalid", "GetImageInfo() call failed on rebase %s", FileName);\r
+            Error (NULL, 0, 3000, "Invalid", "GetImageInfo() call failed on rebase of %s", FileName);\r
             return Status;\r
           }\r
           ImageContext.RelocationsStripped = FALSE;\r
@@ -2196,7 +2196,7 @@ Returns:
     //\r
     MemoryImagePointer = (UINT8 *) malloc ((UINTN) ImageContext.ImageSize + ImageContext.SectionAlignment);\r
     if (MemoryImagePointer == NULL) {\r
-      Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated when rebase %s", FileName);\r
+      Error (NULL, 0, 4001, "Resource", "memory cannot be allocated on rebase of %s", FileName);\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
     memset ((VOID *) MemoryImagePointer, 0, (UINTN) ImageContext.ImageSize + ImageContext.SectionAlignment);\r
@@ -2204,7 +2204,7 @@ Returns:
     \r
     Status =  PeCoffLoaderLoadImage (&ImageContext);\r
     if (EFI_ERROR (Status)) {\r
-      Error (NULL, 0, 3000, "Invalid", "LocateImage() call failed on rebase %s", FileName);\r
+      Error (NULL, 0, 3000, "Invalid", "LocateImage() call failed on rebase of %s", FileName);\r
       free ((VOID *) MemoryImagePointer);\r
       return Status;\r
     }\r
@@ -2212,7 +2212,7 @@ Returns:
     ImageContext.DestinationAddress = NewPe32BaseAddress;\r
     Status                          = PeCoffLoaderRelocateImage (&ImageContext);\r
     if (EFI_ERROR (Status)) {\r
-      Error (NULL, 0, 3000, "Invalid", "RelocateImage() call failed on rebase %s", FileName);\r
+      Error (NULL, 0, 3000, "Invalid", "RelocateImage() call failed on rebase of %s", FileName);\r
       free ((VOID *) MemoryImagePointer);\r
       return Status;\r
     }\r
@@ -2342,7 +2342,7 @@ Returns:
         Cptr --;\r
       }\r
       if (*Cptr != '.') {\r
-        Error (NULL, 0, 3000, "Invalid", "The file %s has not .reloc section", FileName);\r
+        Error (NULL, 0, 3000, "Invalid", "The file %s has no .reloc section.", FileName);\r
         return EFI_ABORTED;\r
       } else {\r
         *(Cptr + 1) = 'e';\r
@@ -2352,7 +2352,7 @@ Returns:
       }\r
       PeFile = fopen (PeFileName, "rb");\r
       if (PeFile == NULL) {\r
-        Error (NULL, 0, 3000, "Invalid", "The file %s has not .reloc section", FileName);\r
+        Error (NULL, 0, 3000, "Invalid", "The file %s has no .reloc section.", FileName);\r
         return EFI_ABORTED;\r
       }\r
       //\r
@@ -2361,7 +2361,7 @@ Returns:
       PeFileSize = _filelength (fileno (PeFile));\r
       PeFileBuffer = (UINT8 *) malloc (PeFileSize);\r
       if (PeFileBuffer == NULL) {\r
-        Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated when rebase %s", FileName);\r
+        Error (NULL, 0, 4001, "Resource", "memory cannot be allocated on rebase of %s", FileName);\r
         return EFI_OUT_OF_RESOURCES;\r
       }\r
       //\r
@@ -2389,7 +2389,7 @@ Returns:
     //\r
     MemoryImagePointer = (UINT8 *) malloc ((UINTN) ImageContext.ImageSize + ImageContext.SectionAlignment);\r
     if (MemoryImagePointer == NULL) {\r
-      Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated when rebase %s", FileName);\r
+      Error (NULL, 0, 4001, "Resource", "memory cannot be allocated on rebase of %s", FileName);\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
     memset ((VOID *) MemoryImagePointer, 0, (UINTN) ImageContext.ImageSize + ImageContext.SectionAlignment);\r
@@ -2397,7 +2397,7 @@ Returns:
 \r
     Status =  PeCoffLoaderLoadImage (&ImageContext);\r
     if (EFI_ERROR (Status)) {\r
-      Error (NULL, 0, 3000, "Invalid", "LocateImage() call failed on rebase %s", FileName);\r
+      Error (NULL, 0, 3000, "Invalid", "LocateImage() call failed on rebase of %s", FileName);\r
       free ((VOID *) MemoryImagePointer);\r
       return Status;\r
     }\r
@@ -2791,7 +2791,7 @@ Returns:
     } else if (strstr (Value, "PersistAcrossReset") != NULL) {\r
       CapInfo->Flags |= CAPSULE_FLAGS_PERSIST_ACROSS_RESET; \r
     } else {\r
-      Error (NULL, 0, 2000, "Invalid parameter", "invalid Flag setting for %s", EFI_CAPSULE_FLAGS_STRING);\r
+      Error (NULL, 0, 2000, "Invalid parameter", "invalid Flag setting for %s.", EFI_CAPSULE_FLAGS_STRING);\r
       return EFI_ABORTED;\r
     }\r
     DebugMsg (NULL, 0, 9, "Capsule Flag", Value);\r
@@ -2892,7 +2892,7 @@ Returns:
   }\r
 \r
   if (CapInfo.HeaderSize < sizeof (EFI_CAPSULE_HEADER)) {\r
-    Error (NULL, 0, 2000, "Invalid parameter", "The specified HeaderSize can't be less than the size of EFI_CAPSULE_HEADER.");\r
+    Error (NULL, 0, 2000, "Invalid parameter", "The specified HeaderSize cannot be less than the size of EFI_CAPSULE_HEADER.");\r
     return EFI_INVALID_PARAMETER;\r
   }\r
   \r
@@ -2928,7 +2928,7 @@ Returns:
   //\r
   CapBuffer = (UINT8 *) malloc (CapSize);\r
   if (CapBuffer == NULL) {\r
-    Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated for capsule");\r
+    Error (NULL, 0, 4001, "Resource", "memory cannot be allocated for creating the capsule.");\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r