From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [patch v2] bluetooth: handle l2cap_create_connless_pdu() errors From: Marcel Holtmann To: David Miller Cc: gustavo@padovan.org, error27@gmail.com, andrei.emeltchenko@nokia.com, linux-bluetooth@vger.kernel.org, netdev@vger.kernel.org, kernel-janitors@vger.kernel.org In-Reply-To: <20100426.111259.112594696.davem@davemloft.net> References: <20100422123347.GA5265@vigoh> <20100426113627.GS29093@bicker> <20100426150919.GA12813@vigoh> <20100426.111259.112594696.davem@davemloft.net> Content-Type: text/plain; charset="UTF-8" Date: Tue, 27 Apr 2010 04:05:26 +0200 Message-ID: <1272333926.22838.116.camel@localhost.localdomain> Mime-Version: 1.0 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Dave, > > * Dan Carpenter [2010-04-26 13:36:27 +0200]: > > > >> l2cap_create_connless_pdu() can sometimes return ERR_PTR(-ENOMEM) or > >> ERR_PTR(-EFAULT). > >> > >> Signed-off-by: Dan Carpenter > >> --- > >> In v2 I wrote the patch on top of Gustavo Padovon's devel tree > > This is the kind of bug that could cause a crash if the path actually > executes. > > Therefore it tires me that that submitter was told to regenerate this > patch against some devel tree that is -next bound, when in fact this > is the kind of fix that warrants inclusion right now into net-2.6 > > Marcel, please do whatever magic you need to so I can get this into > Linus's tree as I did the rest of the ERR_PTR() fixes from Dan already. > No reason to treat Bluetooth special and defer these fixes to -next. my bad here. I looked at the patch and thought it was only an issue inside the development tree. So was letting Gustavo sort this out. You are 100% right, this needs to be fixed right away. If you wanna take this directly, then Acked-by: Marcel Holtmann Regards Marcel