michaelh/zephyr
1
0
Fork 0
Code Releases Activity

subsys/random: fix unused value warning

Browse source 
When using LLVM/Clang, it complains about memcpy() being
casted to (void *):

  warning: expression result unused; should this cast be to 'void'? [-Wunused-value]

So change those to (void) instead as the return of memcpy()
is not used anyway.

Signed-off-by: Daniel Leung <daniel.leung@intel.com>
This commit is contained in:
Daniel Leung 2019-12-17 10:49:29 -08:00 • committed by Anas Nashif
commit 076eee3abc
4 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
subsys/random/rand32_entropy_device.c
View file

@ -80,7 +80,7 @@ static void rand_get(u8_t *dst, size_t outlen)
random_num = k_cycle_get_32(); random_num = k_cycle_get_32();
if ((outlen-len) < sizeof(random_num)) { if ((outlen-len) < sizeof(random_num)) {
blocksize = len; blocksize = len;
(void *)memcpy(&(dst[random_num]), (void)memcpy(&(dst[random_num]),
&random_num, blocksize); &random_num, blocksize);
} else { } else {
*((u32_t *)&dst[len]) = random_num; *((u32_t *)&dst[len]) = random_num;

2
subsys/random/rand32_timer.c
View file

@ -71,7 +71,7 @@ void sys_rand_get(void *dst, size_t outlen)
ret = sys_rand32_get(); ret = sys_rand32_get();
if ((outlen-len) < sizeof(ret)) { if ((outlen-len) < sizeof(ret)) {
blocksize = len; blocksize = len;
(void *)memcpy(udst, &ret, blocksize); (void)memcpy(udst, &ret, blocksize);
} else { } else {
(*udst++) = ret; (*udst++) = ret;
} }

2
subsys/random/rand32_timestamp.c
View file

@ -60,7 +60,7 @@ void sys_rand_get(void *dst, size_t outlen)
ret = sys_rand32_get(); ret = sys_rand32_get();
if ((outlen-len) < sizeof(ret)) { if ((outlen-len) < sizeof(ret)) {
blocksize = len; blocksize = len;
(void *)memcpy(udst, &ret, blocksize); (void)memcpy(udst, &ret, blocksize);
} else { } else {
(*udst++) = ret; (*udst++) = ret;
} }

2
subsys/random/rand32_xoroshiro128.c
View file

@ -107,7 +107,7 @@ void sys_rand_get(void *dst, size_t outlen)
ret = xoroshiro128_next(); ret = xoroshiro128_next();
if ((outlen-len) < sizeof(ret)) { if ((outlen-len) < sizeof(ret)) {
blocksize = len; blocksize = len;
(void *)memcpy(udst, &ret, blocksize); (void)memcpy(udst, &ret, blocksize);
} else { } else {
(*udst++) = ret; (*udst++) = ret;
} }