Andrew Geissler | e34f896 | 2021-04-15 15:53:51 -0500 | [diff] [blame] | 1 | From f723458eb649a9ad1d5d0e3dd6e801b284680fa7 Mon Sep 17 00:00:00 2001 |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 2 | From: Khem Raj <raj.khem@gmail.com> |
| 3 | Date: Thu, 8 Jun 2017 16:50:59 -0700 |
Andrew Geissler | e34f896 | 2021-04-15 15:53:51 -0500 | [diff] [blame] | 4 | Subject: [PATCH 6/6] use intptr_t to fix pointer-to-int-cast issues |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 5 | |
| 6 | uintptr_t is guaranteed to be wide enough that |
| 7 | converting a void* to uintptr_t and back again |
| 8 | will yield the original pointer value |
| 9 | |
| 10 | Signed-off-by: Khem Raj <raj.khem@gmail.com> |
| 11 | --- |
| 12 | src/lib/src/fwts_acpi_tables.c | 4 ++-- |
| 13 | src/lib/src/fwts_mmap.c | 2 +- |
Andrew Geissler | e34f896 | 2021-04-15 15:53:51 -0500 | [diff] [blame] | 14 | src/lib/src/fwts_smbios.c | 2 +- |
| 15 | 3 files changed, 4 insertions(+), 4 deletions(-) |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 16 | |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 17 | --- a/src/lib/src/fwts_acpi_tables.c |
| 18 | +++ b/src/lib/src/fwts_acpi_tables.c |
Andrew Geissler | 9aee500 | 2022-03-30 16:27:02 +0000 | [diff] [blame^] | 19 | @@ -171,7 +171,7 @@ static fwts_acpi_table_rsdp *fwts_acpi_g |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 20 | return NULL; |
| 21 | } |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 22 | |
| 23 | - if ((mem = fwts_mmap((off_t)addr, sizeof(fwts_acpi_table_rsdp))) == FWTS_MAP_FAILED) |
| 24 | + if ((mem = fwts_mmap((uintptr_t)addr, sizeof(fwts_acpi_table_rsdp))) == FWTS_MAP_FAILED) |
| 25 | return NULL; |
| 26 | |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 27 | if (fwts_safe_memcpy(rsdp, mem, sizeof(fwts_acpi_table_rsdp)) != FWTS_OK) { |
Andrew Geissler | 9aee500 | 2022-03-30 16:27:02 +0000 | [diff] [blame^] | 28 | @@ -484,7 +484,7 @@ static int fwts_acpi_load_tables_from_fi |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 29 | if ((rsdp = fwts_acpi_get_rsdp(fw, rsdp_addr, &rsdp_len)) == NULL) |
| 30 | return FWTS_ERROR; |
| 31 | |
| 32 | - fwts_acpi_add_table("RSDP", rsdp, (uint64_t)(off_t)rsdp_addr, rsdp_len, FWTS_ACPI_TABLE_FROM_FIRMWARE); |
| 33 | + fwts_acpi_add_table("RSDP", rsdp, (uint64_t)(uintptr_t)rsdp_addr, rsdp_len, FWTS_ACPI_TABLE_FROM_FIRMWARE); |
| 34 | |
| 35 | /* Load any tables from XSDT if it's valid */ |
| 36 | if (rsdp->xsdt_address) { |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 37 | --- a/src/lib/src/fwts_mmap.c |
| 38 | +++ b/src/lib/src/fwts_mmap.c |
Andrew Geissler | 9aee500 | 2022-03-30 16:27:02 +0000 | [diff] [blame^] | 39 | @@ -82,7 +82,7 @@ int fwts_munmap(void *mem, const size_t |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 40 | off_t offset; |
| 41 | |
| 42 | page_size = fwts_page_size(); |
| 43 | - offset = ((off_t)(mem)) & (page_size - 1); |
| 44 | + offset = ((uintptr_t)(mem)) & (page_size - 1); |
| 45 | |
| 46 | if (munmap((void *)((uint8_t *)mem - offset), size + offset) < 0) |
| 47 | return FWTS_ERROR; |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 48 | --- a/src/lib/src/fwts_smbios.c |
| 49 | +++ b/src/lib/src/fwts_smbios.c |
Andrew Geissler | 9aee500 | 2022-03-30 16:27:02 +0000 | [diff] [blame^] | 50 | @@ -78,7 +78,7 @@ static void *fwts_smbios_find_entry_uefi |
Andrew Geissler | e34f896 | 2021-04-15 15:53:51 -0500 | [diff] [blame] | 51 | if ((addr = fwts_scan_efi_systab(smbios)) != NULL) { |
Brad Bishop | d7bf8c1 | 2018-02-25 22:55:05 -0500 | [diff] [blame] | 52 | fwts_smbios_entry *mapped_entry; |
| 53 | |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 54 | - if ((mapped_entry = fwts_mmap((off_t)addr, size)) != FWTS_MAP_FAILED) { |
| 55 | + if ((mapped_entry = fwts_mmap((uintptr_t)addr, size)) != FWTS_MAP_FAILED) { |
| 56 | if (fwts_safe_memcpy(entry, mapped_entry, size) == FWTS_OK) { |
| 57 | (void)fwts_munmap(mapped_entry, size); |
| 58 | *type = FWTS_SMBIOS; |