[PATCH 5/5] lib: fwts_multiproc: avoid using void * pointer arithmetic
Colin King
colin.king at canonical.com
Thu Sep 11 14:05:29 UTC 2014
From: Colin Ian King <colin.king at canonical.com>
cppcheck reports:
When using void pointers in calculations, the behaviour is undefined
Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
src/lib/src/fwts_multiproc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/lib/src/fwts_multiproc.c b/src/lib/src/fwts_multiproc.c
index 2d17948..69084e9 100644
--- a/src/lib/src/fwts_multiproc.c
+++ b/src/lib/src/fwts_multiproc.c
@@ -78,7 +78,7 @@ static int fwts_mp_get_address(uint32_t *phys_addr)
if ((mem = fwts_mmap(start, size)) == FWTS_MAP_FAILED)
continue;
- for (ptr = mem; ptr < (uint8_t*)(mem + size); ptr+=16) {
+ for (ptr = mem; ptr < (uint8_t*)mem + size; ptr+=16) {
if ((*ptr == '_') &&
(*(ptr + 1) == 'M') &&
(*(ptr + 2) == 'P') &&
@@ -176,7 +176,7 @@ int fwts_mp_data_get(fwts_mp_data *data)
tmp += sizeof(fwts_mp_compat_bus_address_space_entry);
break;
default:
- tmp = mem + data->size; /* Force abort */
+ tmp = (uint8_t *)mem + data->size; /* Force abort */
break;
}
}
--
2.1.0
More information about the fwts-devel
mailing list