Re: [PATCH BlueZ] adapter: Fix segfault when icon is uninitialized

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

 



Hi Bruna,

On Mon, Apr 02, 2012, Bruna Moreira wrote:
> If device type is LE and GAP Appearance characteristic does not exist,
> the icon will not be initialized.
> ---
>  src/adapter.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/adapter.c b/src/adapter.c
> index f8f46f8..9ebfe66 100644
> --- a/src/adapter.c
> +++ b/src/adapter.c
> @@ -2664,7 +2664,7 @@ void adapter_emit_device_found(struct btd_adapter *adapter,
>  {
>  	struct btd_device *device;
>  	char peer_addr[18], local_addr[18];
> -	const char *icon, *paddr = peer_addr;
> +	const char *icon = NULL, *paddr = peer_addr;
>  	dbus_bool_t paired = FALSE, trusted = FALSE;
>  	dbus_int16_t rssi = dev->rssi;
>  	char *alias;

I'd rather do this within the code than by initializing upon
declaration:

                if (read_remote_appearance(&adapter->bdaddr, &dev->bdaddr,
                                                                &app) == 0)
                        icon = gap_appearance_to_icon(app);
+               else
+                       icon = NULL;


Johan
--
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