diff options
author | Denys Vlasenko | 2021-09-09 22:00:44 +0200 |
---|---|---|
committer | Denys Vlasenko | 2021-09-09 22:00:44 +0200 |
commit | 7ab9cd23988b48956fcfe171d5828d61285baf40 (patch) | |
tree | 10b3d522ff45b06ac9dd9f362ddc336c5b058ee0 /networking/nc_bloaty.c | |
parent | 82c5eb8e4681ba345500e5c368fb54741bb0c450 (diff) | |
download | busybox-7ab9cd23988b48956fcfe171d5828d61285baf40.zip busybox-7ab9cd23988b48956fcfe171d5828d61285baf40.tar.gz |
libbb: make bb_lookup_port() abort on bad port names
Also, no need to preserve errno
function old new delta
.rodata 104247 104241 -6
bb_lookup_port 97 83 -14
nc_main 1039 1018 -21
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/3 up/down: 0/-41) Total: -41 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'networking/nc_bloaty.c')
-rw-r--r-- | networking/nc_bloaty.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/networking/nc_bloaty.c b/networking/nc_bloaty.c index 25b9524..cfa133e 100644 --- a/networking/nc_bloaty.c +++ b/networking/nc_bloaty.c @@ -813,8 +813,6 @@ int nc_main(int argc UNUSED_PARAM, char **argv) //if (option_mask32 & OPT_o) /* hexdump log */ if (option_mask32 & OPT_p) { /* local source port */ o_lport = bb_lookup_port(str_p, o_udpmode ? "udp" : "tcp", 0); - if (!o_lport) - bb_error_msg_and_die("bad local port '%s'", str_p); } //if (option_mask32 & OPT_r) /* randomize various things */ //if (option_mask32 & OPT_u) /* use UDP */ @@ -827,9 +825,8 @@ int nc_main(int argc UNUSED_PARAM, char **argv) if (argv[0]) { themaddr = xhost2sockaddr(argv[0], - argv[1] - ? bb_lookup_port(argv[1], o_udpmode ? "udp" : "tcp", 0) - : 0); + bb_lookup_port(argv[1], o_udpmode ? "udp" : "tcp", 0) + ); } /* create & bind network socket */ |