From 9d46740e7346799958dcc263434202d74af086bd Mon Sep 17 00:00:00 2001 From: hj kim Date: Wed, 11 Mar 2020 13:49:48 +0900 Subject: [PATCH] Fix wrong error value check Change-Id: I64632bc46092b3fcafc98dba4eca778f089c5aad --- src/common/media-svc-util.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/src/common/media-svc-util.c b/src/common/media-svc-util.c index eafa52e..67c204a 100644 --- a/src/common/media-svc-util.c +++ b/src/common/media-svc-util.c @@ -1621,15 +1621,9 @@ int _media_svc_create_thumbnail(const char *path, char *thumb_path, media_svc_me unsigned int thumb_h = THUMB_HEIGHT; mm_util_img_codec_type image_type = IMG_CODEC_UNKNOWN_TYPE; - if (path == NULL || thumb_path == NULL) { - media_svc_error("Invalid parameter"); - return MS_MEDIA_ERR_INVALID_PARAMETER; - } - - if (!g_file_test(path, G_FILE_TEST_IS_REGULAR)) { - media_svc_error("Original path(%s) doesn't exist.", path); - return MS_MEDIA_ERR_INVALID_PARAMETER; - } + media_svc_retvm_if(!path, MS_MEDIA_ERR_INVALID_PARAMETER, "Invalid path"); + media_svc_retvm_if(!thumb_path, MS_MEDIA_ERR_INVALID_PARAMETER, "Invalid thumb_path"); + media_svc_retvm_if(!g_file_test(path, G_FILE_TEST_IS_REGULAR), MS_MEDIA_ERR_INVALID_PARAMETER, "File doesn't exist[%s]", path); ms_user_storage_type_e store_type = -1; ret = ms_user_get_storage_type(uid, path, &store_type); @@ -1652,7 +1646,7 @@ int _media_svc_create_thumbnail(const char *path, char *thumb_path, media_svc_me //2. save thumbnail if (media_type == MEDIA_SVC_MEDIA_TYPE_IMAGE) { ret = mm_util_extract_image_info(path, &image_type, &origin_w, &origin_h); - if (ret != MS_MEDIA_ERR_NONE) { + if (ret != MM_UTIL_ERROR_NONE) { media_svc_error("Getting image info is failed err: %d", ret); SAFE_STRLCPY(thumb_path, "", MAX_FILEPATH_LEN); return MS_MEDIA_ERR_INTERNAL; -- 2.7.4