diff options
author | trasz <trasz@FreeBSD.org> | 2010-06-03 14:16:58 +0000 |
---|---|---|
committer | trasz <trasz@FreeBSD.org> | 2010-06-03 14:16:58 +0000 |
commit | 45e71b225065e0fcf3014d806116fb4655506d8f (patch) | |
tree | 030b95fb477c6eb5a6f0f335ceb1f5fed7036d11 /lib/libc | |
parent | 0463f9931f9717fc66d7aa68fbbe014275fce16d (diff) | |
download | FreeBSD-src-45e71b225065e0fcf3014d806116fb4655506d8f.zip FreeBSD-src-45e71b225065e0fcf3014d806116fb4655506d8f.tar.gz |
The 'acl_cnt' field is unsigned; no point in checking if it's >= 0.
Found with: Coverity Prevent
CID: 6192
Diffstat (limited to 'lib/libc')
-rw-r--r-- | lib/libc/posix1e/acl_entry.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/libc/posix1e/acl_entry.c b/lib/libc/posix1e/acl_entry.c index 86ec290..3f8ca62 100644 --- a/lib/libc/posix1e/acl_entry.c +++ b/lib/libc/posix1e/acl_entry.c @@ -55,8 +55,7 @@ acl_create_entry(acl_t *acl_p, acl_entry_t *entry_p) * +1, because we are checking if there is space left for one more * entry. */ - if ((acl_int->acl_cnt + 1 >= ACL_MAX_ENTRIES) || - (acl_int->acl_cnt < 0)) { + if (acl_int->acl_cnt + 1 >= ACL_MAX_ENTRIES) { errno = EINVAL; return (-1); } |