From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alasdair G Kergon Subject: Re: [PATCH 2/6] drivers/md: remove null pointer dereference Date: Wed, 2 Jul 2008 20:03:06 +0100 Message-ID: <20080702190306.GG22522@agk.fab.redhat.com> References: <20080702105434.GF22522@agk.fab.redhat.com> Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Julia Lawall Cc: device-mapper development , kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: dm-devel.ids On Wed, Jul 02, 2008 at 01:51:41PM +0200, Julia Lawall wrote: > In looking at it a little bit, it seems that ps.type is initialized in the > function stored in the ctr field of the target_type structure and this > function is called in the function stored in the message field of the same > structure. The function in the message structure seems to be only called > from the function target_message in dm-ioctl.c, but I don't see the > relation to an invocation of the ctr field. Is that guaranteed to be > invoked earlier? ctr is short for constructor (perhaps we should rename these functions one day) and it has to run first to set up the data structures which a subsequent target_message may reference. Alasdair -- agk@redhat.com