Fix return value when access failure

Change-Id: I0fa8d10fc63b776127fb9ad27c4f53d021d967fa
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

diff --git a/src/thumbnail_util.c b/src/thumbnail_util.c
index 88e42925eaee206e7d866d2cd354de53397662b6..7045c56657d2c4496a9f07972d3aadeec8c325c1 100644 (file)
--- a/src/thumbnail_util.c
+++ b/src/thumbnail_util.c
@@ -602,16 +602,21 @@ int thumbnail_util_extract_to_file(const char *path, unsigned int width, unsigne
 
        /* check thumbnail path is writable */
        check_str = g_path_get_dirname(thumbnail_path);
-       if (check_str != NULL) {
-               if (access(check_str, W_OK) != 0) {
+       if (access(check_str, W_OK) != 0) {
+               if (errno == EACCES || errno == EPERM) {
                        thumbnail_util_error("No permission to write[%s]", check_str);
-                       SAFE_FREE(check_str);
-                       return THUMBNAIL_UTIL_ERROR_PERMISSION_DENIED;
+                       ret = THUMBNAIL_UTIL_ERROR_PERMISSION_DENIED;
                } else {
-                       SAFE_FREE(check_str);
+                       thumbnail_util_error("Does not exists[%s]", check_str);
+                       ret = THUMBNAIL_UTIL_ERROR_INVALID_PARAMETER;
                }
+
+               SAFE_FREE(check_str);
+               return ret;
        }
 
+       SAFE_FREE(check_str);
+
        /* If video file, thumbnail extension is only JPEG */
        if (type == THUMBNAIL_UTIL_VIDEO) {
                char ext[255] = { 0 };