[PATCH] net: lwip: tftp: update image_load_addr after successful transfer

Jerome Forissier jerome.forissier at arm.com
Mon Mar 30 16:33:53 CEST 2026



On 26/03/2026 11:41, Pranav Sanwal wrote:
> do_tftpb() parses the load address into a local variable laddr but
> never updates the global image_load_addr. Commands that rely on
> image_load_addr as their default address (e.g. 'bmp info')
> therefore operate on the wrong address when called without
> an explicit argument after tftpboot.
> 
> Update image_load_addr to laddr only on a successful transfer, so
> that it accurately reflects where data was actually loaded.
> 
> Fixes: 4d4d7838127e ("net: lwip: add TFTP support and tftpboot command")
> Signed-off-by: Pranav Sanwal <pranav.sanwal at amd.com>
> ---
>  net/lwip/tftp.c | 2 ++
>  1 file changed, 2 insertions(+)

Reviewed-by: Jerome Forissier <jerome.forissier at arm.com>

Added to net-next.

Thanks,
-- 
Jerome

> 
> diff --git a/net/lwip/tftp.c b/net/lwip/tftp.c
> index 5c3becc68c6..7f3b28b8507 100644
> --- a/net/lwip/tftp.c
> +++ b/net/lwip/tftp.c
> @@ -368,6 +368,8 @@ int do_tftpb(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
>  
>  	if (tftp_loop(eth_get_dev(), laddr, fname, srvip, port) < 0)
>  		ret = CMD_RET_FAILURE;
> +	else
> +		image_load_addr = laddr;
>  out:
>  	if (arg != net_boot_file_name)
>  		free(arg);



More information about the U-Boot mailing list