diff options
author | rwatson <rwatson@FreeBSD.org> | 2006-09-09 16:09:01 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2006-09-09 16:09:01 +0000 |
commit | 5eee50ca3619c4fd92c6f420733cd71a1d39c3ed (patch) | |
tree | 0fb8c59b5bbf0db92241f8b2d3c58e24b0c344a9 /sys/kern/kern_ktr.c | |
parent | 9eef9bfd9631f60e70e1640784b3f74a5ed867e1 (diff) | |
download | FreeBSD-src-5eee50ca3619c4fd92c6f420733cd71a1d39c3ed.zip FreeBSD-src-5eee50ca3619c4fd92c6f420733cd71a1d39c3ed.tar.gz |
Remove slightly oddly placed suser() call from the KTR/ALQ setup sysctl:
it was present only in the enable path, not the disable path, which one
presumes would be equally of interest. Either way, it was not needed,
as the sysctl framework already calls suser() if the operation is a
write operation, which configuration requests are.
Sponsored by: nCircle Network Security, Inc.
Diffstat (limited to 'sys/kern/kern_ktr.c')
-rw-r--r-- | sys/kern/kern_ktr.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sys/kern/kern_ktr.c b/sys/kern/kern_ktr.c index f12f971..721cc52 100644 --- a/sys/kern/kern_ktr.c +++ b/sys/kern/kern_ktr.c @@ -163,9 +163,6 @@ sysctl_debug_ktr_alq_enable(SYSCTL_HANDLER_ARGS) if (enable) { if (ktr_alq_enabled) return (0); - error = suser(curthread); - if (error) - return (error); error = alq_open(&ktr_alq, (const char *)ktr_alq_file, req->td->td_ucred, ALQ_DEFAULT_CMODE, sizeof(struct ktr_entry), ktr_alq_depth); |