Remove unused 'err_attr_name' for 'mm_file_get_attrs'

Change-Id: Ib58ff35975aed257c5fb6ddd27a8dd515d492af7
(cherry picked from commit 93fac9778dd755f7087187d25ff2fbb047bf2d58)

diff --git a/src/common/media-svc-util.c b/src/common/media-svc-util.c
index fc3f052..1035042 100644 (file)
--- a/src/common/media-svc-util.c
+++ b/src/common/media-svc-util.c
@@ -484,32 +484,25 @@ static bool __media_svc_get_file_ext(const char *file_path, char *file_ext)
 
 static int __media_svc_get_location_value(MMHandleType tag, double *longitude, double *latitude, double *altitude)
 {
-       char *err_attr_name = NULL;
+       int mmf_error = 0;
        double gps_value = 0.0;
-       int mmf_error = FILEINFO_ERROR_NONE;
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_LONGITUDE, &gps_value, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_LONGITUDE, &gps_value, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE) {
                if (longitude != NULL)
                        *longitude = (gps_value == 0.0) ? MEDIA_SVC_DEFAULT_GPS_VALUE : gps_value;
-       } else {
-               SAFE_FREE(err_attr_name);
        }
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_LATIDUE, &gps_value, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_LATIDUE, &gps_value, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE) {
                if (latitude != NULL)
                        *latitude = (gps_value == 0.0) ? MEDIA_SVC_DEFAULT_GPS_VALUE : gps_value;
-       } else {
-               SAFE_FREE(err_attr_name);
        }
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ALTIDUE, &gps_value, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ALTIDUE, &gps_value, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE) {
                if (altitude != NULL)
                        *altitude = (gps_value == 0.0) ? MEDIA_SVC_DEFAULT_GPS_VALUE : gps_value;
-       } else {
-               SAFE_FREE(err_attr_name);
        }
 
        return MS_MEDIA_ERR_NONE;
@@ -1167,7 +1160,6 @@ int _media_svc_extract_music_metadata_for_update(media_svc_content_info_s *conte
        char *p = NULL;
        int size = -1;
        int mmf_error = FILEINFO_ERROR_NONE;
-       char *err_attr_name = NULL;
 
        content_info->storage_uuid = g_strdup(storage_id);
        content_info->path = g_strdup(path);
@@ -1176,31 +1168,23 @@ int _media_svc_extract_music_metadata_for_update(media_svc_content_info_s *conte
        if (mmf_error != FILEINFO_ERROR_NONE)
                return MS_MEDIA_ERR_INTERNAL;
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ALBUM, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ALBUM, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                content_info->media_meta.album = g_strdup(p);
-       else
-               SAFE_FREE(err_attr_name);
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ARTIST, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ARTIST, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                content_info->media_meta.artist = g_strdup(p);
-       else
-               SAFE_FREE(err_attr_name);
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ALBUM_ARTIST, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ALBUM_ARTIST, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                content_info->media_meta.album_artist = g_strdup(p);
-       else
-               SAFE_FREE(err_attr_name);
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_GENRE, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_GENRE, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                content_info->media_meta.genre = g_strdup(p);
-       else
-               SAFE_FREE(err_attr_name);
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_TITLE, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_TITLE, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0) {
                if (!isspace(*p)) {
                        content_info->media_meta.title = g_strdup(p);
@@ -1221,23 +1205,17 @@ int _media_svc_extract_music_metadata_for_update(media_svc_content_info_s *conte
                }
        }
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_DESCRIPTION, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_DESCRIPTION, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                content_info->media_meta.description = g_strdup(p);
-       else
-               SAFE_FREE(err_attr_name);
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_AUTHOR, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_AUTHOR, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                content_info->media_meta.composer = g_strdup(p);
-       else
-               SAFE_FREE(err_attr_name);
 
-       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_COPYRIGHT, &p, &size, NULL);
+       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_COPYRIGHT, &p, &size, NULL);
        if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                content_info->media_meta.copyright = g_strdup(p);
-       else
-               SAFE_FREE(err_attr_name);
 
        mmf_error = mm_file_destroy_tag_attrs(tag);
        if (mmf_error != FILEINFO_ERROR_NONE)
@@ -1254,7 +1232,6 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
        unsigned char *image = NULL;
        unsigned int size = 0;
        int mmf_error = FILEINFO_ERROR_NONE;
-       char *err_attr_name = NULL;
        int album_id = 0;
        int ret = MS_MEDIA_ERR_NONE;
        int cdis_value = 0;
@@ -1263,31 +1240,23 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
        mmf_error = mm_file_create_tag_attrs(&tag, content_info->path);
 
        if (mmf_error == FILEINFO_ERROR_NONE) {
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ALBUM, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ALBUM, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.album = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ARTIST, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ARTIST, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.artist = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ALBUM_ARTIST, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ALBUM_ARTIST, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.album_artist = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_GENRE, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_GENRE, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.genre = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_TITLE, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_TITLE, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0) {
                        if (!isspace(*p)) {
                                content_info->media_meta.title = g_strdup(p);
@@ -1308,17 +1277,13 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
                        }
                }
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_DESCRIPTION, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_DESCRIPTION, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.description = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_SPHERICAL, &content_info->media_meta.is_360, NULL);
-               if (mmf_error != FILEINFO_ERROR_NONE)
-                       SAFE_FREE(err_attr_name);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_SPHERICAL, &content_info->media_meta.is_360, NULL);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_RECDATE, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_RECDATE, &p, &size, NULL);
                if ((mmf_error == FILEINFO_ERROR_NONE) && (size > 0)) {
                        char mime_type[255] = {0, };
                        ret = __media_svc_get_mime_type(content_info->path, mime_type);
@@ -1367,63 +1332,50 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
                                        content_info->media_meta.datetaken = g_strdup(content_info->media_meta.recorded_date);
                                }
                        }
-               } else {
-                       SAFE_FREE(err_attr_name);
                }
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_AUTHOR, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_AUTHOR, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.composer = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_COPYRIGHT, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_COPYRIGHT, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.copyright = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_TRACK_NUM, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_TRACK_NUM, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0)
                        content_info->media_meta.track_num = g_strdup(p);
-               else
-                       SAFE_FREE(err_attr_name);
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_DATE, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_DATE, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size == 4) {
                        int year = 0;
                        if ((p != NULL) && (__media_svc_safe_atoi(p, &year) == MS_MEDIA_ERR_NONE))
                                content_info->media_meta.year = g_strdup(p);
                        else
                                media_svc_debug("Wrong Year Information [%s]", p);
-               } else {
-                       SAFE_FREE(err_attr_name);
                }
 
-               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_RATING, &p, &size, NULL);
+               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_RATING, &p, &size, NULL);
                if (mmf_error == FILEINFO_ERROR_NONE && size > 0) {
                        int rate = 0;
                        if ((p != NULL) && (__media_svc_safe_atoi(p, &rate) == MS_MEDIA_ERR_NONE))
                                content_info->media_meta.rating = rate;
                } else {
-                       SAFE_FREE(err_attr_name);
                        content_info->media_meta.rating = 0;
                }
 
                /*Do not extract artwork for the USB Storage content*/
                if (content_info->storage_type != MS_USER_STORAGE_EXTERNAL_USB) {
-                       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ARTWORK, &image, &size, NULL);
+                       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ARTWORK, &image, &size, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
                                media_svc_error("fail to get tag artwork - err(%x)", mmf_error);
-                               SAFE_FREE(err_attr_name);
                        } else {
                                /*media_svc_debug("artwork size1 [%d]", size); */
                        }
 
-                       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ARTWORK_SIZE, &size, NULL);
+                       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ARTWORK_SIZE, &size, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
                                media_svc_error("fail to get artwork size - err(%x)", mmf_error);
-                               SAFE_FREE(err_attr_name);
                        } else {
                                /*media_svc_debug("artwork size2 [%d]", size); */
                        }
@@ -1432,7 +1384,7 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
                                char thumb_path[MEDIA_SVC_PATHNAME_SIZE] = "\0";
                                int artwork_mime_size = -1;
 
-                               mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ARTWORK_MIME, &p, &artwork_mime_size, NULL);
+                               mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ARTWORK_MIME, &p, &artwork_mime_size, NULL);
                                if ((mmf_error == FILEINFO_ERROR_NONE) && (artwork_mime_size > 0)) {
                                        ret = _media_svc_get_thumbnail_path(content_info->media_type, thumb_path, content_info->path, p, uid);
                                        if (ret != MS_MEDIA_ERR_NONE)
@@ -1454,8 +1406,6 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
                                                        }
                                                }
                                        }
-                               } else {
-                                       SAFE_FREE(err_attr_name);
                                }
                        }
                }
@@ -1488,19 +1438,16 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
                        content_info->media_meta.latitude = latitude;
                        content_info->media_meta.altitude = altitude;
 
-                       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_CDIS, &cdis_value, NULL);
-                       if (mmf_error != FILEINFO_ERROR_NONE) {
+                       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_CDIS, &cdis_value, NULL);
+                       if (mmf_error != FILEINFO_ERROR_NONE)
                                cdis_value = 0;
-                               SAFE_FREE(err_attr_name);
-                       }
 
                        media_svc_debug("CDIS : %d", cdis_value);
 
-                       mmf_error = mm_file_get_attrs(tag, &err_attr_name, MM_FILE_TAG_ROTATE, &p, &size, NULL);
+                       mmf_error = mm_file_get_attrs(tag, MM_FILE_TAG_ROTATE, &p, &size, NULL);
                        if ((mmf_error == FILEINFO_ERROR_NONE) && (size > 0)) {
                                content_info->media_meta.orientation = atoi(p);
                        } else {
-                               SAFE_FREE(err_attr_name);
                                content_info->media_meta.orientation = 0;
                                media_svc_debug("fail to get video orientation attr - err(%x)", mmf_error);
                        }
@@ -1521,9 +1468,8 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
 
        if (mmf_error == FILEINFO_ERROR_NONE) {
                /*Common attribute*/
-               mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_DURATION, &content_info->media_meta.duration, NULL);
+               mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_DURATION, &content_info->media_meta.duration, NULL);
                if (mmf_error != FILEINFO_ERROR_NONE) {
-                       SAFE_FREE(err_attr_name);
                        media_svc_debug("fail to get duration attr - err(%x)", mmf_error);
                } else {
                        /*media_svc_debug("duration : %d", content_info->media_meta.duration); */
@@ -1532,33 +1478,29 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
                /*Sound/Music attribute*/
                if ((content_info->media_type == MEDIA_SVC_MEDIA_TYPE_SOUND) || (content_info->media_type == MEDIA_SVC_MEDIA_TYPE_MUSIC)) {
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_AUDIO_BITRATE, &content_info->media_meta.bitrate, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_AUDIO_BITRATE, &content_info->media_meta.bitrate, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get audio bitrate attr - err(%x)", mmf_error);
                        } else {
                                /*media_svc_debug("bit rate : %d", content_info->media_meta.bitrate); */
                        }
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_AUDIO_SAMPLERATE, &content_info->media_meta.samplerate, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_AUDIO_SAMPLERATE, &content_info->media_meta.samplerate, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get sample rate attr - err(%x)", mmf_error);
                        } else {
                                /*media_svc_debug("sample rate : %d", content_info->media_meta.samplerate); */
                        }
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_AUDIO_CHANNELS, &content_info->media_meta.channel, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_AUDIO_CHANNELS, &content_info->media_meta.channel, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get audio channels attr - err(%x)", mmf_error);
                        } else {
                                /*media_svc_debug("channel : %d", content_info->media_meta.channel); */
                        }
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_AUDIO_BITPERSAMPLE, &content_info->media_meta.bitpersample, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_AUDIO_BITPERSAMPLE, &content_info->media_meta.bitpersample, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get audio bit per sample attr - err(%x)", mmf_error);
                        } else {
                                media_svc_debug("bitpersample : %d", content_info->media_meta.bitpersample);
@@ -1567,17 +1509,15 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
                        int audio_bitrate = 0;
                        int video_bitrate = 0;
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_AUDIO_BITRATE, &audio_bitrate, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_AUDIO_BITRATE, &audio_bitrate, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get audio bitrate attr - err(%x)", mmf_error);
                        } else {
                                /*media_svc_debug("audio bit rate : %d", audio_bitrate); */
                        }
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_VIDEO_BITRATE, &video_bitrate, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_VIDEO_BITRATE, &video_bitrate, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get audio bitrate attr - err(%x)", mmf_error);
                        } else {
                                /*media_svc_debug("video bit rate : %d", video_bitrate); */
@@ -1585,17 +1525,15 @@ int _media_svc_extract_media_metadata(sqlite3 *handle, bool is_direct, media_svc
 
                        content_info->media_meta.bitrate = audio_bitrate + video_bitrate;
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_VIDEO_WIDTH, &content_info->media_meta.width, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_VIDEO_WIDTH, &content_info->media_meta.width, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get video width attr - err(%x)", mmf_error);
                        } else {
                                /*media_svc_debug("width : %d", content_info->media_meta.width); */
                        }
 
-                       mmf_error = mm_file_get_attrs(content, &err_attr_name, MM_FILE_CONTENT_VIDEO_HEIGHT, &content_info->media_meta.height, NULL);
+                       mmf_error = mm_file_get_attrs(content, MM_FILE_CONTENT_VIDEO_HEIGHT, &content_info->media_meta.height, NULL);
                        if (mmf_error != FILEINFO_ERROR_NONE) {
-                               SAFE_FREE(err_attr_name);
                                media_svc_debug("fail to get video height attr - err(%x)", mmf_error);
                        } else {
                                /*media_svc_debug("height : %d", content_info->media_meta.height); */
@@ -1697,7 +1635,6 @@ static void __get_rotation_and_cdis(const char *origin_path, mm_util_magick_rota
        MMHandleType tag = (MMHandleType) NULL;
        char *p = NULL;
        int size = 0;
-       char *err_msg = NULL;
        int _cdis_value = 0;
        mm_util_magick_rotate_type _rot_type = MM_UTIL_ROTATE_NUM;
 
@@ -1709,7 +1646,7 @@ static void __get_rotation_and_cdis(const char *origin_path, mm_util_magick_rota
                return;
        }
 
-       err = mm_file_get_attrs(tag, &err_msg, MM_FILE_TAG_ROTATE, &p, &size, NULL);
+       err = mm_file_get_attrs(tag, MM_FILE_TAG_ROTATE, &p, &size, NULL);
        if (err == FILEINFO_ERROR_NONE && size >= 0) {
                if (p == NULL) {
                        _rot_type = MM_UTIL_ROTATE_0;
@@ -1727,14 +1664,11 @@ static void __get_rotation_and_cdis(const char *origin_path, mm_util_magick_rota
        } else {
                media_svc_debug("There is NOT tag rotate");
                _rot_type = MM_UTIL_ROTATE_0;
-               SAFE_FREE(err_msg);
        }
 
-       err = mm_file_get_attrs(tag, &err_msg, MM_FILE_TAG_CDIS, &_cdis_value, NULL);
-       if (err != FILEINFO_ERROR_NONE) {
+       err = mm_file_get_attrs(tag, MM_FILE_TAG_CDIS, &_cdis_value, NULL);
+       if (err != FILEINFO_ERROR_NONE)
                _cdis_value = 0;
-               SAFE_FREE(err_msg);
-       }
 
        *rot_type = _rot_type;
        *cdis_value = _cdis_value;
@@ -1751,7 +1685,6 @@ static int __get_video_info(int cdis_value, const char *origin_path, int *video_
 {
        int err = MS_MEDIA_ERR_NONE;
        MMHandleType content = (MMHandleType) NULL;
-       char *err_msg = NULL;
        int _video_track_num = 0;
        int _width = 0;
        int _height = 0;
@@ -1770,10 +1703,9 @@ static int __get_video_info(int cdis_value, const char *origin_path, int *video_
                return MS_MEDIA_ERR_INTERNAL;
        }
 
-       err = mm_file_get_attrs(content, &err_msg, MM_FILE_CONTENT_VIDEO_TRACK_COUNT, &_video_track_num, NULL);
+       err = mm_file_get_attrs(content, MM_FILE_CONTENT_VIDEO_TRACK_COUNT, &_video_track_num, NULL);
        if (err != FILEINFO_ERROR_NONE) {
-               media_svc_error("mm_file_get_attrs fails : %s", err_msg);
-               SAFE_FREE(err_msg);
+               media_svc_error("mm_file_get_attrs fails : %d", err);
                mm_file_destroy_content_attrs(content);
                return MS_MEDIA_ERR_INTERNAL;
        }
@@ -1785,7 +1717,7 @@ static int __get_video_info(int cdis_value, const char *origin_path, int *video_
                return MS_MEDIA_ERR_NONE;
        }
 
-       err = mm_file_get_attrs(content, &err_msg,
+       err = mm_file_get_attrs(content,
                                MM_FILE_CONTENT_VIDEO_WIDTH,
                                &_width,
                                MM_FILE_CONTENT_VIDEO_HEIGHT,
@@ -1794,8 +1726,7 @@ static int __get_video_info(int cdis_value, const char *origin_path, int *video_
                                &_size, NULL);
 
        if (err != FILEINFO_ERROR_NONE) {
-               media_svc_error("mm_file_get_attrs fails : %s", err_msg);
-               SAFE_FREE(err_msg);
+               media_svc_error("mm_file_get_attrs fails : %d", err);
                mm_file_destroy_content_attrs(content);
                return MS_MEDIA_ERR_INTERNAL;
        }