size_t is an unsigned type and as such is a bad choice for a counting down loop counter. BRANCH=all BUG=none TEST=editing cli command line does not cause hangs any more Change-Id: I0502553b5e2143052345edeb205a01558fccd9b8 Signed-off-by: Patrick Georgi <pgeorgi@chromium.org> Original-Commit-Id: 1c171f739497fcd26589976676ab94b23cd7ee8b Original-Change-Id: I4aa38379ac356114fc91a32cced2fa45a00a09d6 Original-Signed-off-by: Vadim Bendebury <vbendeb@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/262714 Original-Reviewed-by: Ionela Voinescu <ionela.voinescu@imgtec.com> Original-Tested-by: Ionela Voinescu <ionela.voinescu@imgtec.com> Original-Reviewed-by: David Hendricks <dhendrix@chromium.org> Reviewed-on: http://review.coreboot.org/9891 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
		
			
				
	
	
		
			82 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			82 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
|  /*
 | |
|  * This file is part of the libpayload project.
 | |
|  *
 | |
|  * Copyright (C) 2014 Imagination Technologies
 | |
|  *
 | |
|  * This program is free software; you can redistribute it and/or modify
 | |
|  * it under the terms of the GNU General Public License as published by
 | |
|  * the Free Software Foundation; version 2 of the License.
 | |
|  *
 | |
|  * This program is distributed in the hope that it will be useful,
 | |
|  * but WITHOUT ANY WARRANTY; without even the implied warranty of
 | |
|  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 | |
|  * GNU General Public License for more details.
 | |
|  *
 | |
|  * You should have received a copy of the GNU General Public License
 | |
|  * along with this program; if not, write to the Free Software
 | |
|  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
 | |
|  */
 | |
| 
 | |
| #include <stdint.h>
 | |
| #include "string.h"
 | |
| 
 | |
| /*
 | |
|  * Alternative string functions to the default ones are added
 | |
|  * because there is no guarantee that the provided source and
 | |
|  * destination addresses are properly aligned;
 | |
|  * The default string functions work with multiple of 4 bytes
 | |
|  * (sizeof(unsinged long)); MIPS will use LW/SW instructions
 | |
|  * for these operations and if the source and destination
 | |
|  * addresses are not aligned it will trigger an exception.
 | |
|  */
 | |
| 
 | |
| void *memcpy(void *dest, const void *src, size_t n)
 | |
| {
 | |
| 	u8 *ptr_d = dest;
 | |
| 	const u8 *ptr_s = src;
 | |
| 	size_t i;
 | |
| 
 | |
| 	for (i = 0; i < n; i++)
 | |
| 		*ptr_d++ = *ptr_s++;
 | |
| 
 | |
| 	return dest;
 | |
| }
 | |
| 
 | |
| void *memmove(void *dest, const void *src, size_t n)
 | |
| {
 | |
| 	if ((src < dest) && (dest - src < n)) {
 | |
| 		u8 *ptr_d = dest;
 | |
| 		const u8 *ptr_s = src;
 | |
| 
 | |
| 		/* copy backwards */
 | |
| 		while (n--)
 | |
| 			ptr_d[n] = ptr_s[n];
 | |
| 
 | |
| 		return dest;
 | |
| 	}
 | |
| 
 | |
| 	/* copy forwards */
 | |
| 	return memcpy(dest, src, n);
 | |
| }
 | |
| 
 | |
| void *memset(void *s, int c, size_t n)
 | |
| {
 | |
| 	u8 *ptr = s;
 | |
| 	size_t i;
 | |
| 
 | |
| 	for (i = 0; i < n; i++)
 | |
| 		*ptr++ = c;
 | |
| 
 | |
| 	return s;
 | |
| }
 | |
| 
 | |
| int memcmp(const void *s1, const void *s2, size_t n)
 | |
| {
 | |
| 	size_t i;
 | |
| 
 | |
| 	for (i = 0; i < n; i++)
 | |
| 		if (((u8 *)s1)[i] != ((u8 *)s2)[i])
 | |
| 			return ((u8 *)s1)[i] - ((u8 *)s2)[i];
 | |
| 	return 0;
 | |
| }
 |