From a86eae7fef9819317f591ee11b6e140c0c424f94 Mon Sep 17 00:00:00 2001 From: Arik Nemtsov Date: Thu, 8 Mar 2012 15:44:39 +0200 Subject: [PATCH] Revert "core: Add provision for terminating an ATT connection" This reverts commit f89a77478af78d41c80ab7605662382b9e4e1c36. This is not needed and actually introduces a bug. When the "Disconnect" API of device is called device->attrib is unref-ed via a watch set on G_IO_HUP. The channel is shutdown when the last reference is removed. The code introduced here shuts down the channel and prevents the watch from getting called. This means we leak a reference to device->attrib. This can cause a number of bad things. For example, if the device is temporary, it will never be freed, and we won't be able to pair to it again. --- src/device.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/device.c b/src/device.c index b339ac1c2..0a1de7c28 100644 --- a/src/device.c +++ b/src/device.c @@ -821,14 +821,6 @@ void device_request_disconnect(struct btd_device *device, DBusMessage *msg) browse_request_cancel(device->browse); } - if (device->attrib) { - GIOChannel *io = g_attrib_get_channel(device->attrib); - if (io) { - g_io_channel_shutdown(io, FALSE, NULL); - g_io_channel_unref(io); - } - } - if (msg) device->disconnects = g_slist_append(device->disconnects, dbus_message_ref(msg)); -- 2.47.3