svn commit: r325923 - head/sys/kern

Mateusz Guzik mjg at FreeBSD.org
Fri Nov 17 02:29:07 UTC 2017


Author: mjg
Date: Fri Nov 17 02:29:06 2017
New Revision: 325923
URL: https://svnweb.freebsd.org/changeset/base/325923

Log:
  Check for PRS_NEW without locking the proc in sysctl_kern_proc

Modified:
  head/sys/kern/kern_proc.c

Modified: head/sys/kern/kern_proc.c
==============================================================================
--- head/sys/kern/kern_proc.c	Fri Nov 17 02:27:04 2017	(r325922)
+++ head/sys/kern/kern_proc.c	Fri Nov 17 02:29:06 2017	(r325923)
@@ -1479,11 +1479,9 @@ sysctl_kern_proc(SYSCTL_HANDLER_ARGS)
 			/*
 			 * Skip embryonic processes.
 			 */
-			PROC_LOCK(p);
-			if (p->p_state == PRS_NEW) {
-				PROC_UNLOCK(p);
+			if (p->p_state == PRS_NEW)
 				continue;
-			}
+			PROC_LOCK(p);
 			KASSERT(p->p_ucred != NULL,
 			    ("process credential is NULL for non-NEW proc"));
 			/*


More information about the svn-src-all mailing list