From: Seungbae Shin <seungbae.shin@samsung.com>
Date: Mon, 13 Jul 2020 02:13:47 +0000 (+0900)
Subject: tizenaudio-publish: publish only built-in devices
X-Git-Tag: submit/tizen/20200814.044955^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2e2db377b4dfd8293a393ca8a8cb7a0e303f9807;p=platform%2Fcore%2Fmultimedia%2Fpulseaudio-modules-tizen.git

tizenaudio-publish: publish only built-in devices

[Version] 13.0.24
[Issue Type] Update

Change-Id: I69753da561c813c80c35df7690d4efc1efc48a43
---

diff --git a/packaging/pulseaudio-modules-tizen.spec b/packaging/pulseaudio-modules-tizen.spec
index 78eccb1..9324ca1 100644
--- a/packaging/pulseaudio-modules-tizen.spec
+++ b/packaging/pulseaudio-modules-tizen.spec
@@ -1,6 +1,6 @@
 Name:             pulseaudio-modules-tizen
 Summary:          Pulseaudio modules for Tizen
-Version:          13.0.23
+Version:          13.0.24
 Release:          0
 Group:            Multimedia/Audio
 License:          LGPL-2.1+
diff --git a/src/module-tizenaudio-publish.c b/src/module-tizenaudio-publish.c
index 480b98f..cdead84 100644
--- a/src/module-tizenaudio-publish.c
+++ b/src/module-tizenaudio-publish.c
@@ -329,14 +329,30 @@ static void service_free(struct service *s) {
     pa_xfree(s);
 }
 
+static bool is_builtin_device(pa_proplist *pl)
+{
+    /* FIXME: Determining the built-in device should be retrieved from device-manager.  */
+    return pa_safe_streq(pa_proplist_gets(pl, PA_PROP_DEVICE_FORM_FACTOR), "internal");
+}
+
 static bool shall_ignore(pa_object *o) {
     pa_object_assert_ref(o);
 
-    if (pa_sink_isinstance(o))
+    if (pa_sink_isinstance(o)) {
+        if (!is_builtin_device(PA_SINK(o)->proplist)) {
+            pa_log_error("sink [%s] is not an internal, skip this", PA_SINK(o)->name);
+            return true;
+        }
         return !!(PA_SINK(o)->flags & PA_SINK_NETWORK);
+    }
 
-    if (pa_source_isinstance(o))
+    if (pa_source_isinstance(o)) {
+        if (!is_builtin_device(PA_SOURCE(o)->proplist)) {
+            pa_log_error("source [%s] is not an internal, skip this", PA_SOURCE(o)->name);
+            return true;
+        }
         return PA_SOURCE(o)->monitor_of || (PA_SOURCE(o)->flags & PA_SOURCE_NETWORK);
+    }
 
     pa_assert_not_reached();
 }