From: lokilee73 <changjoo.lee@samsung.com>
Date: Tue, 5 Dec 2017 07:08:42 +0000 (+0900)
Subject: Display: add a flag to prevent duplicated privilege check
X-Git-Tag: submit/tizen_4.0/20171205.105933^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=07cee0dcba690f9490feef66bcbc11d73eb1ff5a;p=platform%2Fcore%2Fapi%2Fdevice.git

Display: add a flag to prevent duplicated privilege check

Change-Id: I78e8be4f5631023f47af571138d26fafa07a73f3
Signed-off-by: lokilee73 <changjoo.lee@samsung.com>
---

diff --git a/src/power.c b/src/power.c
index b585325..d3f6393 100755
--- a/src/power.c
+++ b/src/power.c
@@ -86,6 +86,8 @@ static struct _lock_timeout {
 	.padding = LOCK_CPU_PADDING_TIMEOUT,
 };
 
+static int privilege = -1;
+
 static char *get_state_str(display_state_e state)
 {
 	switch (state) {
@@ -331,6 +333,9 @@ static int lock_state(display_state_e state, unsigned int flag, int timeout_ms)
 	char str_timeout[32];
 	int ret;
 
+	if (privilege == 0)
+		return -EACCES;
+
 	if (flag & GOTO_STATE_NOW)
 		arr[1] = STR_GOTOSTATENOW;
 	else
@@ -346,13 +351,18 @@ static int lock_state(display_state_e state, unsigned int flag, int timeout_ms)
 	snprintf(str_timeout, sizeof(str_timeout), "%d", timeout_ms);
 	arr[3] = str_timeout;
 
-	arr[0] = "privilege check";
-
-	ret = dbus_method_sync(DEVICED_BUS_NAME,
-			DEVICED_PATH_DISPLAY, DEVICED_INTERFACE_DISPLAY,
-			METHOD_LOCK_STATE, "sssi", arr);
-	if (ret == -EACCES || ret == -ECOMM || ret == -EPERM)
-		return -EACCES;
+	if (privilege < 0) {
+		arr[0] = "privilege check";
+
+		ret = dbus_method_sync(DEVICED_BUS_NAME,
+				DEVICED_PATH_DISPLAY, DEVICED_INTERFACE_DISPLAY,
+				METHOD_LOCK_STATE, "sssi", arr);
+		if (ret == -EACCES || ret == -ECOMM || ret == -EPERM) {
+			privilege = 0;
+			return -EACCES;
+		} else
+			privilege = 1;
+	}
 
 	arr[0] = get_state_str(state);
 	if (!arr[0])
@@ -381,6 +391,9 @@ static int unlock_state(display_state_e state, unsigned int flag)
 	char *arr[2];
 	int ret;
 
+	if (privilege == 0)
+		return -EACCES;
+
 	if (flag == PM_SLEEP_MARGIN)
 		arr[1] = STR_SLEEP_MARGIN;
 	else if (flag == PM_RESET_TIMER)
@@ -390,13 +403,18 @@ static int unlock_state(display_state_e state, unsigned int flag)
 	else
 		return -EINVAL;
 
-	arr[0] = "privilege check";
-
-	ret = dbus_method_sync(DEVICED_BUS_NAME,
-			DEVICED_PATH_DISPLAY, DEVICED_INTERFACE_DISPLAY,
-			METHOD_UNLOCK_STATE, "ss", arr);
-	if (ret == -EACCES || ret == -ECOMM || ret == -EPERM)
-		return -EACCES;
+	if (privilege < 0) {
+		arr[0] = "privilege check";
+
+		ret = dbus_method_sync(DEVICED_BUS_NAME,
+				DEVICED_PATH_DISPLAY, DEVICED_INTERFACE_DISPLAY,
+				METHOD_UNLOCK_STATE, "ss", arr);
+		if (ret == -EACCES || ret == -ECOMM || ret == -EPERM) {
+			privilege = 0;
+			return -EACCES;
+		} else
+			privilege = 1;
+	}
 
 	arr[0] = get_state_str(state);
 	if (!arr[0])