Fix error return

Change-Id: I62779efdb4bb7c503d6000551c8e05b005f3c6f7
Signed-off-by: SukHyung, Kang <shine.kang@samsung.com>

diff --git a/src/widget_service.c b/src/widget_service.c
index 8cd0cab56d11eff12c4c39aec229ea49bbdefd9e..71270a1f3af17736a70c0d9e409a41c8bb905746 100644 (file)
--- a/src/widget_service.c
+++ b/src/widget_service.c
@@ -2849,7 +2849,7 @@ EAPI int widget_service_get_widget_disabled(const char *widget_id,
                return WIDGET_ERROR_NOT_SUPPORTED;
        }
 
-       if (widget_id == NULL) {
+       if (widget_id == NULL || is_disabled == NULL) {
                _E("Invalid parameter");
                return WIDGET_ERROR_INVALID_PARAMETER;
        }
@@ -2916,8 +2916,6 @@ EAPI int widget_service_set_disable_event_cb(widget_disable_event_cb callback,
 
 EAPI int widget_service_unset_disable_event_cb(void)
 {
-       int ret = WIDGET_ERROR_NONE;
-
        if (!_is_widget_feature_enabled()) {
                _E("not supported");
                return WIDGET_ERROR_NOT_SUPPORTED;
@@ -2927,19 +2925,18 @@ EAPI int widget_service_unset_disable_event_cb(void)
                return WIDGET_ERROR_PERMISSION_DENIED;
 
        if (_conn) {
-               if (aul_app_com_leave(_conn) < 0)
+               if (aul_app_com_leave(_conn) < 0) {
                        _E("failed to leave app com disable");
+                       return WIDGET_ERROR_IO_ERROR;
+               }
                _conn = NULL;
-               ret = WIDGET_ERROR_IO_ERROR;
-               goto out;
        }
 
-out:
        if (_disable_event_cb)
                _disable_event_cb = NULL;
 
        if (_disable_user_data)
                _disable_user_data = NULL;
 
-       return ret;
+       return WIDGET_ERROR_NONE;
 }