Make thread-safe of hasPermission() with cynara

cynara is not thread-safe so locking mechanism is needed on our side.

Change-Id: Ica0105dc74a9bd3d76e73339d82831181c1469db
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

diff --git a/src/framework/service/access-control-cynara.cpp b/src/framework/service/access-control-cynara.cpp
index 9650e12..d0de0fe 100644 (file)
--- a/src/framework/service/access-control-cynara.cpp
+++ b/src/framework/service/access-control-cynara.cpp
@@ -65,8 +65,13 @@ public:
        bool request(const std::string &user, const std::string &client,
                                 const std::string &session, const std::string &privilege)
        {
-               auto ret = cynara_check(this->m_cynara, client.c_str(), session.c_str(),
-                                                               user.c_str(), privilege.c_str());
+               int ret = CYNARA_API_ACCESS_DENIED;
+
+               {
+                       std::lock_guard<std::mutex> l(this->m_mutex);
+                       ret = cynara_check(this->m_cynara, client.c_str(), session.c_str(),
+                                                          user.c_str(), privilege.c_str());
+               }
 
                switch (ret) {
                case CYNARA_API_ACCESS_ALLOWED: return true;