diff options
author | Ludovic Courtès <ludo@gnu.org> | 2011-11-12 01:25:45 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2011-11-14 09:52:42 +0100 |
commit | 40324ca8b22a538e31aa2f9a8ac03f03f7d0537e (patch) | |
tree | 4a5a2c43a3bf870f64bf5e27fdd5aa342be246d1 /src | |
parent | b8b6dd27a61dce58d6537dfbee65c7428f4ae3cd (diff) | |
download | coreutils-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')
-rw-r--r-- | src/id.c | 20 |
1 files changed, 20 insertions, 0 deletions
@@ -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) |