Re: [PATCH BlueZ] Fix obex protocol for powerpc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



opps, sorry, I sent this new patch a lot too quickly...

uint16_t is unsigned, and can't accept -1...
I need to rework on it

Fabien Proriol


On 28/07/2016 15:12, fabien.proriolpatch@xxxxxxxxx wrote:
> From: Fabien Proriol <fabien.proriol@xxxxxxxx>
>
> omtu and imtu is define as uint16_t in l2cap_get function (use with va_arg).
> ---
>  obexd/client/bluetooth.c  | 4 ++--
>  obexd/plugins/bluetooth.c | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/obexd/client/bluetooth.c b/obexd/client/bluetooth.c
> index be007de..3cc1f90 100644
> --- a/obexd/client/bluetooth.c
> +++ b/obexd/client/bluetooth.c
> @@ -443,8 +443,8 @@ static int bluetooth_getpacketopt(GIOChannel *io, int *tx_mtu, int *rx_mtu)
>  {
>  	int sk = g_io_channel_unix_get_fd(io);
>  	int type;
> -	int omtu = -1;
> -	int imtu = -1;
> +	uint16_t omtu = -1;
> +	uint16_t imtu = -1;
>  	socklen_t len = sizeof(int);
>  
>  	DBG("");
> diff --git a/obexd/plugins/bluetooth.c b/obexd/plugins/bluetooth.c
> index d8b872a..3ee5432 100644
> --- a/obexd/plugins/bluetooth.c
> +++ b/obexd/plugins/bluetooth.c
> @@ -75,8 +75,8 @@ static void connect_event(GIOChannel *io, GError *err, void *user_data)
>  	struct bluetooth_profile *profile = user_data;
>  	struct obex_server *server = profile->server;
>  	int type;
> -	int omtu = BT_TX_MTU;
> -	int imtu = BT_RX_MTU;
> +	uint16_t omtu = BT_TX_MTU;
> +	uint16_t imtu = BT_RX_MTU;
>  	gboolean stream = TRUE;
>  	socklen_t len = sizeof(int);
>  
--
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



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux