power: add handling code to return check 23/287323/2
authorYunhee Seo <yuni.seo@samsung.com>
Thu, 26 Jan 2023 11:19:11 +0000 (20:19 +0900)
committerChanwoo Choi <cw00.choi@samsung.com>
Fri, 27 Jan 2023 03:00:15 +0000 (03:00 +0000)
Add handling code to resolve issue reported by static analysis.

Change-Id: I960daea8e8c670579bf2ab0b98cfb5a5842a9b68
Signed-off-by: Yunhee Seo <yuni.seo@samsung.com>
src/power/power-off.c
src/power/power.c

index de68503..60994b4 100644 (file)
@@ -298,6 +298,7 @@ static void system_shutdown_send_system_event(void)
 void poweroff_prepare(uint64_t state)
 {
        int vconf = VCONFKEY_SYSMAN_POWER_OFF_NONE;
+       int ret;
 
        if (!is_poweroff_state(state))
                return;
@@ -306,7 +307,9 @@ void poweroff_prepare(uint64_t state)
                vconf = VCONFKEY_SYSMAN_POWER_OFF_DIRECT;
        else if (state == DEVICED_POWER_STATE_REBOOT)
                vconf = VCONFKEY_SYSMAN_POWER_OFF_RESTART;
-       vconf_set_int(VCONFKEY_SYSMAN_POWER_OFF_STATUS, vconf);
+       ret = vconf_set_int(VCONFKEY_SYSMAN_POWER_OFF_STATUS, vconf);
+       if (ret < 0)
+               _W("Failed to set vconf value for power off status: %d", vconf_get_ext_errno());
 
        power_disable_autosleep();
        device_notify_once(DEVICE_NOTIFIER_POWEROFF_TRIGGERED, (void *)(intptr_t) vconf);
index 4b60308..9db6eb8 100644 (file)
@@ -704,9 +704,12 @@ static int load_max_wait_timeout(struct parse_result *result, void *user_data)
 
 void power_state_init(void *data)
 {
+       int ret;
        transition_queue = g_queue_new();
 
-       config_parse(POWER_CONF_FILE, load_max_wait_timeout, NULL);
+       ret = config_parse(POWER_CONF_FILE, load_max_wait_timeout, NULL);
+       if (ret < 0)
+               _W("Failed to load '%s'(%d)", POWER_CONF_FILE, ret);
 
        register_notifier(DEVICE_NOTIFIER_DELAYED_INIT, delayed_init_callback);
        register_notifier(DEVICE_NOTIFIER_REQUEST_TRANSITION_STATE, transition_request_callback);