Re: [2.6.38-rc8, patch] fix hci_dev_list locking

From: Gustavo F. Padovan
Date: Mon Mar 14 2011 - 14:04:17 EST


Hi Daniel,

* Daniel J Blueman <daniel.blueman@xxxxxxxxx> [2011-03-13 11:59:27 +0800]:

> Hi Gustavo,
>
> On 12 March 2011 09:47, Gustavo F. Padovan <padovan@xxxxxxxxxxxxxx> wrote:
> > Hi Daniel,
> >
> > * Daniel J Blueman <daniel.blueman@xxxxxxxxx> [2011-03-09 00:07:54 +0800]:
> >
> >> No response from original post, and it missed getting taken into -rc7
> >> and -rc8, so repost:
> >>
> >> Release acquired lock on error path, fixing potential hang up.
> >
> > I already have a patch like this one in my tree, thanks anyway. :)
>
> I guess I'm not looking in the right public tree, but I still couldn't
> detect the fix in:
> http://git.kernel.org/?p=linux/kernel/git/padovan/bluetooth-2.6.git
> or:
> http://git.kernel.org/?p=linux/kernel/git/padovan/bluetooth-next-2.6.git

On bluetooth-next:

commit b2c60d42db0fea1e6c4345739601024863566a13
Author: Jesper Juhl <jj@xxxxxxxxxxxxx>
Date: Fri Jan 14 00:18:49 2011 +0100

Bluetooth: Fix failure to release lock in read_index_list()

If alloc_skb() fails in read_index_list() we'll return -ENOMEM without
releasing 'hci_dev_list_lock'.

Signed-off-by: Jesper Juhl <jj@xxxxxxxxxxxxx>
Signed-off-by: Gustavo F. Padovan <padovan@xxxxxxxxxxxxxx>

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