Re: [PATCH v2 2/2] media: register disconnect watch at transport add

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

 



Hi Frederic,

2012/2/21 Frédéric Dalleau <frederic.dalleau@xxxxxxxxxxxxxxx>:
> disconnect watch is not removed when the media owner is freed if
> an error occurs while resuming in acquire (id == 0).
> ---
>  audio/transport.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/audio/transport.c b/audio/transport.c
> index b319e0e..7bf7309 100644
> --- a/audio/transport.c
> +++ b/audio/transport.c
> @@ -599,6 +599,9 @@ static void media_transport_add(struct media_transport *transport,
>        DBG("Transport %s Owner %s", transport->path, owner->name);
>        transport->owners = g_slist_append(transport->owners, owner);
>        owner->transport = transport;
> +       owner->watch = g_dbus_add_disconnect_watch(transport->conn, owner->name,
> +                                                       media_owner_exit,
> +                                                       owner, NULL);
>  }
>
>  static struct media_owner *media_owner_create(DBusConnection *conn,
> @@ -610,9 +613,6 @@ static struct media_owner *media_owner_create(DBusConnection *conn,
>        owner = g_new0(struct media_owner, 1);
>        owner->name = g_strdup(dbus_message_get_sender(msg));
>        owner->accesstype = g_strdup(accesstype);
> -       owner->watch = g_dbus_add_disconnect_watch(conn, owner->name,
> -                                                       media_owner_exit,
> -                                                       owner, NULL);
>
>        DBG("Owner created: sender=%s accesstype=%s", owner->name,
>                        accesstype);
> --
> 1.7.5.4

Ack

-- 
Luiz Augusto von Dentz
--
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