Change the suser() API to take advantage of td_ucred as well as do a

general cleanup of the API.  The entire API now consists of two functions
similar to the pre-KSE API.  The suser() function takes a thread pointer
as its only argument.  The td_ucred member of this thread must be valid
so the only valid thread pointers are curthread and a few kernel threads
such as thread0.  The suser_cred() function takes a pointer to a struct
ucred as its first argument and an integer flag as its second argument.
The flag is currently only used for the PRISON_ROOT flag.

Discussed on:	smp@

1 files changed, 2 insertions, 2 deletions

diff --git a/sys/alpha/osf1/osf1_misc.c b/sys/alpha/osf1/osf1_misc.c
index 7c82947..cac8ad8 100644
--- a/sys/alpha/osf1/osf1_misc.c
+++ b/sys/alpha/osf1/osf1_misc.c
@@ -1062,7 +1062,7 @@ osf1_setuid(td, uap)
 	uid = SCARG(uap, uid);
 	oldcred = p->p_ucred;
 
-	if ((error = suser_xxx(p->p_ucred, NULL, PRISON_ROOT)) != 0 &&
+	if ((error = suser_cred(p->p_ucred, PRISON_ROOT)) != 0 &&
 	    uid != oldcred->cr_ruid && uid != oldcred->cr_svuid)
 		return (error);
 
@@ -1108,7 +1108,7 @@ osf1_setgid(td, uap)
 	gid = SCARG(uap, gid);
 	oldcred = p->p_ucred;
 
-	if (((error = suser_xxx(p->p_ucred, NULL, PRISON_ROOT)) != 0 ) &&
+	if (((error = suser_cred(p->p_ucred, PRISON_ROOT)) != 0 ) &&
 	    gid != oldcred->cr_rgid && gid != oldcred->cr_svgid)
 		return (error);