From owner-cvs-src@FreeBSD.ORG Thu Nov 17 16:45:40 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 9427016A420; Thu, 17 Nov 2005 16:45:40 +0000 (GMT) (envelope-from nate@root.org) Received: from www.cryptography.com (li-22.members.linode.com [64.5.53.22]) by mx1.FreeBSD.org (Postfix) with ESMTP id C574443D46; Thu, 17 Nov 2005 16:45:32 +0000 (GMT) (envelope-from nate@root.org) Received: from [10.0.5.50] (ppp-71-139-0-107.dsl.snfc21.pacbell.net [71.139.0.107]) by www.cryptography.com (8.12.8/8.12.8) with ESMTP id jAHGjXDe019504 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Thu, 17 Nov 2005 08:45:34 -0800 Message-ID: <437CB38D.3020301@root.org> Date: Thu, 17 Nov 2005 08:45:01 -0800 From: Nate Lawson User-Agent: Mozilla Thunderbird 1.0.6 (Windows/20050716) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Pyun YongHyeon References: <20051117085649.CD31216A46F@hub.freebsd.org> In-Reply-To: <20051117085649.CD31216A46F@hub.freebsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern subr_rman.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Nov 2005 16:45:40 -0000 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