Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 29 Aug 2010 21:28:07 GMT
From:      Edward Tomasz Napierala <trasz@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 183050 for review
Message-ID:  <201008292128.o7TLS7BS081605@skunkworks.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://p4web.freebsd.org/@@183050?ac=10

Change 183050 by trasz@trasz_victim on 2010/08/29 21:27:28

	In this branch, chgproccnt() in unionfs code is not needed, because
	of change_cred().

Affected files ...

.. //depot/projects/soc2009/trasz_limits/sys/fs/unionfs/union_subr.c#11 edit

Differences ...

==== //depot/projects/soc2009/trasz_limits/sys/fs/unionfs/union_subr.c#11 (text+ko) ====

@@ -775,11 +775,6 @@
 	/* Authority change to root */
 	rootinfo = uifind((uid_t)0);
 	cred = crdup(cnp->cn_cred);
-	/*
-	 * The calls to chgproccnt() are needed to compensate for change_ruid()
-	 * calling chgproccnt().
-	 */
-	chgproccnt(cred->cr_ruidinfo, 1, 0);
 	change_euid(cred, rootinfo);
 	change_ruid(cred, rootinfo);
 	change_svuid(cred, (uid_t)0);
@@ -829,7 +824,6 @@
 
 unionfs_mkshadowdir_abort:
 	cnp->cn_cred = credbk;
-	chgproccnt(cred->cr_ruidinfo, -1, 0);
 	crfree(cred);
 
 	return (error);



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