I will resend it shortly. Sorry. On Tue, Dec 22, 2009 at 3:29 PM, Andrei Emeltchenko <andrei.emeltchenko.news@xxxxxxxxx> wrote: > From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > > Trivial fix for double free of skb pointer with kfree_skb to > make code simplier and cleaner. > > skb is already freed within l2cap_data_channel_iframe function. > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > --- > net/bluetooth/l2cap.c | 4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) > > diff --git a/net/bluetooth/l2cap.c b/net/bluetooth/l2cap.c > index 1120cf1..9584b97 100644 > --- a/net/bluetooth/l2cap.c > +++ b/net/bluetooth/l2cap.c > @@ -3574,9 +3574,7 @@ static inline int l2cap_data_channel(struct > l2cap_conn *conn, u16 cid, struct sk > else > err = l2cap_data_channel_sframe(sk, control, skb); > > - if (!err) > - goto done; > - break; > + goto done; > > case L2CAP_MODE_STREAMING: > control = get_unaligned_le16(skb->data); > -- > 1.6.0.4 > -- 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
