diff options
author | Michal Bloch <m.bloch@samsung.com> | 2020-12-07 22:09:12 +0100 |
---|---|---|
committer | Karol Lewandowski <k.lewandowsk@samsung.com> | 2020-12-10 09:43:21 +0000 |
commit | 33a66fa787e33519259f429ca7a1025c8f61cf25 (patch) | |
tree | fe61b092ae44d8ee1fd769278da8c8ef48242ea4 | |
parent | da479b98a011748f82b66bf55b0eb0dd651e1fbf (diff) | |
download | peripheral-bus-accepted/tizen_5.5_unified.tar.gz peripheral-bus-accepted/tizen_5.5_unified.tar.bz2 peripheral-bus-accepted/tizen_5.5_unified.zip |
Fix a memleaksubmit/tizen_5.5/20201218.134312accepted/tizen/5.5/unified/20201220.033127tizen_5.5accepted/tizen_5.5_unified
Change-Id: I8387f40aa78f329b3c2f3293765e1f38ae0a6447
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
(cherry picked from commit 394f4ab4498610ac3b8112c7a5ba7b42edc60933)
-rw-r--r-- | src/util/peripheral_privilege.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/util/peripheral_privilege.c b/src/util/peripheral_privilege.c index 335f9b9..443964b 100644 --- a/src/util/peripheral_privilege.c +++ b/src/util/peripheral_privilege.c @@ -82,21 +82,23 @@ int peripheral_privilege_check(GDBusMethodInvocation *invocation, GDBusConnectio if (!session || !client || !user) { _E("Failed to get client info"); - return -1; + ret = -1; + goto cleanup; } ret = cynara_check(__cynara, client, session, user, PERIPHERAL_PRIVILEGE); if (ret != CYNARA_API_ACCESS_ALLOWED) { _E("Failed to check privilege"); - g_free(session); - g_free(client); - g_free(user); - return -EACCES; + ret = -EACCES; + goto cleanup; } + ret = 0; + +cleanup: g_free(session); g_free(client); g_free(user); - return 0; + return ret; } |