From 397b3428eb3698cc1b82caab2bd51a72da41022c Mon Sep 17 00:00:00 2001 From: Robert Watson Date: Fri, 22 Oct 2004 12:10:50 +0000 Subject: [PATCH] In osethostname(), don't need to call suser() directly as userland_sysctl() will perform all necessary privilege checks for the caller. --- sys/kern/kern_xxx.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/sys/kern/kern_xxx.c b/sys/kern/kern_xxx.c index 4df23d8753f2..c5ba021e4ac8 100644 --- a/sys/kern/kern_xxx.c +++ b/sys/kern/kern_xxx.c @@ -96,10 +96,8 @@ osethostname(td, uap) name[0] = CTL_KERN; name[1] = KERN_HOSTNAME; mtx_lock(&Giant); - if ((error = suser_cred(td->td_ucred, SUSER_ALLOWJAIL)) == 0) { - error = userland_sysctl(td, name, 2, 0, 0, 0, - uap->hostname, uap->len, 0, 0); - } + error = userland_sysctl(td, name, 2, 0, 0, 0, uap->hostname, + uap->len, 0, 0); mtx_unlock(&Giant); return (error); }