Re: [PATCH 3/6] profiles/network: Use send ctrl rsp to unknown BNEP ctrl cmd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Szymon,

On 26 February 2015 at 10:37, Szymon Janc <szymon.janc@xxxxxxxxx> wrote:
>
> Hi Grzegorz,
>
> On Thursday 19 of February 2015 15:16:36 Grzegorz Kolodziejczyk wrote:
> > Use dedicated control response function instead of assembling and
> > sending raw packet over socket.
> > ---
> >  profiles/network/server.c | 12 ++++--------
> >  1 file changed, 4 insertions(+), 8 deletions(-)
> >
> > diff --git a/profiles/network/server.c b/profiles/network/server.c
> > index ebbe056..37bfba4 100644
> > --- a/profiles/network/server.c
> > +++ b/profiles/network/server.c
> > @@ -307,14 +307,10 @@ static gboolean bnep_setup(GIOChannel *chan,
> >       /* Highest known Control command ID
> >        * is BNEP_FILTER_MULT_ADDR_RSP = 0x06 */
> >       if (req->type == BNEP_CONTROL &&
> > -                             req->ctrl > BNEP_FILTER_MULT_ADDR_RSP) {
> > -             uint8_t pkt[3];
> > -
> > -             pkt[0] = BNEP_CONTROL;
> > -             pkt[1] = BNEP_CMD_NOT_UNDERSTOOD;
> > -             pkt[2] = req->ctrl;
> > -
> > -             send(sk, pkt, sizeof(pkt), 0);
> > +                                     req->ctrl > BNEP_FILTER_MULT_ADDR_RSP) {
> > +             error("cmd not understood");
> > +             bnep_send_ctrl_rsp(sk, BNEP_CONTROL, BNEP_CMD_NOT_UNDERSTOOD,
> > +                                                             req->ctrl);
> >
> >               return FALSE;
> >       }
>
> As discussed offline, this is not correct since we need to handle
> BNEP_CMD_NOT_UNDERSTOOD special case in bnep_send_ctrl_rsp.
>
Yes, I've prepared a patch since there is similar issue in pan android
profile code. Will rebase it.
>
> --
> Best regards,
> Szymon Janc

Best regards,
Grzegorz Kołodziejczyk
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux