ACK: [PATCH] lib: fwts_smbios: fix portability warning from cppcheck on void * cast
ivanhu
ivan.hu at canonical.com
Thu Aug 18 02:23:51 UTC 2016
On 2016年08月17日 18:24, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Minor fix, ensure we don't do void * pointer arithmetic as the behaviour
> is undefined (even though most compilers assume it is a char * sized).
> Cast to char * before doing the addition.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
> src/lib/src/fwts_smbios.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/src/lib/src/fwts_smbios.c b/src/lib/src/fwts_smbios.c
> index 73b3f2b..fb814b2 100644
> --- a/src/lib/src/fwts_smbios.c
> +++ b/src/lib/src/fwts_smbios.c
> @@ -124,8 +124,8 @@ static void *fwts_smbios_find_entry_bios(fwts_framework *fw, fwts_smbios_entry *
> (*(mem+i+2) == 'M') &&
> (*(mem+i+3) == '_') &&
> (fwts_checksum(mem + i, 16) == 0)) {
> - addr = (void*)FWTS_SMBIOS_REGION_START + i;
> - memcpy(entry, (void*)(mem + i), sizeof(fwts_smbios_entry));
> + addr = (void *)((uint8_t *)FWTS_SMBIOS_REGION_START + i);
> + memcpy(entry, (void *)(mem + i), sizeof(fwts_smbios_entry));
> *type = FWTS_SMBIOS;
> break;
> }
> @@ -137,8 +137,8 @@ static void *fwts_smbios_find_entry_bios(fwts_framework *fw, fwts_smbios_entry *
> (*(mem+i+4) == '_') &&
> (fwts_checksum(mem + i, 15) == 0)) {
> memset(entry, 0, 16);
> - addr = (void*)FWTS_SMBIOS_REGION_START + i;
> - memcpy(16+((void*)entry), (void*)(mem + i), 15);
> + addr = (void *)((uint8_t *)FWTS_SMBIOS_REGION_START + i);
> + memcpy(16 + ((uint8_t *)entry), (void *)(mem + i), 15);
> *type = FWTS_SMBIOS_DMI_LEGACY;
> break;
> }
> @@ -172,7 +172,7 @@ static void *fwts_smbios30_find_entry_bios(fwts_framework *fw, fwts_smbios30_ent
> (*(mem+i+3) == '3') &&
> (*(mem+i+4) == '_') &&
> (fwts_checksum(mem + i, 24 ) == 0)) {
> - addr = (void *)FWTS_SMBIOS_REGION_START + i;
> + addr = (void *)((uint8_t *)FWTS_SMBIOS_REGION_START + i);
> memcpy(entry, (void*)(mem + i), sizeof(fwts_smbios30_entry));
> break;
> }
>
Acked-by: Ivan Hu <ivan.hu at canonical.com>
More information about the fwts-devel
mailing list