tools/mpris-proxy: unregister object path if player registration fails

The `owner' string, passed as user data, is freed if the player fails
registration, but the object path still exists. Upon program exiting,
the lingering path will be enumerated and the attached user data will be
freed again.

Signed-off-by: Manika Shrivastava <manika.sh@samsung.com>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>

diff --git a/tools/mpris-proxy.c b/tools/mpris-proxy.c
index 9f868e05c2a9cac95027a85c8f21242a4be036f9..5edce434b952444860517dbeac7e9f73935f7d89 100755 (executable)
--- a/tools/mpris-proxy.c
+++ b/tools/mpris-proxy.c
@@ -479,6 +479,7 @@ static void add_player(DBusConnection *conn, const char *name,
        reply = dbus_connection_send_with_reply_and_block(sys, msg, -1, &err);
        if (!reply) {
                fprintf(stderr, "Can't register player\n");
+               dbus_connection_unregister_object_path(sys, path);
                free(owner);
                if (dbus_error_is_set(&err)) {
                        fprintf(stderr, "%s\n", err.message);