2011-12-05 11:54:45

by Santiago Carot

[permalink] [raw]
Subject: [PATCH] btio: Fix byte order conversion when getting L2CAP CID

The L2CAP CID is passed and received in little endian byte order
through the socket interface so a conversion is in place before
passing it onwards.
---
btio/btio.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/btio/btio.c b/btio/btio.c
index f8c5cc7..0d177a3 100644
--- a/btio/btio.c
+++ b/btio/btio.c
@@ -886,7 +886,7 @@ static gboolean l2cap_get(int sock, GError **err, BtIOOption opt1,
break;
case BT_IO_OPT_CID:
*(va_arg(args, uint16_t *)) = src.l2_cid ?
- src.l2_cid : dst.l2_cid;
+ btohs(src.l2_cid) : btohs(dst.l2_cid);
break;
case BT_IO_OPT_OMTU:
*(va_arg(args, uint16_t *)) = l2o.omtu;
--
1.7.8



2011-12-08 12:23:40

by Johan Hedberg

[permalink] [raw]
Subject: Re: [PATCH] btio: Fix byte order conversion when getting L2CAP CID

Hi Santiago,

On Mon, Dec 05, 2011, Santiago Carot-Nemesio wrote:
> The L2CAP CID is passed and received in little endian byte order
> through the socket interface so a conversion is in place before
> passing it onwards.
> ---
> btio/btio.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)

Applied. Thanks.

Johan