michael/u-boot
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

upl: Fix buf array size

Browse Source 
Size of array buf was incorrect due to sizeof returning the
size of an integer (typically 32 bits) instead of a u64 type
(64 bits). Hence, buf array was shorter than expected.

Signed-off-by: Francois Berder <fberder@outlook.fr>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Francois Berder
2025-11-11 11:37:35 +01:00
committed by Tom Rini
parent 74c68b5497
commit 8385ecc8ef
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
boot/upl_write.c
View File

@@ -301,7 +301,7 @@ static int add_upl_memory(const struct upl *upl, ofnode root)
for (i = 0; i < upl->mem.count; i++) {
const struct upl_mem *mem = alist_get(&upl->mem, i,
struct upl_mem);
char buf[mem->region.count * sizeof(64) * 2];
char buf[mem->region.count * sizeof(u64) * 2];
const struct memregion *first;
char name[26];
int ret, len;
@@ -354,7 +354,7 @@ static int add_upl_memmap(const struct upl *upl, ofnode root)
for (i = 0; i < upl->memmap.count; i++) {
const struct upl_memmap *memmap = alist_get(&upl->memmap, i,
struct upl_memmap);
char buf[memmap->region.count * sizeof(64) * 2];
char buf[memmap->region.count * sizeof(u64) * 2];
const struct memregion *first;
char name[26];
int ret, len;
@@ -411,7 +411,7 @@ static int add_upl_memres(const struct upl *upl, ofnode root,
for (i = 0; i < upl->memres.count; i++) {
const struct upl_memres *memres = alist_get(&upl->memres, i,
struct upl_memres);
char buf[memres->region.count * sizeof(64) * 2];
char buf[memres->region.count * sizeof(u64) * 2];
const struct memregion *first;
char name[26];
int ret, len;