core: ignore devices that are not linked when choosing the default sink or source
authorTanu Kaskinen <tanuk@iki.fi>
Wed, 30 Aug 2017 18:51:15 +0000 (21:51 +0300)
committerTanu Kaskinen <tanuk@iki.fi>
Sat, 2 Sep 2017 11:14:49 +0000 (14:14 +0300)
Sources should probably be added to pa_core.sources in pa_source_put(),
but currently they're added in pa_source_new(). A lot of stuff can
happen between the pa_source_new() and pa_source_put() calls, and
it has happened that the default source was updated during this time.
Therefore, pa_core_update_default_source() needs to take it into account
that not every source is necessarily linked.

src/pulsecore/core.c

index e01677d..454c566 100644 (file)
@@ -315,6 +315,9 @@ void pa_core_update_default_sink(pa_core *core) {
     pa_assert(core);
 
     PA_IDXSET_FOREACH(sink, core->sinks, idx) {
+        if (!PA_SINK_IS_LINKED(sink->state))
+            continue;
+
         if (!best) {
             best = sink;
             continue;
@@ -399,6 +402,9 @@ void pa_core_update_default_source(pa_core *core) {
     pa_assert(core);
 
     PA_IDXSET_FOREACH(source, core->sources, idx) {
+        if (!PA_SOURCE_IS_LINKED(source->state))
+            continue;
+
         if (!best) {
             best = source;
             continue;