diff options
author | Stefano Brivio <sbrivio@redhat.com> | 2017-10-31 18:47:54 +0100 |
---|---|---|
committer | Stephen Hemminger <stephen@networkplumber.org> | 2017-11-01 22:10:52 +0100 |
commit | 38509fa903d66cf13f3c8ca5010db820fb874064 (patch) | |
tree | cfcfd5d16951f0013608c585eafde21939ae9a4b | |
parent | acbe9118ce8086f765ffb0da15f80c7c01a8903a (diff) | |
download | iproute2-38509fa903d66cf13f3c8ca5010db820fb874064.tar.gz |
ss: Remove useless width specifier in process context print
Both local address and service, and remote address and service
fields are already printed out in netlink_show_one() before we
start printing process context, by calling sock_addr_print()
twice.
At this point, sock_addr_print() has already forced the remote
service field to be 'serv_width' wide -- that is, 'serv_width'
width has already been consumed, before we print process
context.
Hence, it makes no sense to force the display width of process
context to be 'serv_width' wide again: previous prints have
filled up the line already. Remove the width specifier and
prefix with a space instead, to keep this consistent with fields
which are displayed after the first output line.
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
-rw-r--r-- | misc/ss.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -3594,10 +3594,10 @@ static int netlink_show_one(struct filter *f, getpidcon(pid, &pid_context); if (pid_context != NULL) { - printf("proc_ctx=%-*s ", serv_width, pid_context); + printf(" proc_ctx=%s", pid_context); free(pid_context); } else { - printf("proc_ctx=%-*s ", serv_width, "unavailable"); + printf(" proc_ctx=unavailable"); } } |