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
