From fc1b9c146d4d1e857dc4eb0d4f003b3fd6600295 Mon Sep 17 00:00:00 2001 From: rwatson Date: Tue, 10 Apr 2007 15:48:45 +0000 Subject: Remove unnecessary suser() check in the sysctl to set up ath_hal logging: the sysctl framework will already have checked for privilege if a sysctl value is being set. Discussed a long time ago with: sam --- sys/dev/ath/ah_osdep.c | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/sys/dev/ath/ah_osdep.c b/sys/dev/ath/ah_osdep.c index e580e2f..c5f31e36 100644 --- a/sys/dev/ath/ah_osdep.c +++ b/sys/dev/ath/ah_osdep.c @@ -199,16 +199,13 @@ ath_hal_setlogging(int enable) int error; if (enable) { - error = suser(curthread); - if (error == 0) { - error = alq_open(&ath_hal_alq, ath_hal_logfile, - curthread->td_ucred, ALQ_DEFAULT_CMODE, - sizeof (struct athregrec), ath_hal_alq_qsize); - ath_hal_alq_lost = 0; - ath_hal_alq_emitdev = 1; - printf("ath_hal: logging to %s enabled\n", - ath_hal_logfile); - } + error = alq_open(&ath_hal_alq, ath_hal_logfile, + curthread->td_ucred, ALQ_DEFAULT_CMODE, + sizeof (struct athregrec), ath_hal_alq_qsize); + ath_hal_alq_lost = 0; + ath_hal_alq_emitdev = 1; + printf("ath_hal: logging to %s enabled\n", + ath_hal_logfile); } else { if (ath_hal_alq) alq_close(ath_hal_alq); -- cgit v1.1