fix privcacy popup problem(type bool)
authorshy81.shin <shy81.shin@samsung.com>
Wed, 17 Jul 2013 10:56:33 +0000 (19:56 +0900)
committershy81.shin <shy81.shin@samsung.com>
Wed, 17 Jul 2013 10:56:33 +0000 (19:56 +0900)
setting-privacy/src/setting-privacy-service.c

index 521d4c7..340a2b9 100755 (executable)
@@ -146,7 +146,7 @@ end:
        return ret;
 }
 
-static void _privacy_info_cb(const privacy_info_h* privacy_info, void* data)
+static bool _privacy_info_cb(const privacy_info_h* privacy_info, void* data)
 {
        SETTING_TRACE_BEGIN;
        SettingPrivacyUG *ad = (SettingPrivacyUG *) data;
@@ -157,24 +157,28 @@ static void _privacy_info_cb(const privacy_info_h* privacy_info, void* data)
        char *privacy_desc = NULL;
        bool enabled = false;
        r = privacy_info_get_privacy_id(privacy_info, &privacy_id);
-       if(r != 0){
+       if(r){
                SETTING_TRACE_ERROR("privacy_info_get_privacy_id failed : %d", r);
+               return false;
        }
        ad->privacy_id = strdup(privacy_id);
 
        r = privacy_info_get_privacy_display_name(privacy_info, &privacy_name);
-       if(r != 0){
+       if(r){
                SETTING_TRACE_ERROR("privacy_info_get_privacy_display_name failed : %d", r);
+               return false;
        }
 
        r = privacy_info_get_privacy_description(privacy_info, &privacy_desc);
-       if(r != 0){
+       if(r){
                SETTING_TRACE_ERROR("privacy_info_get_privacy_description failed : %d", r);
+               return false;
        }
 
        r = privacy_info_is_enabled(privacy_info, &enabled);
-       if(r != 0){
+       if(r){
                SETTING_TRACE_ERROR("privacy_info_is_enabled failed : %d", r);
+               return false;
        }
 
        setting_privacy_privacyinfo_t *app_info = NULL;
@@ -202,14 +206,15 @@ static void _privacy_info_cb(const privacy_info_h* privacy_info, void* data)
                ad->privacy_service->userdata = ad;
        } else {
                SETTING_TRACE_ERROR("ad->privacy_service is NULL");
-               return SETTING_RETURN_FAIL;
+               return false;
        }
 
        ad->privacy_check_list = eina_list_append(ad->privacy_check_list, app_info);
 
-       //FREE(privacy_id);
-       //FREE(privacy_name);
-       //FREE(privacy_desc);
+       FREE(privacy_id);
+       FREE(privacy_name);
+       FREE(privacy_desc);
+       return true;
 }
 
 /**