1. UINTN & INTN issue for EBC architecture:

	The MAX_BIT of EBC will no longer be fixed to bit 63. It is defined as (1ULL << (sizeof (INTN) * 8 - 1)).
	Make EdkModulePkg & MdePkg EBC compiler clean: treat all EFI_STATUS error code as variable.
2. PrintLib
	Complete all missing ASSERT()s.
	Fix “\n” & “%\n” issue thanks to the clarification of MWG 0.56d.
	Adjust StatusString array to support EBC build.
3. BaseMemoryLib
	Adjust ASSERT () & function header of ComparaMem, SetMemXX, ScanMemXX to synchronize with MWG 0.56d.
4.SmbusLib
	Change Pec bit to bit 22 SmBusAddress to synchronize MWG 0.56d.
	Add ASSERT()s to check if length is illegal for SmBusBlockWrite() & SmBusProcessBlock() since it is 6 bit now.
5. PerformanceLib
	Rename “EdkDxePerformanceLib” & “EdkPeiPerformanceLib” to “DxePerformanceLib” & “PeiPerformanceLib” respectively.
	Synchronize the function header of GetPerformanceMeasurement() with MWG 0.56d.
6. BasePeCoffLoaderLib.
	Make PeCoffLoaderLoadImage () Assert() if ImageContext is NULL>
	Make PeCoffLoaderLoadImage () return RETURN_INVALID_PARAMETER if the ImageAddress in ImageContext is 0.
	Adjust some coding style.


git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@593 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 22 insertions, 29 deletions

diff --git a/MdePkg/Library/BasePeCoffLib/BasePeCoff.c b/MdePkg/Library/BasePeCoffLib/BasePeCoff.c
index e91e9a6c2f..32a70c00c4 100644
--- a/MdePkg/Library/BasePeCoffLib/BasePeCoff.c
+++ b/MdePkg/Library/BasePeCoffLib/BasePeCoff.c
@@ -14,32 +14,17 @@
 

 **/

 

+/**

+  Performs an Itanium-based specific relocation fixup.

 

+  @param  Reloc       Pointer to the relocation record.

+  @param  Fixup       Pointer to the address to fix up.

+  @param  FixupData   Pointer to a buffer to log the fixups.

+  @param  Adjust      The offset to adjust the fixup.

 

+  @return Status code.

 

-STATIC

-RETURN_STATUS

-PeCoffLoaderGetPeHeader (

-  IN OUT PE_COFF_LOADER_IMAGE_CONTEXT  *ImageContext,

-  OUT    EFI_IMAGE_NT_HEADERS          *PeHdr,

-  OUT    EFI_TE_IMAGE_HEADER           *TeHdr

-  );

-

-STATIC

-RETURN_STATUS

-PeCoffLoaderCheckImageType (

-  IN OUT PE_COFF_LOADER_IMAGE_CONTEXT  *ImageContext,

-  IN     EFI_IMAGE_NT_HEADERS          *PeHdr,

-  IN     EFI_TE_IMAGE_HEADER           *TeHdr

-  );

-

-STATIC

-VOID *

-PeCoffLoaderImageAddress (

-  IN OUT PE_COFF_LOADER_IMAGE_CONTEXT  *ImageContext,

-  IN     UINTN                         Address

-  );

-

+**/

 RETURN_STATUS

 PeCoffLoaderRelocateImageEx (

   IN UINT16      *Reloc,

@@ -193,11 +178,9 @@ PeCoffLoaderCheckImageType (
   @param  ImageContext              Pointer to the image context structure that describes the PE/COFF

                                     image that needs to be examined by this function.

 

-  @retval  RETURN_SUCCESS           The information on the PE/COFF image was collected.

-  @retval  RETURN_INVALID_PARAMETER ImageContext is NULL.

-  @retval  RETURN_UNSUPPORTED       The PE/COFF image is not supported.

-  @retval  Others                   The error status from reading the PE/COFF image

-                                    using the ImageContext->ImageRead() function.

+  @retval RETURN_SUCCESS            The information on the PE/COFF image was collected.

+  @retval RETURN_INVALID_PARAMETER  ImageContext is NULL.

+  @retval RETURN_UNSUPPORTED        The PE/COFF image is not supported.

 

 **/

 RETURN_STATUS

@@ -687,6 +670,7 @@ PeCoffLoaderRelocateImage (
   specified by the ImageAddress and ImageSize fields of ImageContext.  The caller must allocate

   the load buffer and fill in the ImageAddress and ImageSize fields prior to calling this function.

   The EntryPoint, FixupDataSize, CodeView, and PdbPointer fields of ImageContext are computed.

+  If ImageContext is NULL, then ASSERT().

 

   @param  ImageContext              Pointer to the image context structure that describes the PE/COFF

                                     image that is being loaded.

@@ -724,8 +708,11 @@ PeCoffLoaderLoadImage (
   UINTN                                 Size;

   UINT32                                TempDebugEntryRva;

 

+  ASSERT (ImageContext != NULL);

+

   PeHdr = NULL;

   TeHdr = NULL;

+

   //

   // Assume success

   //

@@ -750,7 +737,13 @@ PeCoffLoaderLoadImage (
     ImageContext->ImageError = IMAGE_ERROR_INVALID_IMAGE_SIZE;

     return RETURN_BUFFER_TOO_SMALL;

   }

-

+  if (ImageContext->ImageAddress == 0) {

+    //

+    // Image cannot be loaded into 0 address.

+    //

+    ImageContext->ImageError = IMAGE_ERROR_INVALID_IMAGE_ADDRESS;

+    return RETURN_INVALID_PARAMETER;

+  }

   //

   // If there's no relocations, then make sure it's not a runtime driver,

   // and that it's being loaded at the linked address.