Fix coverity issues

[platform/core/api/media-content.git] / test / media-content_test.c

diff --git a/test/media-content_test.c b/test/media-content_test.c
index a14fa93..63d0474 100755 (executable)
--- a/test/media-content_test.c
+++ b/test/media-content_test.c
@@ -267,10 +267,14 @@ bool gallery_folder_list_cb(media_folder_h folder, void *user_data)
 bool gallery_media_item_cb(media_info_h media, void *user_data)
 {
        media_info_h new_media = NULL;
-       media_info_clone(&new_media, media);
+       int ret = MEDIA_CONTENT_ERROR_NONE;
 
-       GList **list = (GList**)user_data;
-       *list = g_list_append(*list, new_media);
+       ret = media_info_clone(&new_media, media);
+
+       if (ret != MEDIA_CONTENT_ERROR_NONE) {
+               GList **list = (GList**)user_data;
+               *list = g_list_append(*list, new_media);
+       }
 
        return true;
 }
@@ -559,6 +563,7 @@ bool folder_list_cb(media_folder_h folder, void *user_data)
                media_content_debug("folder_id = [%s]", folder_id);
 
                if (media_folder_get_parent_folder_id(folder, &str_val) != MEDIA_CONTENT_ERROR_NONE) {
+                       SAFE_FREE(folder_id);
                        media_content_error("[ERROR] media_folder_get_parent_folder_id is failed");
                        return false;
                }
@@ -593,6 +598,7 @@ bool folder_list_cb(media_folder_h folder, void *user_data)
                        return false;
                }
                media_content_debug("storage_id = [%s]", str_val);
+               SAFE_FREE(str_val);
 
                if (media_folder_get_order(folder, &int_val) != MEDIA_CONTENT_ERROR_NONE) {
                        media_content_error("[ERROR] media_folder_get_order is failed");
@@ -668,6 +674,7 @@ bool test_album_from_db(int album_id)
 
 bool playlist_list_cb(media_playlist_h playlist, void *user_data)
 {
+       int ret = MEDIA_CONTENT_ERROR_NONE;
        int playlist_id = 0;
        char *playlist_name = NULL;
        media_playlist_h playlist_h;
@@ -683,24 +690,37 @@ bool playlist_list_cb(media_playlist_h playlist, void *user_data)
                return false;
        }
 
-       media_playlist_get_playlist_id(playlist, &playlist_id);
+       ret = media_playlist_get_playlist_id(playlist, &playlist_id);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("error media_playlist_get_playlist_id : [%d]", ret);
+
        media_content_debug("playlist_id : %d", playlist_id);
        /* 64bit build issue */
 #if 0
        if (user_data != NULL)
                *list = g_list_append(*list, (gpointer)playlist_id);
 #endif
-       media_playlist_get_name(playlist, &playlist_name);
+       ret = media_playlist_get_name(playlist, &playlist_name);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("error media_playlist_get_name : [%d]", ret);
+
        media_content_debug("playlist_name : %s", playlist_name);
        SAFE_FREE(playlist_name);
 
-       media_playlist_get_thumbnail_path(playlist, &playlist_thumbnail_path);
+       ret = media_playlist_get_thumbnail_path(playlist, &playlist_thumbnail_path);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("error media_playlist_get_thumbnail_path : [%d]", ret);
+
        media_content_debug("playlist_thumbnail_path : %s", playlist_thumbnail_path);
        SAFE_FREE(playlist_thumbnail_path);
 
-       media_playlist_get_playlist_from_db(playlist_id, &playlist_h);
+       ret = media_playlist_get_playlist_from_db(playlist_id, &playlist_h);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("error media_playlist_get_playlist_from_db : [%d]", ret);
 
-       media_playlist_destroy(playlist_h);
+       ret = media_playlist_destroy(playlist_h);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("error media_playlist_destroy : [%d]", ret);
 
        return true;
 }
@@ -2060,6 +2080,9 @@ int test_tag_operation(void)
        ret = media_tag_destroy(tag_2);
        if (ret != MEDIA_CONTENT_ERROR_NONE)
                media_content_error("error media_tag_destroy : [%d]", ret);
+       ret = media_tag_destroy(tag_3);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("error media_tag_destroy : [%d]", ret);
        ret = media_filter_destroy(filter);
        if (ret != MEDIA_CONTENT_ERROR_NONE)
                media_content_error("error media_filter_destroy : [%d]", ret);
@@ -2173,6 +2196,8 @@ int test_bookmark_operation(void)
                        media_content_error("error media_bookmark_set_name : [%d]", ret);
 
                ret = media_bookmark_update_to_db(bookmark_handle);
+               if (ret != MEDIA_CONTENT_ERROR_NONE)
+                       media_content_error("error media_bookmark_update_to_db : [%d]", ret);
        }
 
        ret = media_info_foreach_bookmark_from_db(test_video_id, NULL, bookmarks_cb, NULL);
@@ -2298,6 +2323,8 @@ int test_group_operation(void)
                }
 
                ret = media_group_foreach_group_from_db(g_filter, idx, group_list_cb, &idx);
+               if (ret != MEDIA_CONTENT_ERROR_NONE)
+                       media_content_error("media_group_foreach_group_from_db failed: %d", ret);
        }
        ret = test_filter_destroy();
 
@@ -2681,6 +2708,7 @@ void face_detection_complete_cb(media_content_error_e error, int count, void *us
                        media_content_error("media_info_get_media_id failed: %d", ret);
                if (media_id != NULL) {
                        ret = media_info_foreach_face_from_db(media_id, NULL, face_cb, NULL);
+                       SAFE_FREE(media_id);
                        if (ret != MEDIA_CONTENT_ERROR_NONE)
                                media_content_error("media_info_foreach_face_from_db failed: %d", ret);
                }
@@ -3659,11 +3687,24 @@ int test_face_add_del(void)
        media_content_retvm_if(ret != MEDIA_CONTENT_ERROR_NONE, ret, "fail media_face_create");
 
        ret = media_face_set_face_rect(face, 10, 12, 50, 100);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("media_face_set_face_rect failed: %d", ret);
+
        ret = media_face_set_orientation(face, 5);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("media_face_set_orientation failed: %d", ret);
+
        ret = media_face_set_tag(face, face_tag);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("media_face_set_tag failed: %d", ret);
 
        ret = media_face_insert_to_db(face);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("media_face_insert_to_db failed: %d", ret);
+
        ret = media_face_destroy(face);
+       if (ret != MEDIA_CONTENT_ERROR_NONE)
+               media_content_error("media_face_destroy failed: %d", ret);
 
        /* Update Test */
        GList *all_item_list = NULL;
@@ -3685,9 +3726,17 @@ int test_face_add_del(void)
                face_handle = (media_face_h)g_list_nth_data(all_item_list, 0);
 
                ret = media_face_set_face_rect(face_handle, 20, 22, 70, 70);
+               if (ret != MEDIA_CONTENT_ERROR_NONE)
+                       media_content_error("media_face_set_face_rect failed: %d", ret);
                ret = media_face_set_orientation(face_handle, 3);
+               if (ret != MEDIA_CONTENT_ERROR_NONE)
+                       media_content_error("media_face_set_orientation failed: %d", ret);
                ret = media_face_set_tag(face_handle, NULL);
+               if (ret != MEDIA_CONTENT_ERROR_NONE)
+                       media_content_error("media_face_set_tag failed: %d", ret);
                ret = media_face_update_to_db(face_handle);
+               if (ret != MEDIA_CONTENT_ERROR_NONE)
+                       media_content_error("media_face_update_to_db failed: %d", ret);
 
                media_face_destroy(face_handle);
        }