sink: don't sync monitor suspend state when unlinking

When the sink is unlinked, there's no need to update the monitor suspend
state. In fact, trying to do that causes an assertion failure, because
pa_source_sync_suspend() wasn't written to handle the case where the
sink is unlinked.

Change-Id: I6352eee5ec9b5b59234b738997f9c055a33b8a76
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

diff --git a/src/pulsecore/sink.c b/src/pulsecore/sink.c
index e781660..fa729f9 100644 (file)
--- a/src/pulsecore/sink.c
+++ b/src/pulsecore/sink.c
@@ -560,7 +560,7 @@ static int sink_set_state(pa_sink *s, pa_sink_state_t state, pa_suspend_cause_t
     }
 
 finish:
-    if ((suspending || resuming || suspend_cause_changed) && s->monitor_source)
+    if ((suspending || resuming || suspend_cause_changed) && s->monitor_source && state != PA_SINK_UNLINKED)
         pa_source_sync_suspend(s->monitor_source);
 
     return ret;