Date: Fri, 26 Jun 2026 23:14:33 +0000 From: Warner Losh <imp@FreeBSD.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: git: 9cb1459633a1 - main - loader.efi: Fix build with gcc due to pointer / int issues on 32-bit build Message-ID: <6a3f07d9.34350.2219162f@gitrepo.freebsd.org>
index | next in thread | raw e-mail
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); }home | help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6a3f07d9.34350.2219162f>