aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefano Brivio <sbrivio@redhat.com>2017-10-31 18:47:54 +0100
committerStephen Hemminger <stephen@networkplumber.org>2017-11-01 22:10:52 +0100
commit38509fa903d66cf13f3c8ca5010db820fb874064 (patch)
treecfcfd5d16951f0013608c585eafde21939ae9a4b
parentacbe9118ce8086f765ffb0da15f80c7c01a8903a (diff)
downloadiproute2-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.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/misc/ss.c b/misc/ss.c
index dfb438f9..fa026eb0 100644
--- a/misc/ss.c
+++ b/misc/ss.c
@@ -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");
}
}