summaryrefslogtreecommitdiff
path: root/src/id.c
diff options
context:
space:
mode:
authorLudovic Courtès <ludo@gnu.org>2011-11-12 01:25:45 +0100
committerJim Meyering <meyering@redhat.com>2011-11-14 09:52:42 +0100
commit40324ca8b22a538e31aa2f9a8ac03f03f7d0537e (patch)
tree4a5a2c43a3bf870f64bf5e27fdd5aa342be246d1 /src/id.c
parentb8b6dd27a61dce58d6537dfbee65c7428f4ae3cd (diff)
downloadcoreutils-40324ca8b22a538e31aa2f9a8ac03f03f7d0537e.tar.xz
id: fail when getuid, getgid, etc. fail, e.g., on GNU/Hurd
POSIX-conforming getuid, geteuid, etc. functions cannot fail, but on GNU/Hurd systems and some others, they may. * src/id.c (main) [__GNU__]: Detect and diagnose any such failure. * tests/id/gnu-zero-uids: New file. * tests/Makefile.am (TESTS): Add it to the list. * tests/init.cfg (require_gnu_): New function.
Diffstat (limited to 'src/id.c')
-rw-r--r--src/id.c20
1 files changed, 20 insertions, 0 deletions
diff --git a/src/id.c b/src/id.c
index cfbaa296c..047e40b88 100644
--- a/src/id.c
+++ b/src/id.c
@@ -38,6 +38,13 @@
proper_name ("Arnold Robbins"), \
proper_name ("David MacKenzie")
+/* Whether the functions getuid, geteuid, getgid and getegid may fail. */
+#ifdef __GNU__
+# define GETID_MAY_FAIL 1
+#else
+# define GETID_MAY_FAIL 0
+#endif
+
/* If nonzero, output only the SELinux context. -Z */
static int just_context = 0;
@@ -202,9 +209,22 @@ main (int argc, char **argv)
else
{
euid = geteuid ();
+ if (GETID_MAY_FAIL && euid == -1 && !use_real
+ && !just_group && !just_group_list && !just_context)
+ error (EXIT_FAILURE, errno, _("cannot get effective UID"));
+
ruid = getuid ();
+ if (GETID_MAY_FAIL && ruid == -1 && use_real
+ && !just_group && !just_group_list && !just_context)
+ error (EXIT_FAILURE, errno, _("cannot get real UID"));
+
egid = getegid ();
+ if (GETID_MAY_FAIL && egid == -1 && !use_real && !just_user)
+ error (EXIT_FAILURE, errno, _("cannot get effective GID"));
+
rgid = getgid ();
+ if (GETID_MAY_FAIL && rgid == -1 && use_real && !just_user)
+ error (EXIT_FAILURE, errno, _("cannot get real GID"));
}
if (just_user)