summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--libshouldbeinlibc/ChangeLog6
-rw-r--r--libshouldbeinlibc/idvec-verify.c2
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)