Correct Error usage (FileName, LineNumber, MessageCode, Text, MsgFmt) to print error...
authorlgao4 <lgao4@7335b38e-4728-0410-8992-fb3ffe349368>
Sat, 21 Jun 2008 09:28:46 +0000 (09:28 +0000)
committerlgao4 <lgao4@7335b38e-4728-0410-8992-fb3ffe349368>
Sat, 21 Jun 2008 09:28:46 +0000 (09:28 +0000)
git-svn-id: https://buildtools.tianocore.org/svn/buildtools/trunk/BaseTools@1269 7335b38e-4728-0410-8992-fb3ffe349368

Source/C/EfiRom/EfiRom.c

index 0ea299b..ddcab04 100644 (file)
@@ -54,23 +54,12 @@ Returns:
   UINT32    TotalSize;\r
   UINT32    Size;\r
   CHAR8     *Ptr0;\r
-  FILE_LIST *OutFileList;\r
-\r
 \r
   SetUtilityName(UTILITY_NAME);\r
 \r
   Status  = STATUS_SUCCESS;\r
   FptrOut = NULL;\r
 \r
-  //\r
-  // Create a new out file structure\r
-  //\r
-  OutFileList = (FILE_LIST *) malloc (sizeof (FILE_LIST));\r
-  if (OutFileList == NULL) {\r
-    Error (stdout, 0, 4001, "Resource", "memory cannot be allocated!");\r
-    return STATUS_ERROR;\r
-  }  \r
-  memset ((char *) OutFileList, 0, sizeof (FILE_LIST));\r
   //\r
   // Parse the command line arguments\r
   //\r
@@ -88,21 +77,27 @@ Returns:
   \r
   if (mOptions.Verbose) {\r
     VerboseMsg("%s tool start.\n", UTILITY_NAME);\r
-    }  \r
+  }\r
+  \r
+  if (mOptions.FileList == NULL) {\r
+    Error (NULL, 0, 1002, "No input file", NULL);\r
+    goto BailOut;\r
+  }\r
   //\r
   // If dumping an image, then do that and quit\r
   //\r
-//  if (mOptions.DumpOption) {\r
-//    for (FList = mOptions.FileList; FList != NULL; FList = FList->Next) {\r
-//    if ((Ptr0 = strstr ((CONST CHAR8 *)FList->FileName, DEFAULT_OUTPUT_EXTENSION)) != NULL) {\r
-//    DumpImage (mOptions.FileList);\r
-//    goto BailOut;\r
-//    }\r
-//    }\r
-//    else\r
-//    printf("\n *.rom file has not been generated, so -p option should be used //after the *.rom Option Rom binary generated!");\r
-//    goto BailOut;\r
-//  }\r
+  if (mOptions.DumpOption) {\r
+    for (FList = mOptions.FileList; FList != NULL; FList = FList->Next) {\r
+      if ((Ptr0 = strstr ((CONST CHAR8 *)FList->FileName, DEFAULT_OUTPUT_EXTENSION)) != NULL) {\r
+        DumpImage (mOptions.FileList);\r
+        goto BailOut;\r
+      } else {\r
+        Error (NULL, 0, 1002, "No PciRom input file", "No *.rom input file");\r
+        //printf("\n *.rom file has not been generated, so -d option should be used after the *.rom Option Rom binary generated!");\r
+        goto BailOut;\r
+      }\r
+    }\r
+  }\r
   //\r
   // Determine the output filename. Either what they specified on\r
   // the command line, or the first input filename with a different extension.\r
@@ -133,7 +128,7 @@ Returns:
   for (FList = mOptions.FileList; FList != NULL; FList = FList->Next) {\r
     if (stricmp (mOptions.OutFileName, FList->FileName) == 0) {\r
       Status = STATUS_ERROR;\r
-      Error (NULL, 0, 1002, "Input and output file names must be different - %s = %s.", FList->FileName, mOptions.OutFileName);\r
+      Error (NULL, 0, 1002, "Invalid input paramter", "Input and output file names must be different - %s = %s.", FList->FileName, mOptions.OutFileName);\r
       goto BailOut;\r
     }\r
   }\r
@@ -181,29 +176,25 @@ Returns:
   // Check total size\r
   //\r
   if (TotalSize > MAX_OPTION_ROM_SIZE) {\r
-    Error (NULL, 0, 2000, "Option ROM image size exceeds limit of 0x%X bytes.", MAX_OPTION_ROM_SIZE);\r
+    Error (NULL, 0, 2000, "Invalid paramter", "Option ROM image size exceeds limit of 0x%X bytes.", MAX_OPTION_ROM_SIZE);\r
     Status = STATUS_ERROR;\r
   }\r
 \r
 BailOut:\r
   if (Status == STATUS_SUCCESS) {\r
-  if (mOptions.DumpOption) {\r
-    OutFileList->FileName = mOptions.OutFileName;\r
-    DumpImage(OutFileList);\r
-  }\r
-  } else {\r
-  if (FptrOut != NULL) {\r
-    fclose (FptrOut);\r
-  }\r
-  //\r
-  // Clean up our file list\r
-  //\r
-  while (mOptions.FileList != NULL) {\r
-    FList = mOptions.FileList->Next;\r
-    free (mOptions.FileList);\r
-    mOptions.FileList = FList;\r
-  }\r
+    if (FptrOut != NULL) {\r
+      fclose (FptrOut);\r
+    }\r
+    //\r
+    // Clean up our file list\r
+    //\r
+    while (mOptions.FileList != NULL) {\r
+      FList = mOptions.FileList->Next;\r
+      free (mOptions.FileList);\r
+      mOptions.FileList = FList;\r
+    }\r
   }\r
+\r
   if (mOptions.Verbose) {\r
     VerboseMsg("%s tool done with return code is 0x%x.\n", UTILITY_NAME, GetUtilityStatus ());\r
   }\r
@@ -290,7 +281,7 @@ Returns:
   }\r
 \r
   if (TotalSize > MAX_OPTION_ROM_SIZE) {\r
-    Error (NULL, 0, 3001, "Option ROM image %s size exceeds limit of 0x%X bytes.", InFile->FileName, MAX_OPTION_ROM_SIZE);\r
+    Error (NULL, 0, 3001, "Invalid", "Option ROM image %s size exceeds limit of 0x%X bytes.", InFile->FileName, MAX_OPTION_ROM_SIZE);\r
     Status = STATUS_ERROR;\r
     goto BailOut;\r
   }\r
@@ -399,7 +390,7 @@ BailOut:
   // Print the file name if errors occurred\r
   //\r
   if (Status != STATUS_SUCCESS) {\r
-    Error (NULL, 0, 0003, "Error parsing file: %s", InFile->FileName);\r
+    Error (NULL, 0, 0003, "Error", "Error parsing file: %s", InFile->FileName);\r
   }\r
 \r
   return Status;\r
@@ -454,7 +445,7 @@ Returns:
   // Try to open the input file\r
   //\r
   if ((InFptr = fopen (InFile->FileName, "rb")) == NULL) {\r
-    Error (NULL, 0, 0001, "Error opening file: %s", InFile->FileName);\r
+    Error (NULL, 0, 0001, "Open file error", "Error opening file: %s", InFile->FileName);\r
     return STATUS_ERROR;\r
   }\r
   //\r
@@ -554,7 +545,7 @@ Returns:
   // Check size\r
   //\r
   if (TotalSize > MAX_OPTION_ROM_SIZE) {\r
-    Error (NULL, 0, 2000, "Option ROM image %s size exceeds limit of 0x%X bytes.", InFile->FileName, MAX_OPTION_ROM_SIZE);     \r
+    Error (NULL, 0, 2000, "Invalid", "Option ROM image %s size exceeds limit of 0x%X bytes.", InFile->FileName, MAX_OPTION_ROM_SIZE);  \r
     Status = STATUS_ERROR;\r
     goto BailOut;\r
   }\r
@@ -725,7 +716,7 @@ BailOut:
   // Print the file name if errors occurred\r
   //\r
   if (Status != STATUS_SUCCESS) {\r
-    Error (NULL, 0, 0003, "Error parsing file: %s", InFile->FileName);\r
+    Error (NULL, 0, 0003, "Error parsing", "Error parsing file: %s", InFile->FileName);\r
   }\r
 \r
   return Status;\r
@@ -1195,7 +1186,7 @@ Returns:
   //\r
   // Summary usage\r
   //\r
-  fprintf (stdout, "Usage: %s [options] <-e input_file>|<-b input_file> \n\n", UTILITY_NAME);\r
+  fprintf (stdout, "Usage: %s [options] [file name<s>] \n\n", UTILITY_NAME);\r
   \r
   //\r
   // Copyright declaration\r