Hi Gustavo, On 20:36 Wed 06 Apr, Gustavo F. Padovan wrote: > Hi Vinicius, > > * Vinicius Costa Gomes <vinicius.gomes@xxxxxxxxxxxxx> [2011-04-05 22:51:47 -0300]: > > > From: Anderson Briglia <anderson.briglia@xxxxxxxxxxxxx> > > > > This patch adds initial support for verifying the confirmation value > > that the remote side has sent. > > > > Signed-off-by: Anderson Briglia <anderson.briglia@xxxxxxxxxxxxx> > > Signed-off-by: Vinicius Costa Gomes <vinicius.gomes@xxxxxxxxxxxxx> > > --- > > include/net/bluetooth/l2cap.h | 5 +++++ > > net/bluetooth/smp.c | 17 +++++++++++++++++ > > 2 files changed, 22 insertions(+), 0 deletions(-) > > > > diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h > > index 2b9ca0d..dc4aa63 100644 > > --- a/include/net/bluetooth/l2cap.h > > +++ b/include/net/bluetooth/l2cap.h > > @@ -305,6 +305,11 @@ struct l2cap_conn { > > > > __u8 disc_reason; > > > > + __u8 preq[7]; /* SMP Pairing Request */ > > + __u8 prsp[7]; /* SMP Pairing Response */ > > + __u8 prnd[16]; /* SMP Pairing Random */ > > + __u8 pcnf[16]; /* SMP Pairing Confirm */ > > + > > struct l2cap_chan_list chan_list; > > }; > > > > diff --git a/net/bluetooth/smp.c b/net/bluetooth/smp.c > > index 58047e8..7fa3542 100644 > > --- a/net/bluetooth/smp.c > > +++ b/net/bluetooth/smp.c > > @@ -192,6 +192,8 @@ static void smp_cmd_pairing_req(struct l2cap_conn *conn, struct sk_buff *skb) > > > > BT_DBG("conn %p", conn); > > > > + conn->preq[0] = SMP_CMD_PAIRING_REQ; > > + memcpy(&conn->preq[1], rp, sizeof(*rp)); > > skb_pull(skb, sizeof(*rp)); > > > > rp->io_capability = 0x00; > > @@ -201,17 +203,25 @@ static void smp_cmd_pairing_req(struct l2cap_conn *conn, struct sk_buff *skb) > > rp->resp_key_dist = 0x00; > > rp->auth_req &= (SMP_AUTH_BONDING | SMP_AUTH_MITM); > > > > + conn->prsp[0] = SMP_CMD_PAIRING_RSP; > > + memcpy(&conn->prsp[1], rp, sizeof(*rp)); > > + > > smp_send_cmd(conn, SMP_CMD_PAIRING_RSP, sizeof(*rp), rp); > > } > > > > static void smp_cmd_pairing_rsp(struct l2cap_conn *conn, struct sk_buff *skb) > > { > > + struct smp_cmd_pairing *rp = (void *) skb->data; > > No need for cast here. Yes, the cast is needed. The pointers types are incompatible. > > -- > Gustavo F. Padovan > http://profusion.mobi Cheers, -- Vinicius -- 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