diff options
| author | Tom Rini <[email protected]> | 2025-12-18 10:24:39 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-12-18 10:24:39 -0600 |
| commit | 8d210fa37f49169c5ad0d46a7bc82aed59f51984 (patch) | |
| tree | 527e246a9105130dcaf29c49fce3ad04f6958fbf | |
| parent | 47b50fb1125c539d80ed3e7f739c02c6486e1b52 (diff) | |
| parent | a8a81af848b88d7ecee64d98a08f6415630305b3 (diff) | |
Merge tag 'net-20251218' of https://source.denx.de/u-boot/custodians/u-boot-net
Pull request net-20251218.
net-lwip:
- Avoid NULL dereference in _set_cacert()
- Fix filename handling in TFTP
| -rw-r--r-- | cmd/lwip/wget.c | 3 | ||||
| -rw-r--r-- | net/lwip/tftp.c | 5 |
2 files changed, 5 insertions, 3 deletions
diff --git a/cmd/lwip/wget.c b/cmd/lwip/wget.c index fc9bc11cd83..4883ad61bce 100644 --- a/cmd/lwip/wget.c +++ b/cmd/lwip/wget.c @@ -180,6 +180,8 @@ int do_wget(struct cmd_tbl *cmdtp, int flag, int argc, char * const argv[]) ulong dst_addr; char nurl[1024]; + wget_info = &default_wget_info; + #if CONFIG_IS_ENABLED(WGET_CACERT) if (argc == 4 && !strncmp(argv[1], "cacert", strlen("cacert"))) return set_cacert(argv[2], argv[3]); @@ -214,7 +216,6 @@ int do_wget(struct cmd_tbl *cmdtp, int flag, int argc, char * const argv[]) if (parse_legacy_arg(url, nurl, sizeof(nurl))) return CMD_RET_FAILURE; - wget_info = &default_wget_info; if (wget_do_request(dst_addr, nurl)) return CMD_RET_FAILURE; diff --git a/net/lwip/tftp.c b/net/lwip/tftp.c index 94bacf63075..6c7ffba661e 100644 --- a/net/lwip/tftp.c +++ b/net/lwip/tftp.c @@ -279,7 +279,7 @@ int do_tftpb(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) if (!arg) arg = net_boot_file_name; - if (arg) { + if (*arg) { /* Parse [ip:[port:]]fname */ i = 0; while ((*(words + i) = strsep(&arg, ":"))) @@ -342,6 +342,7 @@ 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; out: - free(arg); + if (arg != net_boot_file_name) + free(arg); return ret; } |
