Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 6 Nov 2003 05:55:20 -0800 (PST)
From:      Andrew Reisse <areisse@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 41583 for review
Message-ID:  <200311061355.hA6DtKLT030993@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=41583

Change 41583 by areisse@areisse_ibook on 2003/11/06 05:54:48

	change setgroups to use crdup

Affected files ...

.. //depot/projects/trustedbsd/sedarwin/apsl/xnu/bsd/kern/kern_prot.c#7 edit

Differences ...

==== //depot/projects/trustedbsd/sedarwin/apsl/xnu/bsd/kern/kern_prot.c#7 (text+ko) ====

@@ -495,7 +495,10 @@
 	ngrp = uap->gidsetsize;
 	if (ngrp < 1 || ngrp > NGROUPS)
 		return (EINVAL);
-	new = crget();
+	pcred_writelock(p);
+	old = pc->pc_ucred;
+	new = crdup (old);
+	pcred_unlock(p);
 	error = copyin((caddr_t)uap->gidset,
 	    (caddr_t)new->cr_groups, ngrp * sizeof(gid_t));
 	if (error) {
@@ -504,11 +507,6 @@
 	}
 	new->cr_ngroups = ngrp;
 	pcred_writelock(p);
-	old = pc->pc_ucred;
-	new->cr_uid = old->cr_uid;
-#ifdef MAC
-	mac_create_cred (old, new);
-#endif
 	pc->pc_ucred = new;
 	pcred_unlock(p);
 	set_security_token(p);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200311061355.hA6DtKLT030993>