diff options
author | rwatson <rwatson@FreeBSD.org> | 2001-09-25 21:08:33 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2001-09-25 21:08:33 +0000 |
commit | 0ae354be64e45642aa94290dd9219ac58129e4df (patch) | |
tree | 0fbbb5caa17562c234ae836528b704289475a4a6 /sys/kern | |
parent | c6a3ab8fc67f4bf691c736efb178a8032e64d625 (diff) | |
download | FreeBSD-src-0ae354be64e45642aa94290dd9219ac58129e4df.zip FreeBSD-src-0ae354be64e45642aa94290dd9219ac58129e4df.tar.gz |
o So, when <dd> e-mailed me and said that the comment was inverted
for securelevel_ge() and securelevel_gt(), I was a little surprised,
but fixed it. Turns out that it was the code that was inverted, during
a whitespace cleanup in my commit tree. This commit inverts the
checks, and restores the comment.
Diffstat (limited to 'sys/kern')
-rw-r--r-- | sys/kern/kern_prot.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/sys/kern/kern_prot.c b/sys/kern/kern_prot.c index e9bbd87..1f52132 100644 --- a/sys/kern/kern_prot.c +++ b/sys/kern/kern_prot.c @@ -1302,8 +1302,8 @@ suser_xxx(cred, proc, flag) /* * Test securelevel values against passed required securelevel. - * _gt implements (securelevel > level), and _ge implements - * (securelevel >= level). Returns 0 oer EPERM. + * _gt implements (level > securelevel), and _ge implements + * (level >= securelevel). Returns 0 oer EPERM. * * cr is permitted to be NULL for the time being, as there were some * existing securelevel checks that occurred without a process/credential @@ -1320,12 +1320,12 @@ securelevel_gt(struct ucred *cr, int level) if (cr == NULL) { printf("securelevel_gt: cr is NULL\n"); - if (securelevel > level) + if (level > securelevel) return (0); else return (EPERM); } else { - if (securelevel > level) + if (level > securelevel) return (0); else return (EPERM); @@ -1338,12 +1338,12 @@ securelevel_ge(struct ucred *cr, int level) if (cr == NULL) { printf("securelevel_ge: cr is NULL\n"); - if (securelevel >= level) + if (level >= securelevel) return (0); else return (EPERM); } else { - if (securelevel >= level) + if (level >= securelevel) return (0); else return (EPERM); |