From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 16 Mar 2012 13:31:10 -0300 From: Johan Hedberg To: Marcel Holtmann Cc: Gustavo Padovan , Szymon Janc , linux-bluetooth@vger.kernel.org, kanak.gupta@stericsson.com Subject: Re: [PATCH 2/2] Bluetooth: mgmt: Don't allow to set invalid value to DeviceID source Message-ID: <20120316163110.GA22943@x220> References: <1331910177-11513-1-git-send-email-szymon.janc@tieto.com> <1331910177-11513-2-git-send-email-szymon.janc@tieto.com> <20120316150615.GB19444@joana> <20120316161310.GC22160@x220> <1331914843.14217.160.camel@aeonflux> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1331914843.14217.160.camel@aeonflux> List-ID: Hi Marcel, On Fri, Mar 16, 2012, Marcel Holtmann wrote: > Hi Johan, > > > > > Reply with MGMT_STATUS_INVALID_PARAMS when userspace is trying to set > > > > source with out-of-scope value. > > > > > > > > Signed-off-by: Szymon Janc > > > > --- > > > > net/bluetooth/mgmt.c | 9 ++++++++- > > > > 1 file changed, 8 insertions(+), 1 deletion(-) > > > > > > This one should go to bluetooth.git first. > > > > > > Acked-by: Gustavo Padovan > > > > I went ahead and applied both directly to my bluetooth-next tree. I > > don't think this one is critical enough to deserve to go to > > bluetooth.git. > > you are both kidding me here right now, right? Of course this is not > going into bluetooth.git since it is not even in net-next or scheduled > for the next merge window. > > And nothing goes into bluetooth.git first (unless it only applies > there). Everything goes into bluetooth-next.git first. You're right. I realized this just as I sent the email. Johan