diff options
| author | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2008-03-16 17:35:03 +0000 |
|---|---|---|
| committer | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2008-03-16 17:35:03 +0000 |
| commit | 889825b6db150b3f0fc684fdb2ec8d95844f7437 (patch) | |
| tree | 06ce99434ddada3f9beb6a9e14d4748ac5d49cc5 /libshouldbeinlibc | |
| parent | 0bdc0832be3b459f721205372c7ba1ea7f617f73 (diff) | |
2008-03-16 Samuel Thibault <samuel.thibault@ens-lyon.org>
* idvec-verify.c (verify_id): Compare id to (uid_t) -1 instead of
checking that is is positive or nul (which is always true).
Reported by Marc-Olivier Mercier <mercier.m@sympatico.ca>.
Diffstat (limited to 'libshouldbeinlibc')
| -rw-r--r-- | libshouldbeinlibc/ChangeLog | 6 | ||||
| -rw-r--r-- | libshouldbeinlibc/idvec-verify.c | 2 |
2 files changed, 7 insertions, 1 deletions
diff --git a/libshouldbeinlibc/ChangeLog b/libshouldbeinlibc/ChangeLog index 930b79a7..26d2424e 100644 --- a/libshouldbeinlibc/ChangeLog +++ b/libshouldbeinlibc/ChangeLog @@ -1,3 +1,9 @@ +2008-03-16 Samuel Thibault <samuel.thibault@ens-lyon.org> + + * idvec-verify.c (verify_id): Compare id to (uid_t) -1 instead of + checking that is is positive or nul (which is always true). + Reported by Marc-Olivier Mercier <mercier.m@sympatico.ca>. + 2007-02-04 Thomas Schwinge <tschwinge@gnu.org> * maptime.h: Fix a comment. diff --git a/libshouldbeinlibc/idvec-verify.c b/libshouldbeinlibc/idvec-verify.c index 55d20e2c..5527f308 100644 --- a/libshouldbeinlibc/idvec-verify.c +++ b/libshouldbeinlibc/idvec-verify.c @@ -258,7 +258,7 @@ verify_id (uid_t id, int is_group, int multiple, /* VERIFY_FN should have been defaulted in idvec_verify if necessary. */ assert (verify_fn); - if (id >= 0) + if (id != (uid_t) -1) do { if (is_group) |