diff options
author | Inkyun Kil <inkyun.kil@samsung.com> | 2019-09-20 16:46:04 +0900 |
---|---|---|
committer | Inkyun Kil <inkyun.kil@samsung.com> | 2019-09-23 16:38:47 +0900 |
commit | d0912521a54f8b9c78a7540ea3a7acc248b63ef9 (patch) | |
tree | d5b308d615352f6f0d3749587b696abcc2e0aed8 | |
parent | 7566487c17fb0867d2c6e758d13d859f6494bf47 (diff) | |
download | alarm-manager-d0912521a54f8b9c78a7540ea3a7acc248b63ef9.tar.gz alarm-manager-d0912521a54f8b9c78a7540ea3a7acc248b63ef9.tar.bz2 alarm-manager-d0912521a54f8b9c78a7540ea3a7acc248b63ef9.zip |
Fix bug for checking flag of appid
Change-Id: I2ce4c4b1c047f04a65d17e3681a049af2f6e0c03
Signed-off-by: Inkyun Kil <inkyun.kil@samsung.com>
-rw-r--r-- | lib/alarm-lib.c | 39 |
1 files changed, 15 insertions, 24 deletions
diff --git a/lib/alarm-lib.c b/lib/alarm-lib.c index fcab652..30c7748 100644 --- a/lib/alarm-lib.c +++ b/lib/alarm-lib.c @@ -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; |