Check parameter before using it 07/199207/1 accepted/tizen/unified/20190208.061618 submit/tizen/20190207.065213
authorhj kim <backto.kim@samsung.com>
Thu, 7 Feb 2019 07:45:35 +0000 (16:45 +0900)
committerhj kim <backto.kim@samsung.com>
Thu, 7 Feb 2019 07:45:35 +0000 (16:45 +0900)
Change-Id: I0983c27db93bfe6adfef1c2cf0d9dbd60f998e69

src/media_group.c

index 675640c..693cedf 100755 (executable)
@@ -80,10 +80,8 @@ int media_album_get_album_from_db(int album_id, media_album_h *album)
        sqlite3_stmt *stmt = NULL;
        char select_query[DEFAULT_QUERY_SIZE] = {0, };
 
-       if (album_id < 0) {
-               media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       media_content_retvm_if(album_id <= 0, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid album_id [%d]", album_id);
+       media_content_retvm_if(album == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid album");
 
        memset(select_query, 0x00, sizeof(select_query));