a2dp: Handle remote SEP disappearing

[platform/upstream/bluez.git] / profiles / audio / avdtp.c

diff --git a/profiles/audio/avdtp.c b/profiles/audio/avdtp.c
index 9298ae1..5ae4501 100644 (file)
--- a/profiles/audio/avdtp.c
+++ b/profiles/audio/avdtp.c
@@ -343,8 +343,11 @@ struct avdtp_remote_sep {
        uint8_t media_type;
        struct avdtp_service_capability *codec;
        gboolean delay_reporting;
+       bool discovered;
        GSList *caps; /* of type struct avdtp_service_capability */
        struct avdtp_stream *stream;
+       avdtp_remote_sep_destroy_t destroy;
+       void *user_data;
 };
 
 struct avdtp_local_sep {
@@ -1332,6 +1335,32 @@ static void avdtp_sep_set_state(struct avdtp *session,
                stream_free(stream);
 }
 
+static void sep_free(gpointer data)
+{
+       struct avdtp_remote_sep *sep = data;
+
+       if (sep->destroy)
+               sep->destroy(sep->user_data);
+
+       g_slist_free_full(sep->caps, g_free);
+       g_free(sep);
+}
+
+static void remove_disappeared(void *data, void *user_data)
+{
+       struct avdtp_remote_sep *sep = data;
+       struct avdtp *session = user_data;
+
+       if (sep->discovered)
+               return;
+
+       DBG("seid %d disappeared", sep->seid);
+
+       session->seps = g_slist_remove(session->seps, sep);
+
+       sep_free(sep);
+}
+
 #ifdef TIZEN_FEATURE_BLUEZ_MODIFY
 void finalize_discovery(struct avdtp *session, int err)
 #else
@@ -1349,6 +1378,9 @@ static void finalize_discovery(struct avdtp *session, int err)
        if (discover->id > 0)
                g_source_remove(discover->id);
 
+       if (!err)
+               g_slist_foreach(session->seps, remove_disappeared, session);
+
        if (discover->cb)
                discover->cb(session, session->seps, err ? &avdtp_err : NULL,
                                                        discover->user_data);
@@ -1373,14 +1405,6 @@ static void release_stream(struct avdtp_stream *stream, struct avdtp *session)
        avdtp_sep_set_state(session, sep, AVDTP_STATE_IDLE);
 }
 
-static void sep_free(gpointer data)
-{
-       struct avdtp_remote_sep *sep = data;
-
-       g_slist_free_full(sep->caps, g_free);
-       g_free(sep);
-}
-
 static void remove_disconnect_timer(struct avdtp *session)
 {
        if (!session->dc_timer)
@@ -3199,8 +3223,11 @@ static gboolean avdtp_discover_resp(struct avdtp *session,
 
                sep = find_remote_sep(session->seps, resp->seps[i].seid);
                if (sep && sep->type == resp->seps[i].type &&
-                               sep->media_type == resp->seps[i].media_type)
+                               sep->media_type == resp->seps[i].media_type &&
+                               sep->codec) {
+                       sep->discovered = true;
                        continue;
+               }
 
                if (resp->seps[i].inuse && !stream)
                        continue;
@@ -3212,6 +3239,7 @@ static gboolean avdtp_discover_resp(struct avdtp *session,
                sep->seid = resp->seps[i].seid;
                sep->type = resp->seps[i].type;
                sep->media_type = resp->seps[i].media_type;
+               sep->discovered = true;
 
                memset(&req, 0, sizeof(req));
                req.acp_seid = sep->seid;
@@ -3801,6 +3829,16 @@ struct avdtp_remote_sep *avdtp_register_remote_sep(struct avdtp *session,
        return sep;
 }
 
+void avdtp_remote_sep_set_destroy(struct avdtp_remote_sep *sep, void *user_data,
+                                       avdtp_remote_sep_destroy_t destroy)
+{
+       if (!sep)
+               return;
+
+       sep->user_data = user_data;
+       sep->destroy = destroy;
+}
+
 static gboolean process_discover(gpointer data)
 {
        struct avdtp *session = data;