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

Source/C/GenFw/GenFw.c

index 08442e4..ed456d4 100644 (file)
@@ -249,7 +249,7 @@ Returns:
   // Generic check for AcpiTable length.\r
   //\r
   if (AcpiHeader->Length > Length) {\r
-    Error (NULL, 0, 3000, "Invalid", "failed to pass AcpiTable Length check", NULL);\r
+    Error (NULL, 0, 3000, "Invalid", "AcpiTable length check failed.", NULL);\r
     return STATUS_ERROR;\r
   }\r
 \r
@@ -274,11 +274,11 @@ Returns:
       ExpectedLength = sizeof(EFI_ACPI_3_0_FIXED_ACPI_DESCRIPTION_TABLE);\r
       break;\r
     default:\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass FACP revision check");\r
+      Error (NULL, 0, 3000, "Invalid", "FACP revision check failed.");\r
       return STATUS_ERROR;\r
     }\r
     if (ExpectedLength != AcpiHeader->Length) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass FACP Length check");\r
+      Error (NULL, 0, 3000, "Invalid", "FACP length check failed.");\r
       return STATUS_ERROR;\r
     }\r
     break;\r
@@ -291,13 +291,13 @@ Returns:
     if ((Facs->Version != 0) &&\r
         (Facs->Version != EFI_ACPI_2_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_VERSION) &&\r
         (Facs->Version != EFI_ACPI_3_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_VERSION)){\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass FACS version check");\r
+      Error (NULL, 0, 3000, "Invalid", "FACS version check failed.");\r
       return STATUS_ERROR;\r
     }\r
     if ((Facs->Length != sizeof(EFI_ACPI_1_0_FIRMWARE_ACPI_CONTROL_STRUCTURE)) &&\r
         (Facs->Length != sizeof(EFI_ACPI_2_0_FIRMWARE_ACPI_CONTROL_STRUCTURE)) &&\r
         (Facs->Length != sizeof(EFI_ACPI_3_0_FIRMWARE_ACPI_CONTROL_STRUCTURE))) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass FACS Length check");\r
+      Error (NULL, 0, 3000, "Invalid", "FACS length check failed.");\r
       return STATUS_ERROR;\r
     }\r
     break;\r
@@ -307,11 +307,11 @@ Returns:
   //\r
   case EFI_ACPI_3_0_DIFFERENTIATED_SYSTEM_DESCRIPTION_TABLE_SIGNATURE:\r
     if (AcpiHeader->Revision > EFI_ACPI_3_0_DIFFERENTIATED_SYSTEM_DESCRIPTION_TABLE_REVISION) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass DSDT revision check");\r
+      Error (NULL, 0, 3000, "Invalid", "DSDT revision check failed.");\r
       return STATUS_ERROR;\r
     }\r
     if (AcpiHeader->Length <= sizeof(EFI_ACPI_DESCRIPTION_HEADER)) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass DSDT Length check");\r
+      Error (NULL, 0, 3000, "Invalid", "DSDT length check failed.");\r
       return STATUS_ERROR;\r
     }\r
     break;\r
@@ -323,11 +323,11 @@ Returns:
     if ((AcpiHeader->Revision != EFI_ACPI_1_0_MULTIPLE_APIC_DESCRIPTION_TABLE_REVISION) &&\r
         (AcpiHeader->Revision != EFI_ACPI_2_0_MULTIPLE_APIC_DESCRIPTION_TABLE_REVISION) &&\r
         (AcpiHeader->Revision != EFI_ACPI_3_0_MULTIPLE_APIC_DESCRIPTION_TABLE_REVISION)) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass APIC revision check");\r
+      Error (NULL, 0, 3000, "Invalid", "APIC revision check failed.");\r
       return STATUS_ERROR;\r
     }\r
     if (AcpiHeader->Length <= sizeof(EFI_ACPI_DESCRIPTION_HEADER) + sizeof(UINT32) + sizeof(UINT32)) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass APIC Length check");\r
+      Error (NULL, 0, 3000, "Invalid", "APIC length check failed.");\r
       return STATUS_ERROR;\r
     }\r
     break;\r
@@ -337,11 +337,11 @@ Returns:
   //\r
   case EFI_ACPI_3_0_PCI_EXPRESS_MEMORY_MAPPED_CONFIGURATION_SPACE_BASE_ADDRESS_DESCRIPTION_TABLE_SIGNATURE:\r
     if (AcpiHeader->Revision != EFI_ACPI_MEMORY_MAPPED_CONFIGURATION_SPACE_ACCESS_TABLE_REVISION) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass MCFG revision check");\r
+      Error (NULL, 0, 3000, "Invalid", "MCFG revision check failed.");\r
       return STATUS_ERROR;\r
     }\r
     if (AcpiHeader->Length <= sizeof(EFI_ACPI_DESCRIPTION_HEADER) + sizeof(UINT64)) {\r
-      Error (NULL, 0, 3000, "Invalid", "failed to pass MCFG Length check");\r
+      Error (NULL, 0, 3000, "Invalid", "MCFG length check failed.");\r
       return STATUS_ERROR;\r
     }\r
     break;\r
@@ -693,7 +693,7 @@ WriteSections(
   memset(CoffFile + CoffSectionsOffset[Idx], 0, Shdr->sh_size);\r
   break;\r
       default:\r
-  Error (NULL, 0, 3000, "Invalid", "%s unhandle section type %x", mInImageName, (UINTN)Shdr->sh_type);\r
+  Error (NULL, 0, 3000, "Invalid", "%s unhandled section type %x.", mInImageName, (UINTN)Shdr->sh_type);\r
       }\r
     }\r
   }\r
@@ -722,7 +722,7 @@ WriteSections(
   if (Sym->st_shndx == SHN_UNDEF\r
       || Sym->st_shndx == SHN_ABS\r
       || Sym->st_shndx > Ehdr->e_shnum) {\r
-    Error (NULL, 0, 3000, "Invalid", "%s bad symbol definition", mInImageName);\r
+    Error (NULL, 0, 3000, "Invalid", "%s bad symbol definition.", mInImageName);\r
   }\r
   SymShdr = GetShdrByIndex(Sym->st_shndx);\r
 \r
@@ -751,7 +751,7 @@ WriteSections(
       - (SecOffset - SecShdr->sh_addr);\r
     break;\r
   default:\r
-    Error (NULL, 0, 3000, "Invalid", "%s unhandle section type %x", mInImageName, ELF_R_TYPE(Rel->r_info));\r
+    Error (NULL, 0, 3000, "Invalid", "%s unhandled section type %x.", mInImageName, ELF_R_TYPE(Rel->r_info));\r
   }\r
       }\r
     }\r
@@ -837,7 +837,7 @@ WriteRelocations(
        EFI_IMAGE_REL_BASED_HIGHLOW);\r
       break;\r
     default:\r
-      Error (NULL, 0, 3000, "Invalid", "%s unhandle section type %x", mInImageName, ELF_R_TYPE(Rel->r_info));\r
+      Error (NULL, 0, 3000, "Invalid", "%s unhandled section type %x.", mInImageName, ELF_R_TYPE(Rel->r_info));\r
     }\r
   }\r
       }\r
@@ -1047,7 +1047,7 @@ Returns:
   LogLevel          = 0;\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.");\r
     Usage ();\r
     return STATUS_ERROR;\r
   }\r
@@ -1200,7 +1200,7 @@ Returns:
         goto Finish;\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, currnt input level is %d", LogLevel);\r
         goto Finish;\r
       }\r
       SetPrintLevel (LogLevel);\r
@@ -1216,7 +1216,7 @@ Returns:
     if ((InputFileNum == 0) && (InputFileName == NULL)) {\r
       InputFileName = (CHAR8 **) malloc (MAXIMUM_INPUT_FILE_NUM * sizeof (CHAR8 *));\r
       if (InputFileName == NULL) {\r
-        Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated");\r
+        Error (NULL, 0, 4001, "Resource", "memory cannot be allocated!");\r
         return EFI_OUT_OF_RESOURCES;\r
       }\r
 \r
@@ -1231,7 +1231,7 @@ Returns:
                                   );\r
 \r
       if (InputFileName == NULL) {\r
-        Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated");\r
+        Error (NULL, 0, 4001, "Resource", "memory cannot be allocated!");\r
         return EFI_OUT_OF_RESOURCES;\r
       }\r
 \r
@@ -1246,7 +1246,7 @@ Returns:
   VerboseMsg ("%s tool start.", UTILITY_NAME);\r
 \r
   if (OutImageType == FW_DUMMY_IMAGE) {\r
-    Error (NULL, 0, 1001, "Missing option", "No action specified, such as -e, -c or -t");\r
+    Error (NULL, 0, 1001, "Missing option", "No create action specified; specify -e, -c or -t!");\r
     goto Finish;\r
   }\r
 \r
@@ -1262,7 +1262,7 @@ Returns:
   // Combine MciBinary files to one file\r
   //\r
   if ((OutImageType == FW_MERGE_IMAGE) && ReplaceFlag) {\r
-    Error (NULL, 0, 1002, "Conflicting option", "-r replace parameter can't be input together with -j merge files");\r
+    Error (NULL, 0, 1002, "Conflicting option", "-r replace option cannot be used with -j merge files option.");\r
     goto Finish;\r
   }\r
    \r
@@ -1347,7 +1347,7 @@ Returns:
       FileLength = _filelength (fileno (fpIn));\r
       FileBuffer = malloc (FileLength);\r
       if (FileBuffer == NULL) {\r
-        Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated");\r
+        Error (NULL, 0, 4001, "Resource", "memory cannot be allocated!");\r
         fclose (fpIn);\r
         goto Finish;\r
       }\r
@@ -1365,7 +1365,7 @@ Returns:
         fwrite (&MciPadValue, 1, 1, fpOut);\r
       }\r
       //\r
-      // free allcoated memory space\r
+      // free allocated memory space\r
       //\r
       free (FileBuffer);\r
       FileBuffer = NULL;\r
@@ -1404,11 +1404,11 @@ Returns:
     // Error if no data.\r
     //\r
     if (FileLength == 0) {\r
-      Error (NULL, 0, 3000, "Invalid", "no parse-able data found in file %s", mInImageName);\r
+      Error (NULL, 0, 3000, "Invalid", "no parseable data found in file %s", mInImageName);\r
       goto Finish;\r
     }\r
     if (FileLength < sizeof (MICROCODE_IMAGE_HEADER)) {\r
-      Error (NULL, 0, 3000, "Invalid", "amount of parse-able data in %s is insufficient to contain a microcode header", mInImageName);\r
+      Error (NULL, 0, 3000, "Invalid", "amount of parseable data in %s is insufficient to contain a microcode header", mInImageName);\r
       goto Finish;\r
     }\r
 \r
@@ -1417,7 +1417,7 @@ Returns:
     //\r
     FileBuffer = malloc (FileLength);\r
     if (FileBuffer == NULL) {\r
-      Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated");\r
+      Error (NULL, 0, 4001, "Resource", "memory cannot be allocated!");\r
       goto Finish;\r
     }\r
     //\r
@@ -1453,7 +1453,7 @@ Returns:
     }\r
 \r
     if (Index != FileLength) {\r
-      Error (NULL, 0, 3000, "Invalid", "file contents of %s (0x%x) do not equal expected TotalSize: 0x%04X", mInImageName, FileLength, Index);\r
+      Error (NULL, 0, 3000, "Invalid", "file length of %s (0x%x) does not equal expected TotalSize: 0x%04X.", mInImageName, FileLength, Index);\r
       goto Finish;\r
     }\r
 \r
@@ -1469,7 +1469,7 @@ Returns:
       Index       += sizeof (UINTN);\r
     }\r
     if (CheckSum != 0) {\r
-      Error (NULL, 0, 3000, "Invalid", "checksum (0x%x) failed on file contents of %s", CheckSum, mInImageName);\r
+      Error (NULL, 0, 3000, "Invalid", "checksum (0x%x) failed on file %s.", CheckSum, mInImageName);\r
       goto Finish;\r
     }\r
     //\r
@@ -1512,7 +1512,7 @@ Returns:
   FileLength = _filelength (fileno (fpIn));\r
   FileBuffer = malloc (FileLength);\r
   if (FileBuffer == NULL) {\r
-    Error (NULL, 0, 4001, "Resource", "memory cannot be allcoated");\r
+    Error (NULL, 0, 4001, "Resource", "memory cannot be allocated!");\r
     fclose (fpIn);\r
     goto Finish;\r
   }\r
@@ -1538,7 +1538,7 @@ Returns:
   if (OutImageType == DUMP_TE_HEADER) {\r
     memcpy (&TEImageHeader, FileBuffer, sizeof (TEImageHeader));\r
     if (TEImageHeader.Signature != EFI_TE_IMAGE_HEADER_SIGNATURE) {\r
-      Error (NULL, 0, 3000, "Invalid", "TE header signature of file %s is not correct", mInImageName);\r
+      Error (NULL, 0, 3000, "Invalid", "TE header signature of file %s is not correct.", mInImageName);\r
       goto Finish;      \r
     }\r
     if (fpInOut != NULL) {\r
@@ -1585,7 +1585,7 @@ Returns:
         // Default TE Image Type is Boot service driver\r
         //\r
         Type = EFI_IMAGE_SUBSYSTEM_EFI_BOOT_SERVICE_DRIVER;\r
-        VerboseMsg ("Efi Image substytem type is efi boot service driver");\r
+        VerboseMsg ("Efi Image subsystem type is efi boot service driver.");\r
       }\r
     } else {\r
       if (stricmp (ModuleType, "BASE") == 0 ||\r
@@ -1602,22 +1602,22 @@ Returns:
           stricmp (ModuleType, "DXE_SMM_DRIVER") == 0  ||\r
           stricmp (ModuleType, "UEFI_DRIVER") == 0) {\r
         Type = EFI_IMAGE_SUBSYSTEM_EFI_BOOT_SERVICE_DRIVER;\r
-        VerboseMsg ("Efi Image substytem type is efi boot service driver");\r
+        VerboseMsg ("Efi Image subsystem type is efi boot service driver.");\r
     \r
       } else if (stricmp (ModuleType, "UEFI_APPLICATION") == 0 || \r
                  stricmp (ModuleType, "APPLICATION") == 0) {\r
         Type = EFI_IMAGE_SUBSYSTEM_EFI_APPLICATION;\r
-        VerboseMsg ("Efi Image substytem type is efi application");\r
+        VerboseMsg ("Efi Image subsystem type is efi application.");\r
     \r
       } else if (stricmp (ModuleType, "DXE_RUNTIME_DRIVER") == 0 || \r
                  stricmp (ModuleType, "RT_DRIVER") == 0) {\r
         Type = EFI_IMAGE_SUBSYSTEM_EFI_RUNTIME_DRIVER;\r
-        VerboseMsg ("Efi Image substytem type is efi runtime driver");\r
+        VerboseMsg ("Efi Image subsystem type is efi runtime driver.");\r
     \r
       } else if (stricmp (ModuleType, "DXE_SAL_DRIVER") == 0 || \r
                  stricmp (ModuleType, "SAL_RT_DRIVER") == 0) {\r
         Type = EFI_IMAGE_SUBSYSTEM_SAL_RUNTIME_DRIVER;\r
-        VerboseMsg ("Efi Image substytem type is efi sal runtime driver");\r
+        VerboseMsg ("Efi Image subsystem type is efi sal runtime driver.");\r
     \r
       } else {\r
         Error (NULL, 0, 1003, "Invalid option value", "EFI_FILETYPE = %s", ModuleType);\r
@@ -1671,12 +1671,12 @@ Returns:
       //\r
       DosHdr = (EFI_IMAGE_DOS_HEADER *) FileBuffer;\r
       if (DosHdr->e_magic != EFI_IMAGE_DOS_SIGNATURE) {\r
-        Error (NULL, 0, 3000, "Invalid", "TE and DOS header signature are not found in %s image", mInImageName);\r
+        Error (NULL, 0, 3000, "Invalid", "TE and DOS header signatures were not found in %s image.", mInImageName);\r
         goto Finish;\r
       }\r
       PeHdr = (EFI_IMAGE_NT_HEADERS *)(FileBuffer + DosHdr->e_lfanew);\r
       if (PeHdr->Signature != EFI_IMAGE_NT_SIGNATURE) {\r
-        Error (NULL, 0, 3000, "Invalid", "PE header signature not found in %s image", mInImageName);\r
+        Error (NULL, 0, 3000, "Invalid", "PE header signature was not found in %s image.", mInImageName);\r
         goto Finish;\r
       }\r
       SectionHeader = (EFI_IMAGE_SECTION_HEADER *) ((UINT8 *) &(PeHdr->OptionalHeader) + PeHdr->FileHeader.SizeOfOptionalHeader);\r
@@ -1727,13 +1727,13 @@ Returns:
   //\r
   DosHdr = (EFI_IMAGE_DOS_HEADER *)FileBuffer;\r
   if (DosHdr->e_magic != EFI_IMAGE_DOS_SIGNATURE) {\r
-    Error (NULL, 0, 3000, "Invalid", "DOS header signature not found in %s image", mInImageName);\r
+    Error (NULL, 0, 3000, "Invalid", "DOS header signature was not found in %s image.", mInImageName);\r
     goto Finish;\r
   }\r
 \r
   PeHdr = (EFI_IMAGE_NT_HEADERS *)(FileBuffer + DosHdr->e_lfanew);\r
   if (PeHdr->Signature != EFI_IMAGE_NT_SIGNATURE) {\r
-    Error (NULL, 0, 3000, "Invalid", "PE header signature not found in %s image", mInImageName);\r
+    Error (NULL, 0, 3000, "Invalid", "PE header signature was not found in %s image.", mInImageName);\r
     goto Finish;\r
   }\r
   \r
@@ -1813,7 +1813,7 @@ Returns:
         }\r
 \r
         if (CheckAcpiTable (FileBuffer + SectionHeader->PointerToRawData, FileLength) != STATUS_SUCCESS) {\r
-          Error (NULL, 0, 3000, "Invalid", "failed to check ACPI table in %s", mInImageName);\r
+          Error (NULL, 0, 3000, "Invalid", "ACPI table check failed in %s.", mInImageName);\r
           goto Finish;\r
         }\r
         \r
@@ -1830,7 +1830,7 @@ Returns:
         goto Finish;\r
       }\r
     }\r
-    Error (NULL, 0, 3000, "Invalid", "failed to get ACPI table from %s", mInImageName);\r
+    Error (NULL, 0, 3000, "Invalid", "failed to get ACPI table from %s.", mInImageName);\r
     goto Finish;\r
   }\r
   //\r
@@ -2035,7 +2035,7 @@ Returns:
       //\r
       // Pack the subsystem and NumberOfSections into 1 byte. Make sure they fit both.\r
       //\r
-      Error (NULL, 0, 3000, "Invalid", "Image subsystem or NumberOfSections of PeImage %s cannot be packed into 1 byte", mInImageName);\r
+      Error (NULL, 0, 3000, "Invalid", "Image's subsystem or NumberOfSections of PeImage %s cannot be packed into 1 byte.", mInImageName);\r
       goto Finish;\r
     }\r
 \r
@@ -2043,11 +2043,11 @@ Returns:
       //\r
       // TeImage has the same section alignment and file alignment.\r
       //\r
-      Error (NULL, 0, 3000, "Invalid", "Section-Alignment and File-Alignment of PeImage %s does not match for TeImage", mInImageName);\r
+      Error (NULL, 0, 3000, "Invalid", "Section-Alignment and File-Alignment of PeImage %s do not match, they must be equal for a TeImage.", mInImageName);\r
       goto Finish;\r
     }\r
     \r
-    DebugMsg (NULL, 0, 9, "TeImage Header Info", "Machine type is %X, Sections number is %X, Stripped size is %X, EntryPoint is %X, BaseOfCode is %X, ImageBase is %X", \r
+    DebugMsg (NULL, 0, 9, "TeImage Header Info", "Machine type is %X, Number of sections is %X, Stripped size is %X, EntryPoint is %X, BaseOfCode is %X, ImageBase is %X", \r
               TEImageHeader.Machine, TEImageHeader.NumberOfSections, TEImageHeader.StrippedSize, TEImageHeader.AddressOfEntryPoint, TEImageHeader.BaseOfCode, TEImageHeader.ImageBase);\r
     //\r
     // Update Image to TeImage\r
@@ -2248,7 +2248,7 @@ Returns:
   // Get time and date that will be set.\r
   //\r
   if (TimeStamp == NULL) {\r
-    Error (NULL, 0, 3000, "Invalid", "TimeData can't be NULL");\r
+    Error (NULL, 0, 3000, "Invalid", "TimeStamp cannot be NULL.");\r
     return EFI_INVALID_PARAMETER;\r
   }\r
   //\r
@@ -2271,7 +2271,7 @@ Returns:
             &stime.tm_min,\r
             &stime.tm_sec\r
             ) != 6) {\r
-      Error (NULL, 0, 3000, "Invalid", "%s Invaild date or time!", TimeStamp);\r
+      Error (NULL, 0, 3000, "Invalid", "%s Invalid date or time!", TimeStamp);\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
@@ -2301,7 +2301,7 @@ Returns:
     //\r
     newtime = mktime (&stime);\r
     if (newtime == (time_t) - 1) {\r
-      Error (NULL, 0, 3000, "Invalid", "%s Invaild date or time!", TimeStamp);\r
+      Error (NULL, 0, 3000, "Invalid", "%s Invalid date or time!", TimeStamp);\r
       return EFI_INVALID_PARAMETER;\r
     }\r
   }\r