Hi Mat, On Tue, Oct 16, 2012 at 04:36:18PM -0700, Mat Martineau wrote: > Processing a move channel request involves getting the channel > structure using the destination channel ID. Previous code could only > look up using the source channel ID. > > Signed-off-by: Mat Martineau <mathewm@xxxxxxxxxxxxxx> > --- > net/bluetooth/l2cap_core.c | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c > index ec2b4d9..b5b849b 100644 > --- a/net/bluetooth/l2cap_core.c > +++ b/net/bluetooth/l2cap_core.c > @@ -100,6 +100,22 @@ static struct l2cap_chan *l2cap_get_chan_by_scid(struct l2cap_conn *conn, > return c; > } > > +/* Find channel with given DCID. > + * Returns locked channel. */ > +static struct l2cap_chan *l2cap_get_chan_by_dcid(struct l2cap_conn *conn, > + u16 cid) > +{ > + struct l2cap_chan *c; > + > + mutex_lock(&conn->chan_lock); > + c = __l2cap_get_chan_by_dcid(conn, cid); > + if (c) > + l2cap_chan_lock(c); > + mutex_unlock(&conn->chan_lock); > + > + return c; > +} > + > static struct l2cap_chan *__l2cap_get_chan_by_ident(struct l2cap_conn *conn, > u8 ident) > { > @@ -4139,6 +4155,7 @@ static inline int l2cap_move_channel_req(struct l2cap_conn *conn, > u16 cmd_len, void *data) > { > struct l2cap_move_chan_req *req = data; > + struct l2cap_chan *chan; This line seems do not belong to the patch. Best regards Andrei Emeltchenko > u16 icid = 0; > u16 result = L2CAP_MR_NOT_ALLOWED; -- 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