Reduce log 13/63113/1
authorJihoon Kim <jihoon48.kim@samsung.com>
Tue, 22 Mar 2016 06:55:29 +0000 (15:55 +0900)
committerJihoon Kim <jihoon48.kim@samsung.com>
Tue, 22 Mar 2016 06:55:29 +0000 (15:55 +0900)
Change-Id: Ia5b84d56781756c360a62e37f0e25f7966b5e90b

src/inputmethod.cpp
src/privilege_checker.cpp

index 6e2305c..cecc9cf 100644 (file)
@@ -355,8 +355,7 @@ ime_error_e _check_privilege()
 
     snprintf(uid, 16, "%d", getuid());
     if (check_privilege(uid, IME_PRIVILEGE) == false) {
-        LOGE("check_privilege(uid, IME_PRIVILEGE) == false");
-        LOGE("uid : %s.", uid);
+        LOGE("check_privilege(uid, IME_PRIVILEGE) == false. uid : %s", uid);
         ret = IME_ERROR_PERMISSION_DENIED;
     }
 
index bba0f35..ffd6d26 100644 (file)
@@ -26,7 +26,9 @@ bool
 inputmethod_cynara_initialize()
 {
     int ret = cynara_initialize(&p_cynara, NULL);
-    LOGD("[inputmethod_cynara_initialize]_check_privilege returned %d.", ret);
+    if (ret != CYNARA_API_SUCCESS)
+        LOGW("Failed to call cynara_initialize (). ret : %d.", ret);
+
     return ret == CYNARA_API_SUCCESS;
 }
 
@@ -49,7 +51,7 @@ check_privilege(const char *uid, const char *privilege)
         return false;
     }
 
-    fp = fopen("/proc/self/attr/current", "r");
+    fp = fopen(smack_label, "r");
     if (fp != NULL) {
         if (fread(smack_label, 1, sizeof(smack_label), fp) <= 0)
             LOGW("Error : fread");
@@ -60,12 +62,13 @@ check_privilege(const char *uid, const char *privilege)
     pid_t pid = getpid();
     char *session = cynara_session_from_pid(pid);
     int ret = cynara_check(p_cynara, smack_label, session, uid, privilege);
-    LOGD("[check_privilege]_check_privilege returned %d.", ret);
     if (session)
         free(session);
 
-    if (ret != CYNARA_API_ACCESS_ALLOWED)
+    if (ret != CYNARA_API_ACCESS_ALLOWED) {
+        LOGW("Access denied. The result of cynara_check() : %d.", ret);
         return false;
+    }
     return true;
 }