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, 18 insertions, 20 deletions

diff --git a/MdePkg/Library/PeiMemoryLib/ScanMem32Wrapper.c b/MdePkg/Library/PeiMemoryLib/ScanMem32Wrapper.c
index 67eb84d8ef..8de55945ce 100644
--- a/MdePkg/Library/PeiMemoryLib/ScanMem32Wrapper.c
+++ b/MdePkg/Library/PeiMemoryLib/ScanMem32Wrapper.c
@@ -26,25 +26,22 @@
 #include "MemLibInternals.h"

 

 /**

-  Scans a target buffer for a 32-bit value, and returns a pointer to the

-  matching 32-bit value in the target buffer.

+  Scans a target buffer for a 32-bit value, and returns a pointer to the matching 32-bit value

+  in the target buffer.

 

-  This function searches target the buffer specified by Buffer and Length from

-  the lowest address to the highest address at 32-bit increments for a 32-bit

-  value that matches Value. If a match is found, then a pointer to the matching

-  value in the target buffer is returned. If no match is found, then NULL is

-  returned. If Length is 0, then NULL is returned.

-

-  If Buffer is NULL, then ASSERT().

+  This function searches target the buffer specified by Buffer and Length from the lowest

+  address to the highest address for a 32-bit value that matches Value.  If a match is found,

+  then a pointer to the matching byte in the target buffer is returned.  If no match is found,

+  then NULL is returned.  If Length is 0, then NULL is returned.

+  If Length > 0 and Buffer is NULL, then ASSERT().

   If Buffer is not aligned on a 32-bit boundary, then ASSERT().

-  If Length is greater than (MAX_ADDRESS - Buffer + 1), then ASSERT().

+  If Length is greater than (MAX_ADDRESS � Buffer + 1), then ASSERT(). 

 

-  @param  Buffer  Pointer to the target buffer to scan.

-  @param  Length  Number of bytes in Buffer to scan.

-  @param  Value   Value to search for in the target buffer.

+  @param  Buffer      Pointer to the target buffer to scan.

+  @param  Length      Number of bytes in Buffer to scan.

+  @param  Value       Value to search for in the target buffer.

 

-  @return Pointer to the first occurrence or NULL if not found.

-  @retval NULL  if Length == 0 or Value was not found.

+  @return A pointer to the matching byte in the target buffer or NULL otherwise.

 

 **/

 VOID *

@@ -55,12 +52,13 @@ ScanMem32 (
   IN      UINT32                    Value

   )

 {

+  if (Length == 0) {

+    return NULL;

+  }

+

   ASSERT (Buffer != NULL);

   ASSERT (((UINTN)Buffer & (sizeof (Value) - 1)) == 0);

-  ASSERT (Length <= MAX_ADDRESS + (UINTN)Buffer + 1);

+  ASSERT ((Length - 1) <= (MAX_ADDRESS - (UINTN)Buffer));

 

-  if ((Length /= sizeof (Value)) == 0) {

-    return NULL;

-  }

-  return (VOID*)InternalMemScanMem32 (Buffer, Length, Value);

+  return (VOID*)InternalMemScanMem32 (Buffer, Length / sizeof (Value), Value);

 }