[PATCH 1/3] toradex: tdx-cfg-block: use only snprintf

Marcel Ziswiler marcel.ziswiler at toradex.com
Thu Jun 30 13:55:40 CEST 2022


On Mon, 2022-06-13 at 19:35 +0200, Philippe Schenker wrote:
> From: Philippe Schenker <philippe.schenker at toradex.com>
> 
> Prevent memory issues that could appear with sprintf. Replace all
> sprintf occurences with snprintf.
> 
> Signed-off-by: Philippe Schenker <philippe.schenker at toradex.com>

Acked-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>

> ---
> 
>  board/toradex/common/tdx-common.c | 27 +++++++++++++++------------
>  1 file changed, 15 insertions(+), 12 deletions(-)
> 
> diff --git a/board/toradex/common/tdx-common.c b/board/toradex/common/tdx-common.c
> index 9db4553e0f..2207818447 100644
> --- a/board/toradex/common/tdx-common.c
> +++ b/board/toradex/common/tdx-common.c
> @@ -89,11 +89,13 @@ int show_board_info(void)
>                 tdx_eth_addr.nic = htonl(tdx_serial << 8);
>                 checkboard();
>         } else {
> -               sprintf(tdx_serial_str, "%08u", tdx_serial);
> -               sprintf(tdx_board_rev_str, "V%1d.%1d%c",
> -                       tdx_hw_tag.ver_major,
> -                       tdx_hw_tag.ver_minor,
> -                       (char)tdx_hw_tag.ver_assembly + 'A');
> +               snprintf(tdx_serial_str, sizeof(tdx_serial_str),
> +                        "%08u", tdx_serial);
> +               snprintf(tdx_board_rev_str, sizeof(tdx_board_rev_str),
> +                        "V%1d.%1d%c",
> +                        tdx_hw_tag.ver_major,
> +                        tdx_hw_tag.ver_minor,
> +                        (char)tdx_hw_tag.ver_assembly + 'A');
>  
>                 env_set("serial#", tdx_serial_str);
>  
> @@ -109,12 +111,13 @@ int show_board_info(void)
>                         tdx_carrier_board_name = (char *)
>                                 toradex_carrier_boards[tdx_car_hw_tag.prodid];
>  
> -                       sprintf(tdx_car_serial_str, "%08u", tdx_car_serial);
> -                       sprintf(tdx_car_rev_str, "V%1d.%1d%c",
> -                               tdx_car_hw_tag.ver_major,
> -                               tdx_car_hw_tag.ver_minor,
> -                               (char)tdx_car_hw_tag.ver_assembly +
> -                               'A');
> +                       snprintf(tdx_car_serial_str, sizeof(tdx_car_serial_str),
> +                                "%08u", tdx_car_serial);
> +                       snprintf(tdx_car_rev_str, sizeof(tdx_car_rev_str),
> +                                "V%1d.%1d%c",
> +                                tdx_car_hw_tag.ver_major,
> +                                tdx_car_hw_tag.ver_minor,
> +                                (char)tdx_car_hw_tag.ver_assembly + 'A');
>  
>                         env_set("carrier_serial#", tdx_car_serial_str);
>                         printf("Carrier: Toradex %s %s, Serial# %s\n",
> @@ -170,7 +173,7 @@ int ft_common_board_setup(void *blob, struct bd_info *bd)
>         if (tdx_hw_tag.ver_major) {
>                 char prod_id[5];
>  
> -               sprintf(prod_id, "%04u", tdx_hw_tag.prodid);
> +               snprintf(prod_id, sizeof(prod_id), "%04u", tdx_hw_tag.prodid);
>                 fdt_setprop(blob, 0, "toradex,product-id", prod_id, 5);
>  
>                 fdt_setprop(blob, 0, "toradex,board-rev", tdx_board_rev_str,


More information about the U-Boot mailing list