Date: Thu, 17 Nov 2005 08:45:01 -0800 From: Nate Lawson <nate@root.org> To: Pyun YongHyeon <yongari@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern subr_rman.c Message-ID: <437CB38D.3020301@root.org> In-Reply-To: <20051117085649.CD31216A46F@hub.freebsd.org> References: <20051117085649.CD31216A46F@hub.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Pyun YongHyeon wrote: > yongari 2005-11-17 08:56:21 UTC > > FreeBSD src repository > > Modified files: > sys/kern subr_rman.c > Log: > Prefer NULL to 0. > Add missing lock/unlock in sysctl handler. > Protect accessing NULL pointer when resource allocation was failed. > style(9) > > Reviewed by: scottl > MFC after: 1 week > > Revision Changes Path > 1.48 +29 -25 src/sys/kern/subr_rman.c It is preferable to commit whitespace/style changes separately from functional changes, that way they are more visible in the diff. Thanks. -- Nate
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?437CB38D.3020301>