]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Core/Dxe/Image/ImageFile.c
Make ASSERT statement more meaningful.
[mirror_edk2.git] / MdeModulePkg / Core / Dxe / Image / ImageFile.c
index 4a8d1034a5412ef29dab2e7538ca56328e201174..9c6729a8fdc6071561a599ce7b0ee5ea4e97a9e6 100644 (file)
@@ -13,11 +13,14 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 **/\r
 \r
 #include "DxeMain.h"\r
+#include "Image.h"\r
 \r
 /**\r
   Search a handle to a device on a specified device path that supports a specified protocol,\r
   interface of that protocol on that handle is another output.\r
 \r
+\r
+\r
   @param  Protocol               The protocol to search for\r
   @param  FilePath               The specified device path\r
   @param  Interface              Interface of the protocol on the handle\r
@@ -87,6 +90,7 @@ CoreOpenImageFile (
   EFI_FILE_HANDLE                   FileHandle;\r
   EFI_FILE_HANDLE                   LastHandle;\r
   EFI_LOAD_FILE_PROTOCOL            *LoadFile;\r
+  EFI_LOAD_FILE2_PROTOCOL           *LoadFile2;\r
   EFI_FIRMWARE_VOLUME2_PROTOCOL     *FwVol;\r
   EFI_SECTION_TYPE                  SectionType;\r
   UINT8                             *Pe32Buffer;\r
@@ -261,22 +265,33 @@ CoreOpenImageFile (
         // figure out how big the file is.\r
         //\r
         FileInfo = NULL;\r
-        FileInfoSize = sizeof (EFI_FILE_INFO);\r
-        while (CoreGrowBuffer (&Status, (VOID **)&FileInfo, FileInfoSize)) {\r
-          //\r
-          // Automatically allocate buffer of the correct size and make the call\r
-          //\r
-          Status = FileHandle->GetInfo (\r
-                                FileHandle,\r
-                                &gEfiFileInfoGuid,\r
-                                &FileInfoSize,\r
-                                FileInfo\r
-                                );\r
+        FileInfoSize = 0;\r
+        Status = FileHandle->GetInfo (\r
+                              FileHandle,\r
+                              &gEfiFileInfoGuid,\r
+                              &FileInfoSize,\r
+                              FileInfo\r
+                              );\r
+        if (Status == EFI_BUFFER_TOO_SMALL) {\r
+          FileInfo = AllocatePool (FileInfoSize);\r
+          if (FileInfo != NULL) {\r
+            Status = FileHandle->GetInfo (\r
+                                  FileHandle,\r
+                                  &gEfiFileInfoGuid,\r
+                                  &FileInfoSize,\r
+                                  FileInfo\r
+                                  );\r
+          } else {\r
+            Status = EFI_OUT_OF_RESOURCES;\r
+            goto Done;\r
+          }\r
         }\r
+        \r
         if (!EFI_ERROR (Status)) {\r
           //\r
           // Allocate space for the file\r
           //\r
+          ASSERT (FileInfo != NULL);\r
           ImageFileHandle->Source = AllocatePool ((UINTN)FileInfo->FileSize);\r
           if (ImageFileHandle->Source != NULL) {\r
             //\r
@@ -290,6 +305,7 @@ CoreOpenImageFile (
             // Close the file since we are done\r
             //\r
             FileHandle->Close (FileHandle);\r
+            CoreFreePool (FileInfo);\r
           } else {\r
             Status = EFI_OUT_OF_RESOURCES;\r
           }\r
@@ -300,6 +316,52 @@ CoreOpenImageFile (
     }\r
   }\r
 \r
+  //\r
+  // Try LoadFile2 style\r
+  //\r
+  if (!BootPolicy) {\r
+    TempFilePath = *FilePath;\r
+    Status = CoreDevicePathToInterface (\r
+               &gEfiLoadFile2ProtocolGuid,\r
+               &TempFilePath,\r
+               (VOID*)&LoadFile2,\r
+               DeviceHandle\r
+               );\r
+    if (!EFI_ERROR (Status)) {\r
+      //\r
+      // Call LoadFile2 with the correct buffer size\r
+      //    \r
+      ASSERT (ImageFileHandle->SourceSize == 0);\r
+      ASSERT (ImageFileHandle->Source == NULL);\r
+      \r
+      Status = LoadFile2->LoadFile (\r
+                           LoadFile2,\r
+                           TempFilePath,\r
+                           BootPolicy,\r
+                           &ImageFileHandle->SourceSize,\r
+                           ImageFileHandle->Source\r
+                           );\r
+      if (Status == EFI_BUFFER_TOO_SMALL) {\r
+        ImageFileHandle->Source = AllocatePool (ImageFileHandle->SourceSize);\r
+        if (ImageFileHandle->Source == NULL) {\r
+        Status = EFI_OUT_OF_RESOURCES;\r
+        } else {\r
+          Status = LoadFile2->LoadFile (\r
+                               LoadFile2,\r
+                               TempFilePath,\r
+                               BootPolicy,\r
+                               &ImageFileHandle->SourceSize,\r
+                               ImageFileHandle->Source\r
+                               );\r
+        }\r
+      }\r
+\r
+      if (!EFI_ERROR (Status)) {\r
+        ImageFileHandle->FreeBuffer = TRUE;\r
+        goto Done;\r
+      }\r
+    }\r
+  }\r
 \r
   //\r
   // Try LoadFile style\r
@@ -316,23 +378,31 @@ CoreOpenImageFile (
     //\r
     // Call LoadFile with the correct buffer size\r
     //\r
-    while (CoreGrowBuffer (&Status, (VOID **)&ImageFileHandle->Source, ImageFileHandle->SourceSize)) {\r
-      Status = LoadFile->LoadFile (\r
-                           LoadFile,\r
-                           TempFilePath,\r
-                           BootPolicy,\r
-                           &ImageFileHandle->SourceSize,\r
-                           ImageFileHandle->Source\r
-                           );\r
-      //\r
-      // If success or other error happens, stop loop\r
-      //\r
-      if (Status != EFI_BUFFER_TOO_SMALL) {\r
-        break;\r
+    ASSERT (ImageFileHandle->SourceSize == 0);\r
+    ASSERT (ImageFileHandle->Source == NULL);\r
+    Status = LoadFile->LoadFile (\r
+                         LoadFile,\r
+                         TempFilePath,\r
+                         BootPolicy,\r
+                         &ImageFileHandle->SourceSize,\r
+                         ImageFileHandle->Source\r
+                         );\r
+    if (Status == EFI_BUFFER_TOO_SMALL) {\r
+      ImageFileHandle->Source = AllocatePool (ImageFileHandle->SourceSize);\r
+      if (ImageFileHandle->Source == NULL) {\r
+        Status = EFI_OUT_OF_RESOURCES;\r
+      } else {\r
+        Status = LoadFile->LoadFile (\r
+                             LoadFile,\r
+                             TempFilePath,\r
+                             BootPolicy,\r
+                             &ImageFileHandle->SourceSize,\r
+                             ImageFileHandle->Source\r
+                             );\r
       }\r
     }\r
 \r
-    if (!EFI_ERROR (Status) || Status == EFI_ALREADY_STARTED) {\r
+    if (!EFI_ERROR (Status)) {\r
       ImageFileHandle->FreeBuffer = TRUE;\r
       goto Done;\r
     }\r
@@ -345,11 +415,10 @@ CoreOpenImageFile (
   Status = EFI_NOT_FOUND;\r
 \r
 Done:\r
-\r
   //\r
   // If the file was not accessed, clean up\r
   //\r
-  if (EFI_ERROR (Status) && (Status != EFI_ALREADY_STARTED)) {\r
+  if (EFI_ERROR (Status)) {\r
     if (ImageFileHandle->FreeBuffer) {\r
       //\r
       // Free the source buffer if we allocated it\r
@@ -407,67 +476,3 @@ CoreReadImageFile (
   return EFI_SUCCESS;\r
 }\r
 \r
-/**\r
-  Helper function called as part of the code needed\r
-  to allocate the proper sized buffer for various\r
-  EFI interfaces.\r
-\r
-  @param  Status                 Current status\r
-  @param  Buffer                 Current allocated buffer, or NULL\r
-  @param  BufferSize             Current buffer size needed\r
-\r
-  @retval TRUE                   if the buffer was reallocated and the caller\r
-                                 should try the API again.\r
-  @retval FALSE                  buffer could not be allocated and the caller\r
-                                 should not try the API again.\r
-\r
-**/\r
-BOOLEAN\r
-CoreGrowBuffer (\r
-  IN OUT EFI_STATUS   *Status,\r
-  IN OUT VOID         **Buffer,\r
-  IN UINTN            BufferSize\r
-  )\r
-{\r
-  BOOLEAN         TryAgain;\r
-\r
-  TryAgain = FALSE;\r
-  //\r
-  // If this is an initial request, buffer will be null with a new buffer size\r
-  //\r
-  if (*Buffer == NULL) {\r
-    *Status = EFI_BUFFER_TOO_SMALL;\r
-  }\r
-\r
-  if (BufferSize == 0) {\r
-    return TRUE;\r
-  }\r
-\r
-  //\r
-  // If the status code is "buffer too small", resize the buffer\r
-  //\r
-  if (*Status == EFI_BUFFER_TOO_SMALL) {\r
-    if (*Buffer != NULL) {\r
-      CoreFreePool (*Buffer);\r
-    }\r
-\r
-    *Buffer = AllocatePool (BufferSize);\r
-    if (*Buffer != NULL) {\r
-      TryAgain = TRUE;\r
-    } else {\r
-      *Status = EFI_OUT_OF_RESOURCES;\r
-    }\r
-  }\r
-\r
-  //\r
-  // If there's an error, free the buffer\r
-  //\r
-  if ((!TryAgain) && (EFI_ERROR (*Status)) && (*Buffer != NULL)) {\r
-    CoreFreePool (*Buffer);\r
-    *Buffer = NULL;\r
-  }\r
-\r
-  return TryAgain;\r
-}\r
-\r
-\r