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

1 files 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);