summaryrefslogtreecommitdiff
path: root/src/id.c
diff options
context:
space:
mode:
authorBernhard Voelker <mail@bernhard-voelker.de>2016-07-27 09:33:19 +0200
committerBernhard Voelker <mail@bernhard-voelker.de>2016-07-27 21:28:49 +0200
commit558ce3044d5e22890b272b4e90f3ac0b236726a0 (patch)
tree904f9bb9fbe50eede508504e4c1b3e70c5b93e27 /src/id.c
parentf0e0f3ac924746b9436f8f54da467162b7c78ee3 (diff)
downloadcoreutils-558ce3044d5e22890b272b4e90f3ac0b236726a0.tar.xz
maint: pacify GCC 6 with -Wnull-dereference
src/id.c:249:29: error: potential null pointer dereference \ [-Werror=null-dereference] pw_name = xstrdup (pwd->pw_name); ~~~^~~~~~~~~ src/whoami.c:89:11: error: potential null pointer dereference \ [-Werror=null-dereference] puts (pw->pw_name); ~~^~~~~~~~~ * src/id.c (main): Explicitly exit with EXIT_FAILURE after an eror to help gcc-6 to detect that the dereferenced pointer is valid. * src/whoami.c (main): Likewise.
Diffstat (limited to 'src/id.c')
-rw-r--r--src/id.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/src/id.c b/src/id.c
index 218ee5a31..35cbeb564 100644
--- a/src/id.c
+++ b/src/id.c
@@ -245,7 +245,10 @@ main (int argc, char **argv)
}
}
if (pwd == NULL)
- error (EXIT_FAILURE, 0, _("%s: no such user"), quote (spec));
+ {
+ error (0, 0, _("%s: no such user"), quote (spec));
+ exit (EXIT_FAILURE);
+ }
pw_name = xstrdup (pwd->pw_name);
ruid = euid = pwd->pw_uid;
rgid = egid = pwd->pw_gid;