[0.6.134] fix svace issue

Change-Id: Ic260cca3ec05b3cc73876fbed997d4397f4d2073

diff --git a/packaging/libmm-player.spec b/packaging/libmm-player.spec
index 528952b737a87a7ac0076ffa4bb5b05c37a3a37b..21eedeed8247c21e9a430b1bacb054d4ca07a32c 100644 (file)
--- a/packaging/libmm-player.spec
+++ b/packaging/libmm-player.spec
@@ -1,6 +1,6 @@
 Name:       libmm-player
 Summary:    Multimedia Framework Player Library
-Version:    0.6.133
+Version:    0.6.134
 Release:    0
 Group:      Multimedia/Libraries
 License:    Apache-2.0

diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c
index 1c7298422961229e6c667bb682554a94ef5071df..ea975b27ceb4534280425d172a671c4f4756de6f 100644 (file)
--- a/src/mm_player_priv.c
+++ b/src/mm_player_priv.c
@@ -897,6 +897,10 @@ void __mmplayer_bus_msg_thread_destroy(MMHandleType hplayer)
        queue = player->bus_msg_q;
        while (!g_queue_is_empty(queue)) {
                msg = (GstMessage *)g_queue_pop_head(queue);
+               if (!msg) {
+                       LOGW("msg is null");
+                       continue;
+               }
                LOGW("remove remained %s msg", GST_MESSAGE_TYPE_NAME(msg));
                gst_message_unref(msg);
        }

diff --git a/src/mm_player_utils.c b/src/mm_player_utils.c
index ff32abe1e03f9d70b2e1316e173f36eddd977a1a..19f4414b2938ebdf5d3198310fc79e5805cfd9d7 100644 (file)
--- a/src/mm_player_utils.c
+++ b/src/mm_player_utils.c
@@ -309,16 +309,23 @@ util_get_cookie_list(const char *cookies)
        /* split */
        cookie_list = g_strsplit(temp, ";", 100);
 
+       if (!cookie_list) {
+               LOGE("failed to get cookie list");
+               goto EXIT;
+       }
+
        for (i = 0; i < g_strv_length(cookie_list); i++) {
                if (cookie_list[i]) {
                        if (strlen(cookie_list[i])) {
                                g_strstrip(cookie_list[i]);
                                SECURE_LOGD("cookie_list[%d] : %d[bytes] - %s \n", i, strlen(cookie_list[i]), cookie_list[i]);
-                       } else
+                       } else {
                                cookie_list[i][0] = '\0';
+                       }
                }
        }
 
+EXIT:
        if (temp)
                g_free(temp);
        temp = NULL;