From cc807dbd0a5f024bfdd350479358e0a7c7373b5a Mon Sep 17 00:00:00 2001 From: Robert Watson Date: Tue, 10 Apr 2007 15:48:45 +0000 Subject: [PATCH] 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 e580e2f29d9..c5f31e3622e 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);