diff options
| author | Mark Marshall | 2018-03-19 10:45:40 +0100 |
|---|---|---|
| committer | Denys Vlasenko | 2018-03-27 23:28:53 +0200 |
| commit | 98592db3231caa905e6d54129c621f3e8e9bbaa3 (patch) | |
| tree | b8f135f0b8abc6a7932f9a05b68eb0f71682a215 | |
| parent | b5820d9fd88bb6cb1dfd1a934a3d346336d5cac2 (diff) | |
| download | busybox-98592db3231caa905e6d54129c621f3e8e9bbaa3.zip busybox-98592db3231caa905e6d54129c621f3e8e9bbaa3.tar.gz | |
netstat: produce numeric-ip output for non-resolved names
If we don't ask for numerical output, and the symbolic look-up failed
we used to get "(null)", but the numeric output would be better.
function old new delta
ip_port_str 109 121 +12
Signed-off-by: Mark Marshall <mark.marshall@omicronenergy.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
| -rw-r--r-- | networking/netstat.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/networking/netstat.c b/networking/netstat.c index 33281e3..f6bcd44 100644 --- a/networking/netstat.c +++ b/networking/netstat.c @@ -397,8 +397,11 @@ static char *ip_port_str(struct sockaddr *addr, int port, const char *proto, int /* Code which used "*" for INADDR_ANY is removed: it's ambiguous * in IPv6, while "0.0.0.0" is not. */ - host = numeric ? xmalloc_sockaddr2dotted_noport(addr) - : xmalloc_sockaddr2host_noport(addr); + host = NULL; + if (!numeric) + host = xmalloc_sockaddr2host_noport(addr); + if (!host) + host = xmalloc_sockaddr2dotted_noport(addr); host_port = xasprintf("%s:%s", host, get_sname(htons(port), proto, numeric)); free(host); |