git: 9cb1459633a1 - main - loader.efi: Fix build with gcc due to pointer / int issues on 32-bit build
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Fri, 26 Jun 2026 23:14:33 UTC
The branch main has been updated by imp:
URL: https://cgit.FreeBSD.org/src/commit/?id=9cb1459633a1f96ed2fd5bd37530be3c4f6ae9e6
commit 9cb1459633a1f96ed2fd5bd37530be3c4f6ae9e6
Author: Warner Losh <imp@FreeBSD.org>
AuthorDate: 2026-06-26 23:12:48 +0000
Commit: Warner Losh <imp@FreeBSD.org>
CommitDate: 2026-06-26 23:12:48 +0000
loader.efi: Fix build with gcc due to pointer / int issues on 32-bit build
Use (uintptr_t) casts to cast the EFI_PHYSICAL_ADDDRESS to a pointer.
Fixes: afee781523e4 ("loader.efi: Recognize new memdisk=<url> and memcd=<url> options")
Sponsored by: Netflix
Reviewed by: rlibby
Differential Revision: https://reviews.freebsd.org/D57893
---
stand/efi/loader/decompress.c | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/stand/efi/loader/decompress.c b/stand/efi/loader/decompress.c
index 1f44d1000f11..1e18618db6f2 100644
--- a/stand/efi/loader/decompress.c
+++ b/stand/efi/loader/decompress.c
@@ -80,9 +80,9 @@ alloc_buffer(decomp_state *dctx, size_t size)
printf("Failed to allocate memory for %llu bytes\n", ULL(dctx->alloc_size));
return (status);
}
- BS->SetMem((void *)dctx->buf, dctx->alloc_size, 0);
- dctx->buf_cur = (uint8_t *)dctx->buf;
- dctx->buf_end = (uint8_t *)dctx->buf + dctx->alloc_size;
+ BS->SetMem((void *)(uintptr_t)dctx->buf, dctx->alloc_size, 0);
+ dctx->buf_cur = (uint8_t *)(uintptr_t)dctx->buf;
+ dctx->buf_end = (uint8_t *)(uintptr_t)dctx->buf + dctx->alloc_size;
return (EFI_SUCCESS);
}
@@ -100,12 +100,12 @@ grow_buffer(decomp_state *dctx)
printf("Failed to allocate memory for %llu bytes\n", ULL(newsz));
return (status);
}
- memcpy((void *)newbuf, (void *)dctx->buf, dctx->alloc_size);
+ memcpy((void *)(uintptr_t)newbuf, (void *)(uintptr_t)dctx->buf, dctx->alloc_size);
BS->FreePages(dctx->buf, dctx->pages);
dctx->buf = newbuf;
dctx->pages = newpages;
- dctx->buf_cur = (uint8_t *)dctx->buf + dctx->alloc_size;
- dctx->buf_end = (uint8_t *)dctx->buf + newsz;
+ dctx->buf_cur = (uint8_t *)(uintptr_t)dctx->buf + dctx->alloc_size;
+ dctx->buf_end = (uint8_t *)(uintptr_t)dctx->buf + newsz;
BS->SetMem(dctx->buf_cur, newsz - dctx->alloc_size, 0);
dctx->alloc_size = newsz;
return (EFI_SUCCESS);
@@ -337,7 +337,7 @@ null_step(decomp_state *dctx, uint8_t *buf, size_t len, size_t offset)
}
CHAR8 *src = buf;
- CHAR8 *dst = (void*)(dctx->buf + offset);
+ CHAR8 *dst = (void*)(uintptr_t)(dctx->buf + offset);
BS->CopyMem(dst, src, len);
return (end == dctx->size ? done : ok);
}
@@ -417,5 +417,5 @@ decomp_buffer_length(decomp_state *dctx)
{
if (dctx == NULL)
return (0);
- return ((void *)dctx->buf_cur - (void *)dctx->buf);
+ return ((uintptr_t)dctx->buf_cur - (uintptr_t)dctx->buf);
}