Gcc issued the warning when compiling CryptoPkg:
CryptoPkg/Library/Include/CrtLibSupport.h:135:17: warning: type of 'memset' does not match original declaration [-Wlto-type-mismatch]
 void           *memset     (void *, int, size_t);
                 ^
CryptoPkg/Library/IntrinsicLib/MemoryIntrinsics.c:27:8: note: type mismatch in parameter 2
 void * memset (void *dest, char ch, size_t count)
        ^
This commit changes the type of ch from char to int to match the
declaration.
Cc: Qin Long <qin.long@intel.com>
Cc: Ting Ye <ting.ye@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Gary Lin <glin@suse.com>
Reviewed-by: Qin Long <qin.long@intel.com>
		
	
		
			
				
	
	
		
			61 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			61 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
/** @file
 | 
						|
  Intrinsic Memory Routines Wrapper Implementation for OpenSSL-based
 | 
						|
  Cryptographic Library.
 | 
						|
 | 
						|
Copyright (c) 2010 - 2017, Intel Corporation. All rights reserved.<BR>
 | 
						|
This program and the accompanying materials
 | 
						|
are licensed and made available under the terms and conditions of the BSD License
 | 
						|
which accompanies this distribution.  The full text of the license may be found at
 | 
						|
http://opensource.org/licenses/bsd-license.php
 | 
						|
 | 
						|
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
 | 
						|
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 | 
						|
 | 
						|
**/
 | 
						|
 | 
						|
#include <Base.h>
 | 
						|
#include <Library/BaseMemoryLib.h>
 | 
						|
#include <Library/BaseLib.h>
 | 
						|
 | 
						|
typedef UINTN  size_t;
 | 
						|
 | 
						|
/* OpenSSL will use floating point support, and C compiler produces the _fltused
 | 
						|
   symbol by default. Simply define this symbol here to satisfy the linker. */
 | 
						|
int _fltused = 1;
 | 
						|
 | 
						|
/* Sets buffers to a specified character */
 | 
						|
void * memset (void *dest, int ch, size_t count)
 | 
						|
{
 | 
						|
  //
 | 
						|
  // NOTE: Here we use one base implementation for memset, instead of the direct
 | 
						|
  //       optimized SetMem() wrapper. Because the IntrinsicLib has to be built 
 | 
						|
  //       without whole program optimization option, and there will be some 
 | 
						|
  //       potential register usage errors when calling other optimized codes.
 | 
						|
  //
 | 
						|
 | 
						|
  //
 | 
						|
  // Declare the local variables that actually move the data elements as
 | 
						|
  // volatile to prevent the optimizer from replacing this function with
 | 
						|
  // the intrinsic memset()
 | 
						|
  //
 | 
						|
  volatile UINT8  *Pointer;
 | 
						|
 | 
						|
  Pointer = (UINT8 *)dest;
 | 
						|
  while (count-- != 0) {
 | 
						|
    *(Pointer++) = (UINT8)ch;
 | 
						|
  }
 | 
						|
  
 | 
						|
  return dest;
 | 
						|
}
 | 
						|
 | 
						|
/* Compare bytes in two buffers. */
 | 
						|
int memcmp (const void *buf1, const void *buf2, size_t count)
 | 
						|
{
 | 
						|
  return (int)CompareMem(buf1, buf2, count);
 | 
						|
}
 | 
						|
 | 
						|
int strcmp (const char *s1, const char *s2)
 | 
						|
{
 | 
						|
  return (int)AsciiStrCmp(s1, s2);
 | 
						|
}
 |