diff options
| author | trasz <trasz@FreeBSD.org> | 2010-06-03 13:43:58 +0000 |
|---|---|---|
| committer | trasz <trasz@FreeBSD.org> | 2010-06-03 13:43:58 +0000 |
| commit | 9985f972fdf974d991675a2a993da9d5f2e0044c (patch) | |
| tree | cb84f406748059729438a60903ad9daae9f64cd5 | |
| parent | cbfca8b888ecc0c15fc757ccc9efc6a0c8febae6 (diff) | |
| download | FreeBSD-src-9985f972fdf974d991675a2a993da9d5f2e0044c.zip FreeBSD-src-9985f972fdf974d991675a2a993da9d5f2e0044c.tar.gz | |
The 'acl_cnt' field is unsigned; no point in checking if it's >= 0.
Found with: Coverity Prevent
CID: 3684
| -rw-r--r-- | sys/kern/subr_acl_posix1e.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/kern/subr_acl_posix1e.c b/sys/kern/subr_acl_posix1e.c index 600067c..f6fc6b2 100644 --- a/sys/kern/subr_acl_posix1e.c +++ b/sys/kern/subr_acl_posix1e.c @@ -558,7 +558,7 @@ acl_posix1e_check(struct acl *acl) */ num_acl_user_obj = num_acl_user = num_acl_group_obj = num_acl_group = num_acl_mask = num_acl_other = 0; - if (acl->acl_cnt > ACL_MAX_ENTRIES || acl->acl_cnt < 0) + if (acl->acl_cnt > ACL_MAX_ENTRIES) return (EINVAL); for (i = 0; i < acl->acl_cnt; i++) { /* |
