diff options
| author | arr <arr@FreeBSD.org> | 2002-03-21 15:27:39 +0000 |
|---|---|---|
| committer | arr <arr@FreeBSD.org> | 2002-03-21 15:27:39 +0000 |
| commit | 68e226a99e4bb06935a253fc6a79afc3fa43bcd0 (patch) | |
| tree | 7bde5895adacd26ebeb471c903ce82ce8daa74e0 /sys/kern/kern_linker.c | |
| parent | ec2a00af74c8240f7fc6f03c84ad40034fbf2aac (diff) | |
| download | FreeBSD-src-68e226a99e4bb06935a253fc6a79afc3fa43bcd0.zip FreeBSD-src-68e226a99e4bb06935a253fc6a79afc3fa43bcd0.tar.gz | |
- Fix a logic error in checking the securelevel that was introduced in the
previous commit.
Pointy hats to: arr, rwatson
Diffstat (limited to 'sys/kern/kern_linker.c')
| -rw-r--r-- | sys/kern/kern_linker.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/sys/kern/kern_linker.c b/sys/kern/kern_linker.c index 9d644d2..1050b33 100644 --- a/sys/kern/kern_linker.c +++ b/sys/kern/kern_linker.c @@ -700,10 +700,8 @@ kldload(struct thread *td, struct kldload_args *uap) mtx_lock(&Giant); - if (securelevel_gt(td->td_ucred, 0) == 0) { - error = EPERM; + if ((error = securelevel_gt(td->td_ucred, 0)) != 0) goto out; - } if ((error = suser_xxx(td->td_ucred, NULL, 0)) != 0) goto out; @@ -749,10 +747,8 @@ kldunload(struct thread *td, struct kldunload_args *uap) mtx_lock(&Giant); - if (securelevel_gt(td->td_ucred, 0) == 0) { - error = EPERM; + if ((error = securelevel_gt(td->td_ucred, 0)) != 0) goto out; - } if ((error = suser_xxx(td->td_ucred, NULL, 0)) != 0) goto out; |
