* [PATCH] Bluetooth-next: Fixes l2cap "command reject" reply according to spec.
@ 2011-07-03 6:56 ilia.kolominsky
2011-07-06 7:13 ` Ilia, Kolominsky
2011-07-06 14:43 ` Gustavo Padovan
0 siblings, 2 replies; 3+ messages in thread
From: ilia.kolominsky @ 2011-07-03 6:56 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Ilia Kolomisnky
From: Ilia Kolomisnky <iliak@ti.com>
There can 3 reasons for the "command reject" reply produced
by the stack. Each such reply should be accompanied by the
relevand data ( as defined in spec. ). Currently there is one
instance of "command reject" reply with reason "invalid cid"
wich is fixed. Also, added clean-up definitions related to the
"command reject" replies.
Signed-off-by: Ilia Kolomisnky <iliak@ti.com>
---
include/net/bluetooth/l2cap.h | 19 ++++++++++++++++++-
net/bluetooth/l2cap_core.c | 15 +++++++++------
2 files changed, 27 insertions(+), 7 deletions(-)
diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h
index 9c18e55..6dee791 100644
--- a/include/net/bluetooth/l2cap.h
+++ b/include/net/bluetooth/l2cap.h
@@ -130,6 +130,12 @@ struct l2cap_conninfo {
#define L2CAP_SDU_END 0x8000
#define L2CAP_SDU_CONTINUE 0xC000
+/* L2CAP Command rej. reasons */
+#define L2CAP_CMD_NOT_UNDERSTOOD 0x0000
+#define L2CAP_CMD_MTU_EXCEEDED 0x0001
+#define L2CAP_CMD_INV_CID 0x0002
+
+
/* L2CAP structures */
struct l2cap_hdr {
__le16 len;
@@ -144,8 +150,19 @@ struct l2cap_cmd_hdr {
} __packed;
#define L2CAP_CMD_HDR_SIZE 4
-struct l2cap_cmd_rej {
+struct l2cap_cmd_rej_unk {
+ __le16 reason;
+} __packed;
+
+struct l2cap_cmd_rej_mtu {
+ __le16 reason;
+ __le16 max_mtu;
+} __packed;
+
+struct l2cap_cmd_rej_cid {
__le16 reason;
+ __le16 loc_cid;
+ __le16 rem_cid;
} __packed;
struct l2cap_conn_req {
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c
index bd5d992..951c311 100644
--- a/net/bluetooth/l2cap_core.c
+++ b/net/bluetooth/l2cap_core.c
@@ -2284,9 +2284,9 @@ done:
static inline int l2cap_command_rej(struct l2cap_conn *conn, struct l2cap_cmd_hdr *cmd, u8 *data)
{
- struct l2cap_cmd_rej *rej = (struct l2cap_cmd_rej *) data;
+ struct l2cap_cmd_rej_unk *rej = (struct l2cap_cmd_rej_unk *) data;
- if (rej->reason != 0x0000)
+ if (rej->reason != L2CAP_CMD_NOT_UNDERSTOOD)
return 0;
if ((conn->info_state & L2CAP_INFO_FEAT_MASK_REQ_SENT) &&
@@ -2531,9 +2531,12 @@ static inline int l2cap_config_req(struct l2cap_conn *conn, struct l2cap_cmd_hdr
sk = chan->sk;
if (chan->state != BT_CONFIG) {
- struct l2cap_cmd_rej rej;
+ struct l2cap_cmd_rej_cid rej;
+
+ rej.reason = cpu_to_le16(L2CAP_CMD_INV_CID);
+ rej.loc_cid = cpu_to_le16(chan->scid);
+ rej.rem_cid = cpu_to_le16(chan->dcid);
- rej.reason = cpu_to_le16(0x0002);
l2cap_send_cmd(conn, cmd->ident, L2CAP_COMMAND_REJ,
sizeof(rej), &rej);
goto unlock;
@@ -3024,12 +3027,12 @@ static inline void l2cap_sig_channel(struct l2cap_conn *conn,
err = l2cap_bredr_sig_cmd(conn, &cmd, cmd_len, data);
if (err) {
- struct l2cap_cmd_rej rej;
+ struct l2cap_cmd_rej_unk rej;
BT_ERR("Wrong link type (%d)", err);
/* FIXME: Map err to a valid reason */
- rej.reason = cpu_to_le16(0);
+ rej.reason = cpu_to_le16(L2CAP_CMD_NOT_UNDERSTOOD);
l2cap_send_cmd(conn, cmd.ident, L2CAP_COMMAND_REJ, sizeof(rej), &rej);
}
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [PATCH] Bluetooth-next: Fixes l2cap "command reject" reply according to spec.
2011-07-03 6:56 [PATCH] Bluetooth-next: Fixes l2cap "command reject" reply according to spec ilia.kolominsky
@ 2011-07-06 7:13 ` Ilia, Kolominsky
2011-07-06 14:43 ` Gustavo Padovan
1 sibling, 0 replies; 3+ messages in thread
From: Ilia, Kolominsky @ 2011-07-06 7:13 UTC (permalink / raw)
To: linux-bluetooth@vger.kernel.org
Cc: ilia.kolominsky@gmail.com, johan.hedberg@gmail.com
Ping
> -----Original Message-----
> From: ilia.kolominsky@gmail.com [mailto:ilia.kolominsky@gmail.com]
> Sent: Sunday, July 03, 2011 9:56 AM
> To: linux-bluetooth@vger.kernel.org
> Cc: Ilia, Kolominsky
> Subject: [PATCH] Bluetooth-next: Fixes l2cap "command reject" reply
> according to spec.
>
> From: Ilia Kolomisnky <iliak@ti.com>
>
> There can 3 reasons for the "command reject" reply produced
> by the stack. Each such reply should be accompanied by the
> relevand data ( as defined in spec. ). Currently there is one
> instance of "command reject" reply with reason "invalid cid"
> wich is fixed. Also, added clean-up definitions related to the
> "command reject" replies.
>
> Signed-off-by: Ilia Kolomisnky <iliak@ti.com>
> ---
> include/net/bluetooth/l2cap.h | 19 ++++++++++++++++++-
> net/bluetooth/l2cap_core.c | 15 +++++++++------
> 2 files changed, 27 insertions(+), 7 deletions(-)
>
> diff --git a/include/net/bluetooth/l2cap.h
> b/include/net/bluetooth/l2cap.h
> index 9c18e55..6dee791 100644
> --- a/include/net/bluetooth/l2cap.h
> +++ b/include/net/bluetooth/l2cap.h
> @@ -130,6 +130,12 @@ struct l2cap_conninfo {
> #define L2CAP_SDU_END 0x8000
> #define L2CAP_SDU_CONTINUE 0xC000
>
> +/* L2CAP Command rej. reasons */
> +#define L2CAP_CMD_NOT_UNDERSTOOD 0x0000
> +#define L2CAP_CMD_MTU_EXCEEDED 0x0001
> +#define L2CAP_CMD_INV_CID 0x0002
> +
> +
> /* L2CAP structures */
> struct l2cap_hdr {
> __le16 len;
> @@ -144,8 +150,19 @@ struct l2cap_cmd_hdr {
> } __packed;
> #define L2CAP_CMD_HDR_SIZE 4
>
> -struct l2cap_cmd_rej {
> +struct l2cap_cmd_rej_unk {
> + __le16 reason;
> +} __packed;
> +
> +struct l2cap_cmd_rej_mtu {
> + __le16 reason;
> + __le16 max_mtu;
> +} __packed;
> +
> +struct l2cap_cmd_rej_cid {
> __le16 reason;
> + __le16 loc_cid;
> + __le16 rem_cid;
> } __packed;
>
> struct l2cap_conn_req {
> diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c
> index bd5d992..951c311 100644
> --- a/net/bluetooth/l2cap_core.c
> +++ b/net/bluetooth/l2cap_core.c
> @@ -2284,9 +2284,9 @@ done:
>
> static inline int l2cap_command_rej(struct l2cap_conn *conn, struct
> l2cap_cmd_hdr *cmd, u8 *data)
> {
> - struct l2cap_cmd_rej *rej = (struct l2cap_cmd_rej *) data;
> + struct l2cap_cmd_rej_unk *rej = (struct l2cap_cmd_rej_unk *)
> data;
>
> - if (rej->reason != 0x0000)
> + if (rej->reason != L2CAP_CMD_NOT_UNDERSTOOD)
> return 0;
>
> if ((conn->info_state & L2CAP_INFO_FEAT_MASK_REQ_SENT) &&
> @@ -2531,9 +2531,12 @@ static inline int l2cap_config_req(struct
> l2cap_conn *conn, struct l2cap_cmd_hdr
> sk = chan->sk;
>
> if (chan->state != BT_CONFIG) {
> - struct l2cap_cmd_rej rej;
> + struct l2cap_cmd_rej_cid rej;
> +
> + rej.reason = cpu_to_le16(L2CAP_CMD_INV_CID);
> + rej.loc_cid = cpu_to_le16(chan->scid);
> + rej.rem_cid = cpu_to_le16(chan->dcid);
>
> - rej.reason = cpu_to_le16(0x0002);
> l2cap_send_cmd(conn, cmd->ident, L2CAP_COMMAND_REJ,
> sizeof(rej), &rej);
> goto unlock;
> @@ -3024,12 +3027,12 @@ static inline void l2cap_sig_channel(struct
> l2cap_conn *conn,
> err = l2cap_bredr_sig_cmd(conn, &cmd, cmd_len, data);
>
> if (err) {
> - struct l2cap_cmd_rej rej;
> + struct l2cap_cmd_rej_unk rej;
>
> BT_ERR("Wrong link type (%d)", err);
>
> /* FIXME: Map err to a valid reason */
> - rej.reason = cpu_to_le16(0);
> + rej.reason = cpu_to_le16(L2CAP_CMD_NOT_UNDERSTOOD);
> l2cap_send_cmd(conn, cmd.ident, L2CAP_COMMAND_REJ,
> sizeof(rej), &rej);
> }
>
> --
> 1.7.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Bluetooth-next: Fixes l2cap "command reject" reply according to spec.
2011-07-03 6:56 [PATCH] Bluetooth-next: Fixes l2cap "command reject" reply according to spec ilia.kolominsky
2011-07-06 7:13 ` Ilia, Kolominsky
@ 2011-07-06 14:43 ` Gustavo Padovan
1 sibling, 0 replies; 3+ messages in thread
From: Gustavo Padovan @ 2011-07-06 14:43 UTC (permalink / raw)
To: ilia.kolominsky; +Cc: linux-bluetooth, Ilia Kolomisnky
Hi Ilia,
I may have missed this patch, sorry.
* ilia.kolominsky@gmail.com <ilia.kolominsky@gmail.com> [2011-07-03 09:56:14 +0300]:
> From: Ilia Kolomisnky <iliak@ti.com>
>
> There can 3 reasons for the "command reject" reply produced
> by the stack. Each such reply should be accompanied by the
> relevand data ( as defined in spec. ). Currently there is one
> instance of "command reject" reply with reason "invalid cid"
> wich is fixed. Also, added clean-up definitions related to the
> "command reject" replies.
>
> Signed-off-by: Ilia Kolomisnky <iliak@ti.com>
> ---
> include/net/bluetooth/l2cap.h | 19 ++++++++++++++++++-
> net/bluetooth/l2cap_core.c | 15 +++++++++------
> 2 files changed, 27 insertions(+), 7 deletions(-)
>
> diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h
> index 9c18e55..6dee791 100644
> --- a/include/net/bluetooth/l2cap.h
> +++ b/include/net/bluetooth/l2cap.h
> @@ -130,6 +130,12 @@ struct l2cap_conninfo {
> #define L2CAP_SDU_END 0x8000
> #define L2CAP_SDU_CONTINUE 0xC000
>
> +/* L2CAP Command rej. reasons */
> +#define L2CAP_CMD_NOT_UNDERSTOOD 0x0000
> +#define L2CAP_CMD_MTU_EXCEEDED 0x0001
s/CMD/REJ
> +#define L2CAP_CMD_INV_CID 0x0002
#define L2CAP_REJ_INVALID_CID
> +
> +
> /* L2CAP structures */
> struct l2cap_hdr {
> __le16 len;
> @@ -144,8 +150,19 @@ struct l2cap_cmd_hdr {
> } __packed;
> #define L2CAP_CMD_HDR_SIZE 4
>
> -struct l2cap_cmd_rej {
> +struct l2cap_cmd_rej_unk {
> + __le16 reason;
> +} __packed;
> +
> +struct l2cap_cmd_rej_mtu {
> + __le16 reason;
> + __le16 max_mtu;
> +} __packed;
> +
> +struct l2cap_cmd_rej_cid {
> __le16 reason;
> + __le16 loc_cid;
> + __le16 rem_cid;
call theses scid and dcid.
Gustavo
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-07-06 14:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-03 6:56 [PATCH] Bluetooth-next: Fixes l2cap "command reject" reply according to spec ilia.kolominsky
2011-07-06 7:13 ` Ilia, Kolominsky
2011-07-06 14:43 ` Gustavo Padovan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).