Fix svace issue 42/151442/3 accepted/tizen_5.0_unified accepted/tizen_5.5_unified accepted/tizen_5.5_unified_mobile_hotfix accepted/tizen_5.5_unified_wearable_hotfix tizen_5.5 tizen_5.5_mobile_hotfix tizen_5.5_tv tizen_5.5_wearable_hotfix accepted/tizen/5.0/unified/20181106.202202 accepted/tizen/5.5/unified/20191031.033745 accepted/tizen/5.5/unified/mobile/hotfix/20201027.060630 accepted/tizen/5.5/unified/wearable/hotfix/20201027.091833 accepted/tizen/unified/20170922.065710 submit/tizen/20170922.011219 submit/tizen_5.0/20181101.000009 submit/tizen_5.0/20181106.000001 submit/tizen_5.5/20191031.000009 submit/tizen_5.5/20191031.000011 submit/tizen_5.5/20191031.000013 submit/tizen_5.5_mobile_hotfix/20201026.1851010 submit/tizen_5.5_mobile_hotfix/20201027.114301 submit/tizen_5.5_wearable_hotfix/20201026.1843010 submit/tizen_5.5_wearable_hotfix/20201027.114701 tizen_5.5.m2_release
authorJooseok Song <seogii.song@samsung.com>
Thu, 21 Sep 2017 02:16:29 +0000 (11:16 +0900)
committerJooseok Song <seogii.song@samsung.com>
Thu, 21 Sep 2017 11:01:23 +0000 (20:01 +0900)
Change-Id: I5a88d4582e1820522c66021e827e155111bb3de0

src/my-account-view-main.c

index 19c5cbf..9ef2ddd 100644 (file)
@@ -303,6 +303,7 @@ bool __myaccount_get_capablity_cb(const char* capability_type, account_capabilit
 
        myaccount_capability_data *cap_data = (myaccount_capability_data*)malloc(sizeof(myaccount_capability_data));
        if (!cap_data) {
+               myaccount_release_account_info(account_info);
                MA_DEBUG_ERR("malloc failed\n");
                return true;
        }
@@ -320,6 +321,7 @@ bool __myaccount_get_capablity_cb(const char* capability_type, account_capabilit
                        if (strlen(capability_textbuf) > 0) {
                                char *temp2 = strdup(capability_textbuf);
                                if (!temp2) {
+                                       myaccount_release_account_info(account_info);
                                        MA_DEBUG_ERR("strdup returns NULL\n");
                                        return false;
                                }
@@ -331,6 +333,8 @@ bool __myaccount_get_capablity_cb(const char* capability_type, account_capabilit
                }
        }
 
+       myaccount_release_account_info(account_info);
+
        return true;
 }
 
@@ -409,7 +413,7 @@ bool __myaccount_get_account_list_info_cb(account_h account, void *user_data)
        ret = account_get_secret(account, &secret);
        if (ret != ACCOUNT_ERROR_NONE)
                MA_DEBUG_ERR("account_get_secret: Failed \n");
-       else if (secret >= ACCOUNT_SECRECY_INVALID && secret < ACCOUNT_SECRECY_MAX)
+       else if (secret > ACCOUNT_SECRECY_INVALID && secret < ACCOUNT_SECRECY_MAX)
                account_info->secret = secret;
        else
                account_info->secret = ACCOUNT_SECRECY_INVALID;