On Mon, 2013-09-16 at 13:06 +0200, Alexander Couzens wrote: > @@ -528,11 +529,14 @@ void pa__done(pa_module *m) { > pa_sink_unlink(u->sink); > > if (u->thread) { > - pa_asyncmsgq_send(u->thread_mq.inq, NULL, PA_MESSAGE_SHUTDOWN, NULL, 0, NULL); > + pa_asyncmsgq_send(u->thread_mq->inq, NULL, PA_MESSAGE_SHUTDOWN, NULL, 0, NULL); > pa_thread_free(u->thread); > } > > - pa_thread_mq_done(&u->thread_mq); > + if (u->thread_mq) { > + pa_thread_mq_done(u->thread_mq); > + pa_xfree(u->thread); This should be u->thread_mq, not u->thread. Otherwise looks good, I pushed now this and the other (1/2) patch. -- Tanu