[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