[PATCH] arm64: versal2: Fix buffer overflow in soc_name_decode
Francois Berder
fberder at outlook.fr
Thu May 21 15:05:44 CEST 2026
The size of name buffer was not computed correctly.
The suffix format is "--rel.-el" (9 chars instead of 6),
and the longest platform name is "emu-mmd" (7 chars instead of 4).
Fix comment and name size.
Signed-off-by: Francois Berder <fberder at outlook.fr>
---
board/amd/versal2/board.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/board/amd/versal2/board.c b/board/amd/versal2/board.c
index d94c1494d53..81daba1c5ef 100644
--- a/board/amd/versal2/board.c
+++ b/board/amd/versal2/board.c
@@ -81,12 +81,13 @@ char *soc_name_decode(void)
}
/*
- * --rev. are 6 chars
- * max platform name is qemu which is 4 chars
+ * --rev.-el are 9 chars
+ * max platform name is emu-mmd which is 7 chars
* platform version number are 1+1
- * Plus 1 char for \n
+ * el is 1 char
+ * Plus 1 char for NULL byte
*/
- name = calloc(1, strlen(CONFIG_SYS_BOARD) + 13);
+ name = calloc(1, strlen(CONFIG_SYS_BOARD) + 20);
if (!name)
return NULL;
--
2.43.0
More information about the U-Boot
mailing list