Fix input value check of screen backlight time 68/256468/1 accepted/tizen/unified/20210406.083531 submit/tizen/20210405.061001
authorJinWang An <jinwang.an@samsung.com>
Mon, 5 Apr 2021 05:54:43 +0000 (14:54 +0900)
committerJinWang An <jinwang.an@samsung.com>
Mon, 5 Apr 2021 05:54:43 +0000 (14:54 +0900)
 - To support setting 0(== always on) for

Change-Id: I8ccbe0d2151b68ca5fb0e1d0d92c7f7df20a4c91
Signed-off-by: JinWang An <jinwang.an@samsung.com>
src/sst_screen.c

index 8825e76..7f1d4ab 100644 (file)
 
 int sst_screen_set_backlight_time(sst_interface *iface, int value)
 {
-       if (value <= 0 || 600 < value) {
+       if (value < 0 || 600 < value) {
                ERR("Invalid Value(%d)", value);
                return SYSTEM_SETTINGS_ERROR_INVALID_PARAMETER;
        }
 
        if (vconf_set_int(iface->vconf_key, value)) {
                ERR("vconf_set_int(%s, %d) Fail", iface->vconf_key, value);
-               return SYSTEM_SETTINGS_ERROR_IO_ERROR;
+               /* vconf_set_int returns error with 0 value,
+                  that means there is no write privilege about lcd back light time */
+               return value? SYSTEM_SETTINGS_ERROR_IO_ERROR: SYSTEM_SETTINGS_ERROR_INVALID_PARAMETER;
        }
 
        return SYSTEM_SETTINGS_ERROR_NONE;