[PATCH] arm64: versal2: Fix buffer overflow in soc_name_decode

[Michal Simek]

On 5/21/26 15:05, Francois Berder wrote:
> 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;
>   


Applied with

Fixes: 40f5046c221a ("arm64: versal2: Add support for AMD Versal Gen 2")

Thanks,
Michal