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>