From owner-cvs-all@FreeBSD.ORG  Sat Aug 14 17:15:17 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 29B8816A4CE; Sat, 14 Aug 2004 17:15:17 +0000 (GMT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 227F043D1D; Sat, 14 Aug 2004 17:15:17 +0000 (GMT)
	(envelope-from rwatson@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.11/8.12.11) with ESMTP id i7EHFHG6049585;
	Sat, 14 Aug 2004 17:15:17 GMT
	(envelope-from rwatson@repoman.freebsd.org)
Received: (from rwatson@localhost)
	by repoman.freebsd.org (8.12.11/8.12.11/Submit) id i7EHFH16049584;
	Sat, 14 Aug 2004 17:15:17 GMT
	(envelope-from rwatson)
Message-Id: <200408141715.i7EHFH16049584@repoman.freebsd.org>
From: Robert Watson <rwatson@FreeBSD.org>
Date: Sat, 14 Aug 2004 17:15:17 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/kern kern_proc.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 14 Aug 2004 17:15:17 -0000

rwatson     2004-08-14 17:15:16 UTC

  FreeBSD src repository

  Modified files:
    sys/kern             kern_proc.c 
  Log:
  Cause pfind() not to return processes in the PRS_NEW state.  As a result,
  threads consuming the result of pfind() will not need to check for a NULL
  credential pointer or other signs of an incompletely created process.
  However, this also means that pfind() cannot be used to test for the
  existence or find such a process.  Annotate pfind() to indicate that this
  is the case.  A review of curent consumers seems to indicate that this is
  not a problem for any of them.  This closes a number of race conditions
  that could result in NULL pointer dereferences and related failure modes.
  Other related races continue to exist, especially during iteration of the
  allproc list without due caution.
  
  Discussed with: tjr, green
  
  Revision  Changes    Path
  1.215     +8 -1      src/sys/kern/kern_proc.c