]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
upl: Fix buf array size
authorFrancois Berder <fberder@outlook.fr>
Tue, 11 Nov 2025 10:37:35 +0000 (11:37 +0100)
committerTom Rini <trini@konsulko.com>
Sat, 22 Nov 2025 14:48:13 +0000 (08:48 -0600)
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>
boot/upl_write.c

index 7d637c15ba056832d9647fa4712c57392cb42a40..4185e59becf5baa5b02ee0429860ddbcb79403b5 100644 (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;