Check input parameter

[platform/core/api/image-util.git] / src / image_util.c

diff --git a/src/image_util.c b/src/image_util.c
old mode 100644 (file)
new mode 100755 (executable)
index f2df142..510f251
--- a/src/image_util.c
+++ b/src/image_util.c
@@ -201,7 +201,7 @@ static int __mm_util_transform(transformation_s *handle)
                if (ret != MM_UTIL_ERROR_NONE) {
                        __mm_destroy_temp_buffer(dst_buf);
                        image_util_error("mm_util_crop_image failed");
-                       return _image_error_capi(ERR_TYPE_TRANSFORM, ret);
+                       return _image_error_capi(ret);
                }
 
                dst_buf[dst_index] = res_buffer;
@@ -215,7 +215,7 @@ static int __mm_util_transform(transformation_s *handle)
                if (ret != MM_UTIL_ERROR_NONE) {
                        __mm_destroy_temp_buffer(dst_buf);
                        image_util_error("mm_util_resize_image failed");
-                       return _image_error_capi(ERR_TYPE_TRANSFORM, ret);
+                       return _image_error_capi(ret);
                }
 
                dst_buf[dst_index] = res_buffer;
@@ -231,7 +231,7 @@ static int __mm_util_transform(transformation_s *handle)
                if (ret != MM_UTIL_ERROR_NONE) {
                        __mm_destroy_temp_buffer(dst_buf);
                        image_util_error("mm_util_convert_colorspace failed");
-                       return _image_error_capi(ERR_TYPE_TRANSFORM, ret);
+                       return _image_error_capi(ret);
                }
 
                dst_buf[dst_index] = res_buffer_conv;
@@ -248,7 +248,7 @@ static int __mm_util_transform(transformation_s *handle)
                if (ret != MM_UTIL_ERROR_NONE) {
                        __mm_destroy_temp_buffer(dst_buf);
                        image_util_error("mm_util_rotate_image failed");
-                       return _image_error_capi(ERR_TYPE_TRANSFORM, ret);
+                       return _image_error_capi(ret);
                }
 
                dst_buf[dst_index] = res_buffer_rotate;
@@ -506,6 +506,7 @@ int image_util_transform_set_colorspace(transformation_h handle, image_util_colo
        image_util_debug("Set colorspace_convert_info [%d]", colorspace);
 
        image_util_retvm_if((_handle == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid Handle");
+       image_util_retvm_if((is_valid_colorspace(colorspace) == FALSE), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
 
        _handle->dst_format = colorspace;
        _handle->set_convert = true;
@@ -719,7 +720,7 @@ int image_util_calculate_buffer_size(int width, int height, image_util_colorspac
 
        *size = (unsigned int)size_ptr;
 
-       return _image_error_capi(ERR_TYPE_COMMON, err);
+       return _image_error_capi(err);
 }
 
 int image_util_extract_color_from_memory(const unsigned char *image_buffer, int width, int height, unsigned char *rgb_r, unsigned char *rgb_g, unsigned char *rgb_b)
@@ -757,7 +758,7 @@ int image_util_extract_color_from_memory(const unsigned char *image_buffer, int
                module = NULL;
        }
 
-       return _image_error_capi(ERR_TYPE_COMMON, ret);
+       return _image_error_capi(ret);
 }
 
 int image_util_foreach_supported_colorspace(image_util_type_e image_type, image_util_supported_colorspace_cb callback, void *user_data)