Fix bug for checking flag of appid 25/214325/2
authorInkyun Kil <inkyun.kil@samsung.com>
Fri, 20 Sep 2019 07:46:04 +0000 (16:46 +0900)
committerInkyun Kil <inkyun.kil@samsung.com>
Mon, 23 Sep 2019 07:38:47 +0000 (16:38 +0900)
Change-Id: I2ce4c4b1c047f04a65d17e3681a049af2f6e0c03
Signed-off-by: Inkyun Kil <inkyun.kil@samsung.com>
lib/alarm-lib.c

index fcab652b5563659c50c4f7d6449a748c2e2336d2..30c77484a5862802ed071dc3a70c8c911c5b413d 100644 (file)
@@ -500,6 +500,18 @@ static void __adjust_current_milliseconds(alarm_info_t *alarm_info)
        }
 }
 
+static void __check_appid(void)
+{
+       if (flag_appid_checked == 0) {
+               if (aul_app_get_appid_bypid(getpid(), g_appid, sizeof(g_appid)) != AUL_R_OK) {
+                       LOGE("PID[%d] may not be app. Please call alarmmgr_init(caller name) in advance.", getpid());
+               } else {
+                       LOGD("Get appid only once. appid[%s]", g_appid);
+                       flag_appid_checked = 1;
+               }
+       }
+}
+
 EXPORT_API int alarmmgr_init(const char *appid)
 {
        SECURE_LOGD("Enter");
@@ -1354,14 +1366,7 @@ static int _alarmmgr_add_alarm_withcb(int alarm_type, time_t trigger_at_time,
        alarm_info_t alarm_info;
        int ret = 0;
 
-       if (flag_appid_checked == 0) {
-               if (aul_app_get_appid_bypid(getpid(), g_appid, sizeof(g_appid)) != AUL_R_OK)
-                       LOGE("PID[%d] may not be app. Please call alarmmgr_init(caller name) in advance.", getpid());
-               else
-                       LOGD("Get appid only once. appid[%s]", g_appid);
-               flag_appid_checked = 1;
-       }
-
+       __check_appid();
        ret = alarmmgr_init(g_appid);
        if (ret < 0)
                return ret;
@@ -1571,14 +1576,7 @@ EXPORT_API int alarmmgr_add_periodic_alarm_withcb(int interval, periodic_method_
        int error_code = 0;
        int ret = 0;
 
-       if (flag_appid_checked == 0) {
-               if (aul_app_get_appid_bypid(getpid(), g_appid, sizeof(g_appid)) != AUL_R_OK)
-                       LOGE("PID[%d] may not be app. Please call alarmmgr_init(caller name) in advance.", getpid());
-               else
-                       LOGD("Get appid only once. appid[%s]", g_appid);
-               flag_appid_checked = 1;
-       }
-
+       __check_appid();
        ret = alarmmgr_init(g_appid);
        if (ret < 0)
                return ret;
@@ -1604,14 +1602,7 @@ EXPORT_API int alarmmgr_add_reference_periodic_alarm_withcb(int interval,
        int error_code = 0;
        int ret = 0;
 
-       if (flag_appid_checked == 0) {
-               if (aul_app_get_appid_bypid(getpid(), g_appid, sizeof(g_appid)) != AUL_R_OK)
-                       LOGE("PID[%d] may not be app. Please call alarmmgr_init(caller name) in advance.", getpid());
-               else
-                       LOGD("Get appid only once. appid[%s]", g_appid);
-               flag_appid_checked = 1;
-       }
-
+       __check_appid();
        ret = alarmmgr_init(g_appid);
        if (ret < 0)
                return ret;