Remove TYER value checker 34/306934/5
authorminje.ahn <minje.ahn@samsung.com>
Thu, 29 Feb 2024 02:25:12 +0000 (11:25 +0900)
committerMinje ahn <minje.ahn@samsung.com>
Mon, 11 Mar 2024 02:14:42 +0000 (02:14 +0000)
If it meets the ID3 tag rules, there is no need to check the value.

Change-Id: I817a8822bf18993227ef1ebb2391b7093aeb6c2e
Signed-off-by: minje.ahn <minje.ahn@samsung.com>
src/common/media-svc-util.c

index c9e9b2d..2a2f0e2 100644 (file)
@@ -264,25 +264,6 @@ static bool __media_svc_get_file_ext(const char *file_path, char *file_ext)
        return false;
 }
 
-static int __media_svc_safe_atoi(char *buffer, int *si)
-{
-       char *end = NULL;
-       errno = 0;
-       media_svc_retvm_if(buffer == NULL || si == NULL, MS_MEDIA_ERR_INTERNAL, "invalid parameter");
-
-       const long sl = strtol(buffer, &end, 10);
-
-       media_svc_retvm_if(end == buffer, MS_MEDIA_ERR_INTERNAL, "not a decimal number");
-       media_svc_retvm_if('\0' != *end, MS_MEDIA_ERR_INTERNAL, "extra characters at end of input: %s", end);
-       media_svc_retvm_if((LONG_MIN == sl || LONG_MAX == sl) && (ERANGE == errno), MS_MEDIA_ERR_INTERNAL, "out of range of type long");
-       media_svc_retvm_if(sl > INT_MAX, MS_MEDIA_ERR_INTERNAL, "greater than INT_MAX");
-       media_svc_retvm_if(sl < INT_MIN, MS_MEDIA_ERR_INTERNAL, "less than INT_MIN");
-
-       *si = (int)sl;
-
-       return MS_MEDIA_ERR_NONE;
-}
-
 static int __media_svc_save_image(unsigned char *image, unsigned int size, char *image_path, uid_t uid)
 {
        int ret = MS_MEDIA_ERR_NONE;
@@ -589,7 +570,6 @@ int _media_svc_extract_audio_metadata(sqlite3 *handle, bool is_direct, media_svc
        int mmf_error = FILEINFO_ERROR_NONE;
        int album_id = 0;
        int ret = MS_MEDIA_ERR_NONE;
-       int convert_value = 0;
        bool support_albumart = ms_user_thumb_support(uid, content_info->path);
 
        /*Get Content Tag attribute ===========*/
@@ -623,13 +603,10 @@ int _media_svc_extract_audio_metadata(sqlite3 *handle, bool is_direct, media_svc
                        content_info->media_meta.recorded_date = g_strdup(MEDIA_SVC_TAG_UNKNOWN);
 
                mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_DATE, &p, &size, NULL);
-               if (mmf_error == FILEINFO_ERROR_NONE && size == 4) {
-                       if (__media_svc_safe_atoi(p, &convert_value) == MS_MEDIA_ERR_NONE)
-                               content_info->media_meta.year = g_strdup(p);
-               }
-
-               if (!content_info->media_meta.year)
-                               content_info->media_meta.year = g_strdup(MEDIA_SVC_TAG_UNKNOWN);
+               if (mmf_error == FILEINFO_ERROR_NONE && size == 4)
+                       content_info->media_meta.year = g_strdup(p);
+               else
+                       content_info->media_meta.year = g_strdup(MEDIA_SVC_TAG_UNKNOWN);
 
                /*Do not extract artwork for the USB Storage content*/
                if (support_albumart) {