avdtp: Fix removing all remote SEPs when loading from cache

If avdtp_discover is called after cache has been loaded it end up
removing all remote SEPs as they have not been discovered yet.

Fixes: https://github.com/bluez/bluez/issues/102
Signed-off-by: Anuj Jain <anuj01.jain@samsung.com>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>

diff --git a/profiles/audio/avdtp.c b/profiles/audio/avdtp.c
index eab333d..58f5e5a 100644 (file)
--- a/profiles/audio/avdtp.c
+++ b/profiles/audio/avdtp.c
@@ -3906,10 +3906,18 @@ int avdtp_discover(struct avdtp *session, avdtp_discover_cb_t cb,
        session->discover = g_new0(struct discover_callback, 1);
 
        if (session->seps) {
-               session->discover->cb = cb;
-               session->discover->user_data = user_data;
-               session->discover->id = g_idle_add(process_discover, session);
-               return 0;
+               struct avdtp_remote_sep *sep = session->seps->data;
+
+               /* Check that SEP have been discovered as it may be loaded from
+                * cache.
+                */
+               if (sep->discovered) {
+                       session->discover->cb = cb;
+                       session->discover->user_data = user_data;
+                       session->discover->id = g_idle_add(process_discover,
+                                                               session);
+                       return 0;
+               }
        }
 
        err = send_request(session, FALSE, NULL, AVDTP_DISCOVER, NULL, 0);