There's no need to check 'pid_context' before calling free(). Signed-off-by: Stefano Brivio <sbri...@redhat.com> --- misc/ss.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/misc/ss.c b/misc/ss.c index fa026eb0934b..fb80d84122fc 100644 --- a/misc/ss.c +++ b/misc/ss.c @@ -3593,12 +3593,8 @@ static int netlink_show_one(struct filter *f, else if (pid > 0) getpidcon(pid, &pid_context); - if (pid_context != NULL) { - printf(" proc_ctx=%s", pid_context); - free(pid_context); - } else { - printf(" proc_ctx=unavailable"); - } + printf(" proc_ctx=%s", pid_context ? : "unavailable"); + free(pid_context); } if (show_details) { -- 2.9.4