svn commit: r268002 - head/sys/kern

Mateusz Guzik mjg at FreeBSD.org
Sat Jun 28 05:51:45 UTC 2014


Author: mjg
Date: Sat Jun 28 05:51:45 2014
New Revision: 268002
URL: http://svnweb.freebsd.org/changeset/base/268002

Log:
  Call fdcloseexec right after fdunshare.
  
  No functional changes.
  
  MFC after:	1 week

Modified:
  head/sys/kern/kern_exec.c

Modified: head/sys/kern/kern_exec.c
==============================================================================
--- head/sys/kern/kern_exec.c	Sat Jun 28 05:41:53 2014	(r268001)
+++ head/sys/kern/kern_exec.c	Sat Jun 28 05:51:45 2014	(r268002)
@@ -595,6 +595,8 @@ interpret:
 	 * be shared after an exec.
 	 */
 	fdunshare(td);
+	/* close files on exec */
+	fdcloseexec(td);
 
 	/*
 	 * Malloc things before we need locks.
@@ -608,8 +610,6 @@ interpret:
 		bcopy(imgp->args->begin_argv, newargs->ar_args, i);
 	}
 
-	/* close files on exec */
-	fdcloseexec(td);
 	vn_lock(imgp->vp, LK_SHARED | LK_RETRY);
 
 	/* Get a reference to the vnode prior to locking the proc */


More information about the svn-src-head mailing list