linux-kernel - Re: [BUG] Bluetooth broken post rc7

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LRH.2.00.1107162141340.7178@twin.jikos.cz>
Date:	Sat, 16 Jul 2011 21:43:06 +0200 (CEST)
From:	Jiri Kosina <jkosina@...e.cz>
To:	Ed Tomlinson <edt@....ca>
Cc:	Luiz Augusto von Dentz <luiz.dentz@...il.com>,
	linux-kernel@...r.kernel.org, linux-bluetooth@...r.kernel.org,
	"Gustavo F. Padovan" <padovan@...fusion.mobi>,
	Ilia Kolomisnky <iliak@...com>,
	David Miller <davem@...emloft.net>,
	Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: [BUG] Bluetooth broken post rc7


[ adding Linus to CC ... would be good to have this fixed before 3.0-final 
  indeed ]

On Fri, 15 Jul 2011, Ed Tomlinson wrote:

> > > I booted to todays linux git and bluetooth is no longer working.  
> > > My magic mouse will not connect correctly.
> > >
> > > Reverting:
> > >
> > > commit 9fa7e4f76f3658ba1f44fbdb95c77e7df3f53f95
> > > Author: Gustavo F. Padovan <padovan@...fusion.mobi>
> > > Date:   Thu Jun 30 16:11:30 2011 -0300
> > >
> > >    Bluetooth: Fix regression with incoming L2CAP connections
> > >
> > >    PTS test A2DP/SRC/SRC_SET/TC_SRC_SET_BV_02_I revealed that
> > >    ( probably after the df3c3931e commit ) the l2cap connection
> > >    could not be established in case when the "Auth Complete" HCI
> > >    event does not arive before the initiator send "Configuration
> > >    request", in which case l2cap replies with "Command rejected"
> > >    since the channel is still in BT_CONNECT2 state.
> > >
> > >    Based on patch from: Ilia Kolomisnky <iliak@...com>
> > >
> > >    Signed-off-by: Gustavo F. Padovan <padovan@...fusion.mobi>
> > >
> > > Fixes the bug here (things work like rc7).
> > >
> > > Do we really need 9fa7e4f76f3658ba1f44fbdb95c77e7df3f53f95 in 3.0?
> > >
> > > Thanks,
> > >
> > > Ed Tomlinson
> > > --
> > 
> > I've changed the check to something like this:
> > 
> > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c
> > index ebff14c..f88b9c8 100644
> > --- a/net/bluetooth/l2cap_core.c
> > +++ b/net/bluetooth/l2cap_core.c
> > @@ -2323,8 +2323,7 @@ static inline int l2cap_config_req(struct
> > l2cap_conn *conn, struct l2cap_cmd_hdr
> > 
> >         sk = chan->sk;
> > 
> > -       if ((bt_sk(sk)->defer_setup && sk->sk_state != BT_CONNECT2) ||
> > -                (!bt_sk(sk)->defer_setup && sk->sk_state != BT_CONFIG)) {
> > +       if ((sk->sk_state != BT_CONNECT2) && (sk->sk_state != BT_CONFIG)) {
> >                 struct l2cap_cmd_rej rej;
> > 
> >                 rej.reason = cpu_to_le16(0x0002);
> > 
> > Seems to work fine.
> 
> And here too.  You can can my tested by
>  
> Tested By: Ed Tomlinson <edt@....ca>
> 
> IMHO this should be in 3.0.0
> 
> Thanks!
> Ed Tomlinson

Luiz,

could you please repost the patch with proper changelog and your 
Signed-off-by, so that it could (hopefully) be applied for 3.0 still?

Thanks,

-- 
Jiri Kosina
SUSE Labs

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/