Fix Coverity issues

Change-Id: Ie63d53ed8665b43457a8a6201bdf909c9daef264

diff --git a/svc/media_controller_db_util.c b/svc/media_controller_db_util.c
index d7080bf72a7110f02d1f125c93bd8100167e21b0..88d263609118d0ad17c34ba68fe133c9ad508b12 100644 (file)
--- a/svc/media_controller_db_util.c
+++ b/svc/media_controller_db_util.c
@@ -527,6 +527,7 @@ int mc_db_util_init_latest_server_table(void *handle)
                if (MC_STRING_VALID((const char *)sqlite3_column_text(stmt, 1))) {
                        latest_server_name = strdup((const char *)sqlite3_column_text(stmt, 1));
                        mc_debug("latest_server_name: %s", latest_server_name);
+                       break;
                }
                ret = sqlite3_step(stmt);
        }
@@ -538,6 +539,7 @@ int mc_db_util_init_latest_server_table(void *handle)
 
        /* Update server_state and playback_state to 0 */
        sql_str = sqlite3_mprintf("UPDATE '%q' SET server_state=%d, playback_state=%d;", latest_server_name, MC_SERVER_STATE_DEACTIVATE, MC_PLAYBACK_STATE_STOPPED);
+       MC_SAFE_FREE(latest_server_name);
        mc_retvm_if(!MC_STRING_VALID(sql_str), MEDIA_CONTROLLER_ERROR_INVALID_OPERATION, "SQL string is null");
 
        ret = mc_db_util_update_db(handle, sql_str);

diff --git a/test/server_test/media_controller_server_test.c b/test/server_test/media_controller_server_test.c
index 9801bd9680ecc849c3a12ac5bb45600771826020..692e7917c4dde6cd092d5dcd1bde5df192d2d8a9 100755 (executable)
--- a/test/server_test/media_controller_server_test.c
+++ b/test/server_test/media_controller_server_test.c
@@ -343,6 +343,8 @@ static int __update_playlist(mc_server_h server, mc_playlist_update_mode_e mode)
                        mc_debug("Get playlist [%s]", playlist_name);
 
                        MC_SAFE_FREE(playlist_name);
+
+                       ret = mc_playlist_destroy(playlist_2);
                }
 
        } else {