Return-Path: MIME-Version: 1.0 In-Reply-To: <1295258917-31092-1-git-send-email-segoon@openwall.com> References: <1295258917-31092-1-git-send-email-segoon@openwall.com> Date: Mon, 17 Jan 2011 12:28:29 +0200 Message-ID: Subject: Re: [PATCH] net: bluetooth: fix locking problem From: Andrei Emeltchenko To: Vasiliy Kulikov Cc: kernel-janitors@vger.kernel.org, Marcel Holtmann , "Gustavo F. Padovan" , "David S. Miller" , linux-bluetooth@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 List-ID: On Mon, Jan 17, 2011 at 12:08 PM, Vasiliy Kulikov wro= te: > If alloc_skb() failed we still hold hci_dev_list_lock. =A0The code should > unlock it before exit. > > Signed-off-by: Vasiliy Kulikov > --- > =A0Compile tested only. > > =A0net/bluetooth/mgmt.c | =A0 =A04 +++- > =A01 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > index f827fd9..ace8726 100644 > --- a/net/bluetooth/mgmt.c > +++ b/net/bluetooth/mgmt.c > @@ -111,8 +111,10 @@ static int read_index_list(struct sock *sk) > > =A0 =A0 =A0 =A0body_len =3D sizeof(*ev) + sizeof(*rp) + (2 * count); > =A0 =A0 =A0 =A0skb =3D alloc_skb(sizeof(*hdr) + body_len, GFP_ATOMIC); > - =A0 =A0 =A0 if (!skb) > + =A0 =A0 =A0 if (!skb) { > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 read_unlock(&hci_dev_list_lock); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0return -ENOMEM; > + =A0 =A0 =A0 } patch was send already on weekend > > =A0 =A0 =A0 =A0hdr =3D (void *) skb_put(skb, sizeof(*hdr)); > =A0 =A0 =A0 =A0hdr->opcode =3D cpu_to_le16(MGMT_EV_CMD_COMPLETE); > -- > 1.7.0.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" i= n > the body of a message to majordomo@vger.kernel.org > More majordomo info at =A0http://vger.kernel.org/majordomo-info.html > Please read the FAQ at =A0http://www.tux.org/lkml/ >