* [PATCH 0/1 v2] Fix adapter drivers unload order
@ 2011-03-18 14:18 Dmitriy Paliy
2011-03-18 14:18 ` [PATCH] " Dmitriy Paliy
0 siblings, 1 reply; 2+ messages in thread
From: Dmitriy Paliy @ 2011-03-18 14:18 UTC (permalink / raw)
To: linux-bluetooth, hadess
Hi,
This patch includes corrections done based on Bastien's comments.
Thanks for reviewing.
BR,
Dmitriy
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH] Fix adapter drivers unload order
2011-03-18 14:18 [PATCH 0/1 v2] Fix adapter drivers unload order Dmitriy Paliy
@ 2011-03-18 14:18 ` Dmitriy Paliy
0 siblings, 0 replies; 2+ messages in thread
From: Dmitriy Paliy @ 2011-03-18 14:18 UTC (permalink / raw)
To: linux-bluetooth, hadess; +Cc: Dmitriy Paliy
Order of unloading drivers shell be preserved as given in
audio_manager_exit. It was reverted in 'c772636 Fix unloading of adapter
drivers', which causes in turn bluetoothd crash on exit.
Removing of media end-point results in removing of A2DP end-point for
the corresponding A2DP server but not vice versa. Therefore, unregistering
and destroying A2DP server, which clears all A2DP end-points, before media
server is incorrect.
---
src/adapter.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/adapter.c b/src/adapter.c
index cc4f43e..3455237 100644
--- a/src/adapter.c
+++ b/src/adapter.c
@@ -2494,6 +2494,7 @@ static void remove_driver(gpointer data, gpointer user_data)
static void unload_drivers(struct btd_adapter *adapter)
{
+ adapter->loaded_drivers = g_slist_reverse(adapter->loaded_drivers);
g_slist_foreach(adapter->loaded_drivers, remove_driver, adapter);
g_slist_free(adapter->loaded_drivers);
adapter->loaded_drivers = NULL;
--
1.7.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-03-18 14:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-18 14:18 [PATCH 0/1 v2] Fix adapter drivers unload order Dmitriy Paliy
2011-03-18 14:18 ` [PATCH] " Dmitriy Paliy
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox