Consider as normal operation when destroy is executed before job function of data...

Change-Id: I59a9766e50cbe426d71814522e286aa5514dc19a

diff --git a/packaging/mused.spec b/packaging/mused.spec
index b127764..7565622 100644 (file)
--- a/packaging/mused.spec
+++ b/packaging/mused.spec
@@ -1,6 +1,6 @@
 Name:       mused
 Summary:    A multimedia daemon
-Version:    0.3.91
+Version:    0.3.92
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0 and BSD-3-Clause

diff --git a/server/src/muse_server_ipc.c b/server/src/muse_server_ipc.c
index 37e79a6..7eec7f2 100644 (file)
--- a/server/src/muse_server_ipc.c
+++ b/server/src/muse_server_ipc.c
@@ -459,7 +459,11 @@ gboolean ms_ipc_data_job_function(ms_workqueue_job_t *job)
        muse_return_val_if_fail(ms_is_server_ready(), FALSE);
 
        m = (muse_module_h)job->user_data;
-       muse_return_val_if_fail(muse_server_module_is_valid(m), FALSE);
+       if (!muse_server_module_is_valid(m)) {
+               free(job);
+               LOGW("destroy is already called");
+               return TRUE;
+       }
 
        m->ch[MUSE_CHANNEL_DATA].thread = g_thread_try_new(DATA_THREAD_NAME, _ms_ipc_data_worker, (gpointer)m, &error);
        if (!m->ch[MUSE_CHANNEL_DATA].thread && error) {