svn commit: r189066 - head/sys/kern

Ed Schouten ed at FreeBSD.org
Thu Feb 26 11:30:25 PST 2009


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 at freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe at freebsd.org"


More information about the svn-src-head mailing list