Return-Path: MIME-Version: 1.0 In-Reply-To: <20140612112210.GD27592@aemeltch-MOBL1> References: <1402569825-27168-1-git-send-email-Andrei.Emeltchenko.news@gmail.com> <20140612112210.GD27592@aemeltch-MOBL1> Date: Thu, 12 Jun 2014 13:24:29 +0200 Message-ID: Subject: Re: [PATCH] android/gatt: Fix wrong initial value From: Marcin Kraglak To: Andrei Emeltchenko , Marcin Kraglak , "linux-bluetooth@vger.kernel.org development" Content-Type: text/plain; charset=UTF-8 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi, On 12 June 2014 13:22, Andrei Emeltchenko wrote: > On Thu, Jun 12, 2014 at 01:05:45PM +0200, Marcin Kraglak wrote: >> Hi Andrei, >> >> On 12 June 2014 12:43, Andrei Emeltchenko >> wrote: >> > From: Andrei Emeltchenko >> > >> > Initialize value_len to -1 since otherwise check value_len >= 0 does not >> > make sense. >> > --- >> > android/gatt.c | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/android/gatt.c b/android/gatt.c >> > index 8da77d0..8260a8d 100644 >> > --- a/android/gatt.c >> > +++ b/android/gatt.c >> > @@ -4147,7 +4147,7 @@ static void read_requested_attributes(void *data, void *user_data) >> > struct request_processing_data *process_data = user_data; >> > uint32_t permissions; >> > uint8_t *value, error; >> > - int value_len = 0; >> > + int value_len = -1; >> >> What will happen if check_device_permissions will fail? > > nothing? Should we send message? we should set error in resp_data. Anyway it is repaired in android/gatt: Set state of pending responses > > Best regards > Andrei Emeltchenko > BR Marcin