Fix coverity issue of OVERRUN and UNUSED_VALUE

Change-Id: I62c2a7a4c46bd5fb77fef1b89dd7731ea2013c05

diff --git a/core/src/muse_core.c b/core/src/muse_core.c
index af4ae9a..2cd55da 100644 (file)
--- a/core/src/muse_core.c
+++ b/core/src/muse_core.c
@@ -132,8 +132,15 @@ out:
 
 static gboolean _muse_msg_is_mem_ptr_valid(gpointer ptr)
 {
-       size_t page_size = sysconf(_SC_PAGESIZE);
-       gpointer base = (gpointer)((size_t)ptr / page_size * page_size);
+       gpointer base;
+       long page_size = sysconf(_SC_PAGESIZE);
+
+       if (page_size < 0) {
+               LOGE("sysconf is failed errno (%d)", errno);
+               return FALSE;
+       }
+
+       base = (gpointer)((size_t)ptr / page_size * page_size);
 
        return msync(base, page_size, MS_ASYNC) == 0;
 }

diff --git a/packaging/mused.spec b/packaging/mused.spec
index 65ea6b8..771d87f 100644 (file)
--- a/packaging/mused.spec
+++ b/packaging/mused.spec
@@ -1,6 +1,6 @@
 Name:       mused
 Summary:    A multimedia daemon
-Version:    0.3.166
+Version:    0.3.167
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0

diff --git a/server/src/muse_server_ipc.c b/server/src/muse_server_ipc.c
index 4967b76..18844cf 100644 (file)
--- a/server/src/muse_server_ipc.c
+++ b/server/src/muse_server_ipc.c
@@ -302,7 +302,6 @@ static gpointer _ms_ipc_dispatch_worker(gpointer data)
                                }
                        }
                        muse_core_msg_object_free(jobj);
-                       jobj = NULL;
                        m->msg_offset += parse_len;
                        parse_len = len - parse_len;
                }