diff options
author | Peilin Ye <peilin.ye@bytedance.com> | 2022-05-24 19:51:48 -0700 |
---|---|---|
committer | David Ahern <dsahern@kernel.org> | 2022-05-30 09:53:50 -0600 |
commit | 2d866c6d93dbf4fe8169dda4a3a8f4a67fa0a5b9 (patch) | |
tree | 80d60c494f904b72a6ef7e7e218568fde7ad3146 | |
parent | 21c07b45688fee4e3b690816ecc4d5206d15a135 (diff) | |
download | iproute2-2d866c6d93dbf4fe8169dda4a3a8f4a67fa0a5b9.tar.gz |
ss: Use assignment-suppression character in sscanf()
Use the '*' assignment-suppression character, instead of an
inappropriately named temporary variable.
Signed-off-by: Peilin Ye <peilin.ye@bytedance.com>
Signed-off-by: David Ahern <dsahern@kernel.org>
-rw-r--r-- | misc/ss.c | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -621,9 +621,8 @@ static void user_ent_hash_build(void) char *p; int pid, pos; DIR *dir1; - char crap; - if (sscanf(d->d_name, "%d%c", &pid, &crap) != 1) + if (sscanf(d->d_name, "%d%*c", &pid) != 1) continue; if (getpidcon(pid, &pid_context) != 0) @@ -647,7 +646,7 @@ static void user_ent_hash_build(void) ssize_t link_len; char tmp[1024]; - if (sscanf(d1->d_name, "%d%c", &fd, &crap) != 1) + if (sscanf(d1->d_name, "%d%*c", &fd) != 1) continue; snprintf(name+pos, sizeof(name) - pos, "%d", fd); |