linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* pull request: bluetooth-2.6 2011-06-08
@ 2011-06-08 19:46 Gustavo F. Padovan
  2011-06-09 17:59 ` John W. Linville
  0 siblings, 1 reply; 5+ messages in thread
From: Gustavo F. Padovan @ 2011-06-08 19:46 UTC (permalink / raw)
  To: linville, linux-wireless, linux-bluetooth

Hi John,

First round of fixed for 3.0! There are here a regression fix for 1.1 and
earlier devices and a fix in the shutdown of SCO sockets.  Other 3 patches are
warning shut up. And one byte leak to userspace fix.

Please pull, thanks!


The following changes since commit 51e65257142a87fe46a1ce5c35c86c5baf012614:

  iwlegacy: fix channel switch locking (2011-06-08 14:19:05 -0400)

are available in the git repository at:
  master.kernel.org:/pub/scm/linux/kernel/git/padovan/bluetooth-2.6.git master

David Miller (2):
      Bluetooth: Do not ignore errors returned from strict_strtol()
      Bluetooth: Kill set but unused variable 'cmd' in cmtp_recv_capimsg()

Filip Palian (1):
      Bluetooth: l2cap and rfcomm: fix 1 byte infoleak to userspace.

Luiz Augusto von Dentz (1):
      Bluetooth: fix shutdown on SCO sockets

Stephen Boyd (1):
      Bluetooth: Silence DEBUG_STRICT_USER_COPY_CHECKS=y warning

Ville Tervo (1):
      Bluetooth: Do not send SET_EVENT_MASK for 1.1 and earlier devices

 drivers/bluetooth/btmrvl_debugfs.c |   12 ++++++++++++
 net/bluetooth/cmtp/capi.c          |    3 +--
 net/bluetooth/hci_event.c          |   18 ++++++++++--------
 net/bluetooth/l2cap_sock.c         |    1 +
 net/bluetooth/rfcomm/sock.c        |    4 +++-
 net/bluetooth/sco.c                |   13 ++++++++++++-
 6 files changed, 39 insertions(+), 12 deletions(-)


	Gustavo 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: pull request: bluetooth-2.6 2011-06-08
  2011-06-08 19:46 pull request: bluetooth-2.6 2011-06-08 Gustavo F. Padovan
@ 2011-06-09 17:59 ` John W. Linville
  2011-06-09 18:34   ` Gustavo F. Padovan
  0 siblings, 1 reply; 5+ messages in thread
From: John W. Linville @ 2011-06-09 17:59 UTC (permalink / raw)
  To: linux-wireless, linux-bluetooth

On Wed, Jun 08, 2011 at 04:46:58PM -0300, Gustavo F. Padovan wrote:
> Hi John,
> 
> First round of fixed for 3.0! There are here a regression fix for 1.1 and
> earlier devices and a fix in the shutdown of SCO sockets.  Other 3 patches are
> warning shut up. And one byte leak to userspace fix.
> 
> Please pull, thanks!
> 
> 
> The following changes since commit 51e65257142a87fe46a1ce5c35c86c5baf012614:
> 
>   iwlegacy: fix channel switch locking (2011-06-08 14:19:05 -0400)
> 
> are available in the git repository at:
>   master.kernel.org:/pub/scm/linux/kernel/git/padovan/bluetooth-2.6.git master
> 
> David Miller (2):

...

>       Bluetooth: Kill set but unused variable 'cmd' in cmtp_recv_capimsg()

Well, the patch submitter sounds familiar.  :-)  But killing an unused
variable isn't a bug fix.  It doesn't do any damage, but I suspect that
Dave expects to be held to the same standards as everyone else.

Can you drop that one (and reapply it for 3.1)?  Sorry for the trouble...

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: pull request: bluetooth-2.6 2011-06-08
  2011-06-09 17:59 ` John W. Linville
@ 2011-06-09 18:34   ` Gustavo F. Padovan
  2011-06-10 17:53     ` John W. Linville
  0 siblings, 1 reply; 5+ messages in thread
From: Gustavo F. Padovan @ 2011-06-09 18:34 UTC (permalink / raw)
  To: John W. Linville; +Cc: linux-wireless, linux-bluetooth

* John W. Linville <linville@tuxdriver.com> [2011-06-09 13:59:34 -0400]:

> On Wed, Jun 08, 2011 at 04:46:58PM -0300, Gustavo F. Padovan wrote:
> > Hi John,
> > 
> > First round of fixed for 3.0! There are here a regression fix for 1.1 and
> > earlier devices and a fix in the shutdown of SCO sockets.  Other 3 patches are
> > warning shut up. And one byte leak to userspace fix.
 > 
> > Please pull, thanks!
> > 
> > 
> > The following changes since commit 51e65257142a87fe46a1ce5c35c86c5baf012614:
> > 
> >   iwlegacy: fix channel switch locking (2011-06-08 14:19:05 -0400)
> > 
> > are available in the git repository at:
> >   master.kernel.org:/pub/scm/linux/kernel/git/padovan/bluetooth-2.6.git master
> > 
> > David Miller (2):
> 
> ...
> 
> >       Bluetooth: Kill set but unused variable 'cmd' in cmtp_recv_capimsg()
> 
> Well, the patch submitter sounds familiar.  :-)  But killing an unused
> variable isn't a bug fix.  It doesn't do any damage, but I suspect that
> Dave expects to be held to the same standards as everyone else.
> 
> Can you drop that one (and reapply it for 3.1)?  Sorry for the trouble...

Sorry for that, I tough it was not that bad fix this for 3.0. New pull request
follows. Thanks a lot!

The following changes since commit 51e65257142a87fe46a1ce5c35c86c5baf012614:

  iwlegacy: fix channel switch locking (2011-06-08 14:19:05 -0400)

are available in the git repository at:
  master.kernel.org:/pub/scm/linux/kernel/git/padovan/bluetooth-2.6.git master

David Miller (1):
      Bluetooth: Do not ignore errors returned from strict_strtol()

Filip Palian (1):
      Bluetooth: l2cap and rfcomm: fix 1 byte infoleak to userspace.

Luiz Augusto von Dentz (1):
      Bluetooth: fix shutdown on SCO sockets

Stephen Boyd (1):
      Bluetooth: Silence DEBUG_STRICT_USER_COPY_CHECKS=y warning

Ville Tervo (1):
      Bluetooth: Do not send SET_EVENT_MASK for 1.1 and earlier devices

 drivers/bluetooth/btmrvl_debugfs.c |   12 ++++++++++++
 net/bluetooth/hci_event.c          |   18 ++++++++++--------
 net/bluetooth/l2cap_sock.c         |    1 +
 net/bluetooth/rfcomm/sock.c        |    4 +++-
 net/bluetooth/sco.c                |   13 ++++++++++++-
 5 files changed, 38 insertions(+), 10 deletions(-)

	Gustavo

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: pull request: bluetooth-2.6 2011-06-08
  2011-06-09 18:34   ` Gustavo F. Padovan
@ 2011-06-10 17:53     ` John W. Linville
  2011-06-10 18:05       ` Gustavo F. Padovan
  0 siblings, 1 reply; 5+ messages in thread
From: John W. Linville @ 2011-06-10 17:53 UTC (permalink / raw)
  To: linux-wireless, linux-bluetooth

On Thu, Jun 09, 2011 at 03:34:31PM -0300, Gustavo F. Padovan wrote:
> * John W. Linville <linville@tuxdriver.com> [2011-06-09 13:59:34 -0400]:
> 
> > On Wed, Jun 08, 2011 at 04:46:58PM -0300, Gustavo F. Padovan wrote:

> > >       Bluetooth: Kill set but unused variable 'cmd' in cmtp_recv_capimsg()
> > 
> > Well, the patch submitter sounds familiar.  :-)  But killing an unused
> > variable isn't a bug fix.  It doesn't do any damage, but I suspect that
> > Dave expects to be held to the same standards as everyone else.
> > 
> > Can you drop that one (and reapply it for 3.1)?  Sorry for the trouble...
> 
> Sorry for that, I tough it was not that bad fix this for 3.0. New pull request
> follows. Thanks a lot!

> Stephen Boyd (1):
>       Bluetooth: Silence DEBUG_STRICT_USER_COPY_CHECKS=y warning

Sorry, I missed this one too -- can you redirect it for 3.1 as well?

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: pull request: bluetooth-2.6 2011-06-08
  2011-06-10 17:53     ` John W. Linville
@ 2011-06-10 18:05       ` Gustavo F. Padovan
  0 siblings, 0 replies; 5+ messages in thread
From: Gustavo F. Padovan @ 2011-06-10 18:05 UTC (permalink / raw)
  To: John W. Linville; +Cc: linux-wireless, linux-bluetooth

* John W. Linville <linville@tuxdriver.com> [2011-06-10 13:53:03 -0400]:

> On Thu, Jun 09, 2011 at 03:34:31PM -0300, Gustavo F. Padovan wrote:
> > * John W. Linville <linville@tuxdriver.com> [2011-06-09 13:59:34 -0400]:
> > 
> > > On Wed, Jun 08, 2011 at 04:46:58PM -0300, Gustavo F. Padovan wrote:
> 
> > > >       Bluetooth: Kill set but unused variable 'cmd' in cmtp_recv_capimsg()
> > > 
> > > Well, the patch submitter sounds familiar.  :-)  But killing an unused
> > > variable isn't a bug fix.  It doesn't do any damage, but I suspect that
> > > Dave expects to be held to the same standards as everyone else.
> > > 
> > > Can you drop that one (and reapply it for 3.1)?  Sorry for the trouble...
> > 
> > Sorry for that, I tough it was not that bad fix this for 3.0. New pull request
> > follows. Thanks a lot!
> 
> > Stephen Boyd (1):
> >       Bluetooth: Silence DEBUG_STRICT_USER_COPY_CHECKS=y warning
> 
> Sorry, I missed this one too -- can you redirect it for 3.1 as well?

The following changes since commit 51e65257142a87fe46a1ce5c35c86c5baf012614:            
                                                                                        
  iwlegacy: fix channel switch locking (2011-06-08 14:19:05 -0400)                      
                                                                                        
are available in the git repository at:                                                 
  master.kernel.org:/pub/scm/linux/kernel/git/padovan/bluetooth-2.6.git master          
                                                                                        
David Miller (1):                                                                       
      Bluetooth: Do not ignore errors returned from strict_strtol()                     
                                                                                        
Filip Palian (1):                                                                       
      Bluetooth: l2cap and rfcomm: fix 1 byte infoleak to userspace.                    
                                                                                        
Luiz Augusto von Dentz (1):                                                             
      Bluetooth: fix shutdown on SCO sockets                                            
                                                                                        
Ville Tervo (1):                                                                        
      Bluetooth: Do not send SET_EVENT_MASK for 1.1 and earlier devices                 
                                                                                        
 drivers/bluetooth/btmrvl_debugfs.c |   12 ++++++++++++
 net/bluetooth/hci_event.c          |   18 ++++++++++--------
 net/bluetooth/l2cap_sock.c         |    1 +
 net/bluetooth/rfcomm/sock.c        |    1 +
 net/bluetooth/sco.c                |   13 ++++++++++++-
 5 files changed, 36 insertions(+), 9 deletions(-)

	Gustavo

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-06-10 18:05 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-08 19:46 pull request: bluetooth-2.6 2011-06-08 Gustavo F. Padovan
2011-06-09 17:59 ` John W. Linville
2011-06-09 18:34   ` Gustavo F. Padovan
2011-06-10 17:53     ` John W. Linville
2011-06-10 18:05       ` Gustavo F. Padovan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).