apply content_retip_if_fail() for invalid parameters
[platform/core/api/media-content.git] / src / media_info.c
index 3cb6688..5c21f8a 100644 (file)
@@ -33,7 +33,7 @@ static int __media_info_get_media_path_by_id_from_db(const char *media_id, char
        sqlite3_stmt *stmt = NULL;
        char *select_query = NULL;
 
-       content_retvm_if(!STRING_VALID(media_id), MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid media_id");
+       content_retip_if_fail(STRING_VALID(media_id));
 
        select_query = sqlite3_mprintf(SELECT_MEDIA_PATH_BY_ID, media_id);
 
@@ -242,7 +242,7 @@ static int __media_info_check_file_validity(const char *path)
        char *folder_path = NULL;
        int ret = MEDIA_CONTENT_ERROR_NONE;
 
-       content_retvm_if(!STRING_VALID(path), MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid path");
+       content_retip_if_fail(path);
 
        content_retvm_if(_media_util_is_ignorable_file(path), MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "Invalid ignore path");
 
@@ -354,7 +354,7 @@ int _media_info_get_media_info_from_db(const char *path, const char *storage_id,
        char *select_query = NULL;
        media_info_s *_media = (media_info_s*)media;
 
-       content_retvm_if(_media == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid media");
+       content_retip_if_fail(media);
 
        select_query = sqlite3_mprintf(SELECT_MEDIA_BY_PATH, storage_id, path);
 
@@ -381,7 +381,8 @@ int _media_info_get_media_info_from_db(const char *path, media_info_h media)
        char *select_query = NULL;
        media_info_s *_media = (media_info_s*)media;
 
-       content_retvm_if(_media == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid media");
+       content_retip_if_fail(path);
+       content_retip_if_fail(media);
 
        select_query = sqlite3_mprintf(SELECT_MEDIA_BY_PATH, path);
 
@@ -516,8 +517,8 @@ int media_info_get_media_info_by_path_from_db(const char* path, media_info_h* me
        int ret = MEDIA_CONTENT_ERROR_NONE;
        char storage_id[MEDIA_CONTENT_UUID_SIZE+1] = {0, };
 
-       content_retvm_if(!STRING_VALID(path), MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid path");
-       content_retvm_if(media == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid info");
+       content_retip_if_fail(STRING_VALID(path));
+       content_retip_if_fail(media);
 
        ret = media_svc_get_storage_id(_content_get_db_handle(), path, storage_id, tzplatform_getuid(TZ_USER_NAME));
        if (ret != MS_MEDIA_ERR_NONE) {
@@ -541,10 +542,7 @@ int media_info_delete_from_db(const char *media_id)
        char *storage_id = NULL;
        content_warn("DEPRECATION WARNING: media_info_delete_from_db() is deprecated and will be removed from next release. Use media_content_scan_file() instead.");
 
-       if (!STRING_VALID(media_id)) {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
 
        ret = __media_info_get_media_path_by_id_from_db(media_id, &path);
        if (ret != MEDIA_CONTENT_ERROR_NONE) {
@@ -570,7 +568,8 @@ int media_info_delete_from_db(const char *media_id)
 int media_info_destroy(media_info_h media)
 {
        media_info_s *_media = (media_info_s*)media;
-       content_retvm_if(_media == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "Handle is null");
+
+       content_retip_if_fail(media);
 
        g_free(_media->media_id);
        g_free(_media->file_path);
@@ -739,113 +738,64 @@ int media_info_clone(media_info_h *dst, media_info_h src)
 
 int media_info_get_media_count_from_db(filter_h filter, int *media_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if (media_count == NULL) {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
+       content_retip_if_fail(media_count);
 
-               return ret;
-       }
-
-       ret = _media_db_get_group_item_count(NULL, filter, MEDIA_GROUP_NONE, media_count);
-
-       return ret;
+       return _media_db_get_group_item_count(NULL, filter, MEDIA_GROUP_NONE, media_count);
 }
 
 int media_info_foreach_media_from_db(filter_h filter, media_info_cb callback, void *user_data)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       content_retvm_if(callback == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "Invalid callback");
-
-       ret = _media_db_get_group_item(NULL, filter, callback, user_data, MEDIA_GROUP_NONE);
+       content_retip_if_fail(callback);
 
-       return ret;
+       return _media_db_get_group_item(NULL, filter, callback, user_data, MEDIA_GROUP_NONE);
 }
 
 int media_info_get_tag_count_from_db(const char *media_id, filter_h filter, int *tag_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if (STRING_VALID(media_id) && tag_count) {
-               ret = _media_db_get_group_item_count(media_id, filter, MEDIA_GROUP_TAG_BY_MEDIA_ID, tag_count);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(tag_count);
 
-       return ret;
+       return _media_db_get_group_item_count(media_id, filter, MEDIA_GROUP_TAG_BY_MEDIA_ID, tag_count);
 }
 
 int media_info_foreach_tag_from_db(const char *media_id, filter_h filter, media_tag_cb callback, void *user_data)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if ((callback != NULL) && STRING_VALID(media_id)) {
-               ret = _media_db_get_tag(media_id, filter, callback, user_data);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(callback);
 
-       return ret;
+       return _media_db_get_tag(media_id, filter, callback, user_data);
 }
 
 int media_info_get_bookmark_count_from_db(const char *media_id, filter_h filter, int *bookmark_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if (STRING_VALID(media_id) && bookmark_count) {
-               ret = _media_db_get_group_item_count(media_id, filter, MEDIA_GROUP_BOOKMARK_BY_MEDIA_ID, bookmark_count);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(bookmark_count);
 
-       return ret;
+       return _media_db_get_group_item_count(media_id, filter, MEDIA_GROUP_BOOKMARK_BY_MEDIA_ID, bookmark_count);
 }
 
 int media_info_foreach_bookmark_from_db(const char *media_id, filter_h filter, media_bookmark_cb callback, void *user_data)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if ((callback != NULL) && STRING_VALID(media_id)) {
-               ret = _media_db_get_bookmark(media_id, filter, callback, user_data);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(callback);
 
-       return ret;
+       return _media_db_get_bookmark(media_id, filter, callback, user_data);
 }
 
 int media_info_get_face_count_from_db(const char *media_id, filter_h filter, int *face_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if (STRING_VALID(media_id) && face_count) {
-               ret = _media_db_get_group_item_count(media_id, filter, MEDIA_GROUP_FACE_BY_MEDIA_ID, face_count);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(face_count);
 
-       return ret;
+       return _media_db_get_group_item_count(media_id, filter, MEDIA_GROUP_FACE_BY_MEDIA_ID, face_count);
 }
 
 int media_info_foreach_face_from_db(const char *media_id, filter_h filter, media_face_cb callback, void *user_data)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if ((callback != NULL) && STRING_VALID(media_id)) {
-               ret = _media_db_get_face(media_id, filter, callback, user_data);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(callback);
 
-       return ret;
+       return _media_db_get_face(media_id, filter, callback, user_data);
 }
 
 int media_info_get_image(media_info_h media, image_meta_h *image)
@@ -984,17 +934,14 @@ int media_info_get_display_name(media_info_h media, char **name)
 
 int media_info_get_media_type(media_info_h media, media_content_type_e *type)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
-       if (_media && type) {
-               *type = _media->media_type;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
 
-       return ret;
+       content_retip_if_fail(media);
+       content_retip_if_fail(type);
+
+       *type = _media->media_type;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_mime_type(media_info_h media, char **mime_type)
@@ -1011,63 +958,50 @@ int media_info_get_mime_type(media_info_h media, char **mime_type)
 
 int media_info_get_size(media_info_h media, unsigned long long *size)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media && size) {
-               *size = _media->size;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(size);
 
-       return ret;
+       *size = _media->size;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_added_time(media_info_h media, time_t *added_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
-       if (_media && added_time) {
-               *added_time = _media->added_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
 
-       return ret;
+       content_retip_if_fail(media);
+       content_retip_if_fail(added_time);
+
+       *added_time = _media->added_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_modified_time(media_info_h media, time_t* time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
-       if (_media && time) {
-               *time = _media->modified_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
 
-       return ret;
+       content_retip_if_fail(media);
+       content_retip_if_fail(time);
+
+       *time = _media->modified_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_timeline(media_info_h media, time_t* time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
-       if (_media && time) {
-               *time = _media->timeline;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
 
-       return ret;
+       content_retip_if_fail(media);
+       content_retip_if_fail(time);
+
+       *time = _media->timeline;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_thumbnail_path(media_info_h media, char **path)
@@ -1108,81 +1042,61 @@ int media_info_get_description(media_info_h media, char **description)
 
 int media_info_get_longitude(media_info_h media, double* longitude)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media && longitude) {
-               *longitude = _media->longitude;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(longitude);
 
-       return ret;
+       *longitude = _media->longitude;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 int media_info_get_latitude(media_info_h media, double* latitude)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media && latitude) {
-               *latitude = _media->latitude;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(latitude);
 
-       return ret;
+       *latitude = _media->latitude;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_altitude(media_info_h media, double *altitude)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media && altitude) {
-               *altitude = _media->altitude;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(altitude);
 
-       return ret;
+       *altitude = _media->altitude;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_rating(media_info_h media, int *rating)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media && rating) {
-               *rating = _media->rating;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(rating);
 
-       return ret;
+       *rating = _media->rating;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_favorite(media_info_h media, bool* favorite)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media && favorite) {
-               *favorite = _media->favourite;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(favorite);
 
-       return ret;
+       *favorite = _media->favourite;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_storage_id(media_info_h media, char **storage_id)
@@ -1200,51 +1114,39 @@ int media_info_get_storage_id(media_info_h media, char **storage_id)
 
 int media_info_is_drm(media_info_h media, bool *is_drm)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               *is_drm = _media->is_drm;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(is_drm);
 
-       return ret;
+       *is_drm = _media->is_drm;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_is_360_content(media_info_h media, bool *is_360)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               *is_360 = _media->is_360;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(is_360);
 
-       return ret;
+       *is_360 = _media->is_360;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_storage_type(media_info_h media, media_content_storage_e *storage_type)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        content_warn("DEPRECATION WARNING: media_info_get_storage_type() is deprecated and will be removed from next release. Use storage_get_type_dev() instead.");
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media && storage_type) {
-               *storage_type = _media->storage_type;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(storage_type);
 
-       return ret;
+       *storage_type = _media->storage_type;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 #ifdef _USE_SENIOR_MODE
@@ -1327,10 +1229,7 @@ int media_info_delete_item(const char *media_id)
        char *path = NULL;
        char *storage_id = NULL;
 
-       if (!STRING_VALID(media_id)) {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
 
        ret = __media_info_get_media_path_by_id_from_db(media_id, &path);
        if (ret != MEDIA_CONTENT_ERROR_NONE) {
@@ -1387,147 +1286,109 @@ int media_info_get_app_data(media_info_h media, char **app_data)
 #ifdef _USE_TVPD_MODE
 int media_info_get_played_count(media_info_h media, int *played_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               *played_count = _media->played_count;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(played_count);
 
-       return ret;
+       *played_count = _media->played_count;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_set_played_count(media_info_h media, int played_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               _media->played_count = played_count;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
 
-       return ret;
+       _media->played_count = played_count;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_set_played_position(media_info_h media, int played_position)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if ((_media != NULL) && (played_position >= 0)) {
-               _media->played_position = played_position;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(played_position >= 0);
 
-       return ret;
+       _media->played_position = played_position;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_played_position(media_info_h media, int *played_position)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               *played_position = _media->played_position;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(played_position);
 
-       return ret;
+       *played_position = _media->played_position;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_set_played_time(media_info_h media)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        time_t current_time;
 
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media != NULL) {
-               time(&current_time);
-               _media->played_time = current_time;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
 
-       return ret;
+       time(&current_time);
+       _media->played_time = current_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_played_time(media_info_h media, time_t* played_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               *played_time = _media->played_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(played_time);
 
-       return ret;
+       *played_time = _media->played_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_extract_flag(media_info_h media, int *extract_flag)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s *)media;
 
-       if (_media && extract_flag) {
-               *extract_flag = _media->extract_flag;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(extract_flag);
 
-       return ret;
+       *extract_flag = _media->extract_flag;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_stitched_state(media_info_h media, int *stitched_info)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               *stitched_info = (_media->stitched_info & 0xFFFF0000) >> 16;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(stitched_info);
 
-       return ret;
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_stitched_engine(media_info_h media, int *stitched_info)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media) {
-               *stitched_info = _media->stitched_info & 0x0000FFFF;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(stitched_info);
 
-       return ret;
+       *stitched_info = _media->stitched_info & 0x0000FFFF;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_get_modified_month(media_info_h media, char **modified_month)
@@ -1585,20 +1446,16 @@ int media_info_get_media_from_db(const char *media_id, media_info_h *media)
 
 int media_info_set_favorite(media_info_h media, bool favorite)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media != NULL) {
-               if (favorite == TRUE)
-                       time(&_media->favourite);
-               else
-                       _media->favourite = 0;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
 
-       return ret;
+       if (favorite == TRUE)
+               time(&_media->favourite);
+       else
+               _media->favourite = 0;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_update_to_db(media_info_h media)
@@ -1608,62 +1465,60 @@ int media_info_update_to_db(media_info_h media)
        char *set_sql = NULL;
        char *sql = NULL;
 
-       if (_media != NULL && STRING_VALID(_media->media_id)) {
-               /* This sql is due to sqlite3_mprintf's wrong operation when using floating point in the text format */
-               /* This code will be removed when sqlite3_mprintf works clearly */
-               char *test_sql = sqlite3_mprintf("%f, %f, %f", _media->longitude, _media->latitude, _media->altitude);
-               SQLITE3_SAFE_FREE(test_sql);
-
-               /*Update Pinyin If Support Pinyin*/
-               char *file_name_pinyin = NULL;
-               char *description_pinyin = NULL;
-               bool pinyin_support = FALSE;
-
-               /*Update Pinyin If Support Pinyin*/
-               media_svc_check_pinyin_support(&pinyin_support);
-               if (pinyin_support) {
-                       if (STRING_VALID(_media->display_name))
-                               media_svc_get_pinyin(_media->display_name, &file_name_pinyin);
-                       if (STRING_VALID(_media->description))
-                               media_svc_get_pinyin(_media->description, &description_pinyin);
-               }
+       content_retip_if_fail(media);
+       content_retip_if_fail(STRING_VALID(_media->media_id));
+
+       /* This sql is due to sqlite3_mprintf's wrong operation when using floating point in the text format */
+       /* This code will be removed when sqlite3_mprintf works clearly */
+       char *test_sql = sqlite3_mprintf("%f, %f, %f", _media->longitude, _media->latitude, _media->altitude);
+       SQLITE3_SAFE_FREE(test_sql);
+
+       /*Update Pinyin If Support Pinyin*/
+       char *file_name_pinyin = NULL;
+       char *description_pinyin = NULL;
+       bool pinyin_support = FALSE;
+
+       /*Update Pinyin If Support Pinyin*/
+       media_svc_check_pinyin_support(&pinyin_support);
+       if (pinyin_support) {
+               if (STRING_VALID(_media->display_name))
+                       media_svc_get_pinyin(_media->display_name, &file_name_pinyin);
+               if (STRING_VALID(_media->description))
+                       media_svc_get_pinyin(_media->description, &description_pinyin);
+       }
 #ifdef _USE_TVPD_MODE
 #ifdef _USE_SENIOR_MODE
-               if (_media_content_is_support_senior_mode()) {
-                       set_sql = sqlite3_mprintf("media_display_name=%Q, media_description=%Q, media_longitude=%f, media_latitude=%f, media_file_name_pinyin=%Q, media_description_pinyin=%Q, played_count=%d, last_played_time=%d, last_played_position=%d, media_rating=%d, media_favourite=%d, contact=%Q, app_data=%Q",
-                               _media->display_name, _media->description, _media->longitude, _media->latitude, file_name_pinyin, description_pinyin, _media->played_count, _media->played_time, _media->played_position, _media->rating, _media->favourite, _media->contact, _media->app_data);
-               } else
+       if (_media_content_is_support_senior_mode()) {
+               set_sql = sqlite3_mprintf("media_display_name=%Q, media_description=%Q, media_longitude=%f, media_latitude=%f, media_file_name_pinyin=%Q, media_description_pinyin=%Q, played_count=%d, last_played_time=%d, last_played_position=%d, media_rating=%d, media_favourite=%d, contact=%Q, app_data=%Q",
+                       _media->display_name, _media->description, _media->longitude, _media->latitude, file_name_pinyin, description_pinyin, _media->played_count, _media->played_time, _media->played_position, _media->rating, _media->favourite, _media->contact, _media->app_data);
+       } else
 #endif
-               {
-                       set_sql = sqlite3_mprintf("media_display_name=%Q, media_description=%Q, media_longitude=%f, media_latitude=%f, media_file_name_pinyin=%Q, media_description_pinyin=%Q, played_count=%d, last_played_time=%d, last_played_position=%d, media_rating=%d, media_favourite=%d",
-                               _media->display_name, _media->description, _media->longitude, _media->latitude, file_name_pinyin, description_pinyin, _media->played_count, _media->played_time, _media->played_position, _media->rating, _media->favourite);
-               }
+       {
+               set_sql = sqlite3_mprintf("media_display_name=%Q, media_description=%Q, media_longitude=%f, media_latitude=%f, media_file_name_pinyin=%Q, media_description_pinyin=%Q, played_count=%d, last_played_time=%d, last_played_position=%d, media_rating=%d, media_favourite=%d",
+                       _media->display_name, _media->description, _media->longitude, _media->latitude, file_name_pinyin, description_pinyin, _media->played_count, _media->played_time, _media->played_position, _media->rating, _media->favourite);
+       }
 #else
-               set_sql = sqlite3_mprintf("media_display_name=%Q, media_description=%Q, media_longitude=%f, media_latitude=%f, media_file_name_pinyin=%Q, media_description_pinyin=%Q, media_rating=%d, media_favourite=%d", _media->display_name, _media->description, _media->longitude, _media->latitude, file_name_pinyin, description_pinyin, _media->rating, _media->favourite);
+       set_sql = sqlite3_mprintf("media_display_name=%Q, media_description=%Q, media_longitude=%f, media_latitude=%f, media_file_name_pinyin=%Q, media_description_pinyin=%Q, media_rating=%d, media_favourite=%d", _media->display_name, _media->description, _media->longitude, _media->latitude, file_name_pinyin, description_pinyin, _media->rating, _media->favourite);
 #endif
 #ifdef _USE_TVPD_MODE
-               sql = sqlite3_mprintf("UPDATE %Q SET %s WHERE media_id=%Q", _media->storage_uuid, set_sql, _media->media_id);
+       sql = sqlite3_mprintf("UPDATE %Q SET %s WHERE media_id=%Q", _media->storage_uuid, set_sql, _media->media_id);
 #else
-               sql = sqlite3_mprintf("UPDATE %q SET %s WHERE media_id=%Q", DB_TABLE_MEDIA, set_sql, _media->media_id);
+       sql = sqlite3_mprintf("UPDATE %q SET %s WHERE media_id=%Q", DB_TABLE_MEDIA, set_sql, _media->media_id);
 #endif
-               ret = _content_query_sql(sql);
-               SQLITE3_SAFE_FREE(set_sql);
-               SQLITE3_SAFE_FREE(sql);
-
-               SAFE_FREE(file_name_pinyin);
-               SAFE_FREE(description_pinyin);
-
-               if (ret == MEDIA_CONTENT_ERROR_NONE) {
-                       /* Send notification for this update */
-                       content_debug("Update is successfull. Send notification for this");
-                       if (_media->file_path && _media->mime_type)
-                               media_svc_publish_noti(MS_MEDIA_ITEM_UPDATE, _media->file_path, _media->media_type, _media->media_id, _media->mime_type);
-                       else
-                               content_error("Can't Send Noti : path or mime type is NULL");
-               }
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
+       ret = _content_query_sql(sql);
+       SQLITE3_SAFE_FREE(set_sql);
+       SQLITE3_SAFE_FREE(sql);
+
+       SAFE_FREE(file_name_pinyin);
+       SAFE_FREE(description_pinyin);
+
+       if (ret == MEDIA_CONTENT_ERROR_NONE) {
+               /* Send notification for this update */
+               content_debug("Update is successfull. Send notification for this");
+               if (_media->file_path && _media->mime_type)
+                       media_svc_publish_noti(MS_MEDIA_ITEM_UPDATE, _media->file_path, _media->media_type, _media->media_id, _media->mime_type);
+               else
+                       content_error("Can't Send Noti : path or mime type is NULL");
        }
 
        return ret;
@@ -1674,8 +1529,8 @@ int media_info_move_to_db(media_info_h media, const char* dst_path)
        int ret = MEDIA_CONTENT_ERROR_NONE;
        char repl_path[MAX_PATH_LEN] = {0, };
 
-       content_retvm_if(media == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid media");
-       content_retvm_if(!STRING_VALID(dst_path), MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid dst_path");
+       content_retip_if_fail(media);
+       content_retip_if_fail(STRING_VALID(dst_path));
 
        ret = _media_content_replace_path(dst_path, repl_path);
        content_retvm_if(!STRING_VALID(repl_path), MEDIA_CONTENT_ERROR_INVALID_OPERATION, "path replacement failed");
@@ -1761,15 +1616,12 @@ int media_info_cancel_thumbnail(media_info_h media)
        content_warn("DEPRECATION WARNING: media_info_cancel_thumbnail() is deprecated and will be removed from next release.");
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media != NULL && STRING_VALID(_media->media_id) && _media->request_id > 0) {
-               ret = thumbnail_request_cancel_media(_media->request_id);
-               ret = _content_error_capi(ret);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(STRING_VALID(_media->media_id));
+       content_retip_if_fail(_media->request_id > 0);
 
-       return ret;
+       ret = thumbnail_request_cancel_media(_media->request_id);
+       return _content_error_capi(ret);
 }
 
 int media_info_start_face_detection(media_info_h media, media_face_detection_completed_cb callback, void *user_data)
@@ -1817,15 +1669,12 @@ int media_info_cancel_face_detection(media_info_h media)
                return MEDIA_CONTENT_ERROR_NOT_SUPPORTED;
        }
 
-       if (_media != NULL && STRING_VALID(_media->media_id) && _media->face_request_id > 0) {
-               ret = dcm_request_cancel_face(_media->face_request_id);
-               ret = _content_error_capi(ret);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
+       content_retip_if_fail(STRING_VALID(_media->media_id));
+       content_retip_if_fail(_media->face_request_id > 0);
 
-       return ret;
+       ret = dcm_request_cancel_face(_media->face_request_id);
+       return _content_error_capi(ret);
 }
 
 int media_info_set_description(media_info_h media, const char *description)
@@ -1843,50 +1692,38 @@ int media_info_set_description(media_info_h media, const char *description)
 
 int media_info_set_longitude(media_info_h media, double longitude)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        content_warn("DEPRECATION WARNING: media_info_set_longitude() is deprecated and will be removed from next release.");
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media != NULL) {
-               _media->longitude = longitude;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
 
-       return ret;
+       _media->longitude = longitude;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_set_latitude(media_info_h media, double latitude)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        content_warn("DEPRECATION WARNING: media_info_set_latitude() is deprecated and will be removed from next release.");
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media != NULL) {
-               _media->latitude = latitude;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
 
-       return ret;
+       _media->latitude = latitude;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_info_set_rating(media_info_h media, int rating)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        content_warn("DEPRECATION WARNING: media_info_set_rating() is deprecated and will be removed from next release.");
        media_info_s *_media = (media_info_s*)media;
 
-       if (_media != NULL) {
-               _media->rating = rating;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media);
 
-       return ret;
+       _media->rating = rating;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 #ifdef TIZEN_FEATURE_COMPATIBILITY