Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 26 Feb 2009 12:12:34 +0000 (UTC)
From:      Ed Schouten <ed@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r189066 - head/sys/kern
Message-ID:  <200902261212.n1QCCYI6027315@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ed
Date: Thu Feb 26 12:12:34 2009
New Revision: 189066
URL: http://svn.freebsd.org/changeset/base/189066

Log:
  Remove redundant assignment of `p'.
  
  `p' is already initialized with `td->td_proc'. Because td is always
  curthread, it is safe to initialize it without any locks.
  
  Found by:	LLVM's scan-build

Modified:
  head/sys/kern/subr_prf.c

Modified: head/sys/kern/subr_prf.c
==============================================================================
--- head/sys/kern/subr_prf.c	Thu Feb 26 12:06:46 2009	(r189065)
+++ head/sys/kern/subr_prf.c	Thu Feb 26 12:12:34 2009	(r189066)
@@ -137,7 +137,6 @@ uprintf(const char *fmt, ...)
 		return (0);
 
 	sx_slock(&proctree_lock);
-	p = td->td_proc;
 	PROC_LOCK(p);
 	if ((p->p_flag & P_CONTROLT) == 0) {
 		PROC_UNLOCK(p);
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"



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