From 4e24fd3057e41f64fa3e2402365ca8915e984e4f Mon Sep 17 00:00:00 2001 From: Soham Date: Fri, 13 Oct 2023 11:32:58 +0530 Subject: [PATCH] Coverage Improvement of Contacts Service Change-Id: I84d703c0dac6d15a94b4d97fc1e490035e604e49 Signed-off-by: Soham --- common/ctsvc_handle.c | 5 +- common/ctsvc_image_util.c | 12 +-- common/ctsvc_localize_utils.c | 6 ++ common/ctsvc_record_group.c | 21 +++-- common/ctsvc_record_person.c | 26 +++---- common/ctsvc_record_phonelog.c | 24 +++--- common/ctsvc_record_speeddial.c | 18 ++--- common/ctsvc_socket.c | 13 ++-- common/ctsvc_vcard.c | 167 ++++++++++++++++++++++++++++++++-------- server/ctsvc_localize_ch.c | 12 +-- server/ctsvc_localize_jp.c | 18 +---- server/ctsvc_localize_kor.c | 4 +- server/ctsvc_normalize.c | 24 +++--- server/ctsvc_number_utils.c | 51 ++++++------ server/ctsvc_schema_recovery.c | 14 ++-- server/ctsvc_server_phonelog.c | 26 ++++--- server/ctsvc_server_service.c | 9 ++- server/ctsvc_server_setting.c | 22 +++++- server/ctsvc_server_sim.c | 74 ++---------------- server/ctsvc_server_socket.c | 31 ++------ server/ctsvc_server_sqlite.c | 72 +---------------- server/ctsvc_server_update.c | 18 +---- server/ctsvc_server_utils.c | 12 +-- server/ctsvc_utils_string.c | 37 +++++++-- 24 files changed, 349 insertions(+), 367 deletions(-) diff --git a/common/ctsvc_handle.c b/common/ctsvc_handle.c index 2f31d09..729ed33 100644 --- a/common/ctsvc_handle.c +++ b/common/ctsvc_handle.c @@ -42,7 +42,7 @@ int ctsvc_handle_destroy(contacts_h contact) base = NULL; return CONTACTS_ERROR_NONE; } - +/* LCOV_EXCL_START */ int ctsvc_handle_clone(contacts_h contact, contacts_h *pcontact) { RETV_IF(NULL == contact, CONTACTS_ERROR_INVALID_PARAMETER); @@ -51,10 +51,8 @@ int ctsvc_handle_clone(contacts_h contact, contacts_h *pcontact) ctsvc_base_s *base = (ctsvc_base_s*)contact; ctsvc_base_s *clone = calloc(1, sizeof(ctsvc_base_s)); if (NULL == clone) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } clone->connection_count = base->connection_count; clone->version = base->version; @@ -63,3 +61,4 @@ int ctsvc_handle_clone(contacts_h contact, contacts_h *pcontact) return CONTACTS_ERROR_NONE; } +/* LCOV_EXCL_STOP */ diff --git a/common/ctsvc_image_util.c b/common/ctsvc_image_util.c index 2658346..539eb4f 100644 --- a/common/ctsvc_image_util.c +++ b/common/ctsvc_image_util.c @@ -39,6 +39,7 @@ int ctsvc_image_util_get_mimetype(image_util_colorspace_e colorspace, case IMAGE_UTIL_COLORSPACE_YUV422: mimetype = MEDIA_FORMAT_422P; break; + /* LCOV_EXCL_START */ case IMAGE_UTIL_COLORSPACE_NV12: mimetype = MEDIA_FORMAT_NV12; break; @@ -57,9 +58,11 @@ int ctsvc_image_util_get_mimetype(image_util_colorspace_e colorspace, case IMAGE_UTIL_COLORSPACE_ARGB8888: mimetype = MEDIA_FORMAT_ARGB; break; + /* LCOV_EXCL_STOP */ case IMAGE_UTIL_COLORSPACE_RGBA8888: mimetype = MEDIA_FORMAT_RGBA; break; + /* LCOV_EXCL_START */ case IMAGE_UTIL_COLORSPACE_NV21: mimetype = MEDIA_FORMAT_NV21; break; @@ -67,6 +70,7 @@ int ctsvc_image_util_get_mimetype(image_util_colorspace_e colorspace, mimetype = MEDIA_FORMAT_NV16; break; case IMAGE_UTIL_COLORSPACE_BGRA8888: /* not supported */ + /* LCOV_EXCL_STOP */ case IMAGE_UTIL_COLORSPACE_BGRX8888: /* not supported */ case IMAGE_UTIL_COLORSPACE_NV61: /* not supported */ case IMAGE_UTIL_COLORSPACE_YV12: /* not supported */ @@ -342,7 +346,7 @@ static int _ctsvc_image_util_transform_run(transformation_h transform, return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ int ctsvc_image_util_rotate(media_packet_h packet, image_util_rotation_e rotation, void **p_buffer, uint64_t *p_size) { @@ -351,19 +355,15 @@ int ctsvc_image_util_rotate(media_packet_h packet, image_util_rotation_e rotatio ret = image_util_transform_create(&transform); if (IMAGE_UTIL_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("image_util_transform_create() Fail(%d)", ret); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } ret = image_util_transform_set_rotation(transform, rotation); if (IMAGE_UTIL_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("image_util_transform_set_rotation() Fail(%d)", ret); image_util_transform_destroy(transform); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } ret = _ctsvc_image_util_transform_run(transform, packet, p_buffer, p_size); @@ -371,7 +371,7 @@ int ctsvc_image_util_rotate(media_packet_h packet, image_util_rotation_e rotatio image_util_transform_destroy(transform); return ret; } - + /* LCOV_EXCL_STOP */ int ctsvc_image_util_resize(media_packet_h packet, int width, int height, void **p_buffer, uint64_t *p_size) { diff --git a/common/ctsvc_localize_utils.c b/common/ctsvc_localize_utils.c index 673e373..9448a2f 100644 --- a/common/ctsvc_localize_utils.c +++ b/common/ctsvc_localize_utils.c @@ -28,6 +28,7 @@ int ctsvc_check_utf8(char c) { if ((c & 0xff) < (128 & 0xff)) return 1; + /* LCOV_EXCL_START */ else if ((c & (char)0xe0) == (char)0xc0) return 2; else if ((c & (char)0xf0) == (char)0xe0) @@ -40,6 +41,7 @@ int ctsvc_check_utf8(char c) return 6; else return CONTACTS_ERROR_INVALID_PARAMETER; + /* LCOV_EXCL_STOP */ } /** @@ -102,16 +104,20 @@ int ctsvc_check_language(UChar *word) break; /* korean */ + /* LCOV_EXCL_START */ case UBLOCK_HANGUL_JAMO: /* =30, [1100] */ + /* LCOV_EXCL_STOP */ case UBLOCK_HANGUL_COMPATIBILITY_JAMO: /* =65, [3130] */ case UBLOCK_HANGUL_SYLLABLES: /* =74, [AC00] */ case UBLOCK_HANGUL_JAMO_EXTENDED_A: /* =180, [A960] */ case UBLOCK_HANGUL_JAMO_EXTENDED_B: /* =185, [D7B0] */ + /* LCOV_EXCL_START */ type = CTSVC_LANG_KOREAN; break; /* chainese */ case UBLOCK_CJK_RADICALS_SUPPLEMENT: /* =58, [2E80] */ + /* LCOV_EXCL_STOP */ case UBLOCK_CJK_SYMBOLS_AND_PUNCTUATION: /* =61, [3000] */ case UBLOCK_ENCLOSED_CJK_LETTERS_AND_MONTHS: /* =68, [3200] */ case UBLOCK_CJK_STROKES: /* =130, [31C0] */ diff --git a/common/ctsvc_record_group.c b/common/ctsvc_record_group.c index 800d1ca..98a5c1f 100644 --- a/common/ctsvc_record_group.c +++ b/common/ctsvc_record_group.c @@ -117,7 +117,7 @@ static int __ctsvc_group_clone(contacts_record_h record, contacts_record_h *out_ *out_record = (contacts_record_h)out_data; return CONTACTS_ERROR_NONE; } - +/* LCOV_EXCL_START */ static int __ctsvc_group_get_int(contacts_record_h record, unsigned int property_id, int *out) { ctsvc_group_s *group = (ctsvc_group_s*)record; @@ -130,14 +130,12 @@ static int __ctsvc_group_get_int(contacts_record_h record, unsigned int property *out = group->addressbook_id; break; default: - /* LCOV_EXCL_START */ ERR("property_id(%d) is not supported in value(group)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - +/* LCOV_EXCL_STOP */ static int __ctsvc_group_get_str_real(contacts_record_h record, unsigned int property_id, char **out_str, bool copy) { ctsvc_group_s *group = (ctsvc_group_s*)record; @@ -161,8 +159,8 @@ static int __ctsvc_group_get_str_real(contacts_record_h record, unsigned int pro case CTSVC_PROPERTY_GROUP_EXTRA_DATA: *out_str = GET_STR(copy, group->extra_data); break; - default: /* LCOV_EXCL_START */ + default: ERR("property_id(%d) is not supported in value(group)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ @@ -185,18 +183,20 @@ static int __ctsvc_group_set_int(contacts_record_h record, unsigned int property ctsvc_group_s *group = (ctsvc_group_s*)record; switch (property_id) { + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_GROUP_ID: CHECK_DIRTY_VAL(group->id, value, is_dirty); group->id = value; break; + /* LCOV_EXCL_STOP */ case CTSVC_PROPERTY_GROUP_ADDRESSBOOK_ID: RETVM_IF(0 < group->id, CONTACTS_ERROR_INVALID_PARAMETER, "property_id(%d) is a read-only value (group)", property_id); CHECK_DIRTY_VAL(group->addressbook_id, value, is_dirty); group->addressbook_id = value; break; - default: /* LCOV_EXCL_START */ + default: ERR("property_id(%d) is not supported in value(group)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ @@ -233,15 +233,15 @@ static int __ctsvc_group_set_str(contacts_record_h record, unsigned int property CHECK_DIRTY_STR(group->extra_data, str, is_dirty); FREEandSTRDUP(group->extra_data, str); break; - default: /* LCOV_EXCL_START */ + default: ERR("property_id(%d) is not supported in value(group)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static int __ctsvc_group_get_bool(contacts_record_h record, unsigned int property_id, bool *value) { ctsvc_group_s *group = (ctsvc_group_s*)record; @@ -250,10 +250,8 @@ static int __ctsvc_group_get_bool(contacts_record_h record, unsigned int propert *value = group->is_read_only; break; default: - /* LCOV_EXCL_START */ ERR("property_id(%d) is not supported in value(group)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } @@ -270,10 +268,9 @@ static int __ctsvc_group_set_bool(contacts_record_h record, unsigned int propert group->is_read_only = value; break; default: - /* LCOV_EXCL_START */ ERR("property_id(%d) is not supported in value(group)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } + /* LCOV_EXCL_STOP */ \ No newline at end of file diff --git a/common/ctsvc_record_person.c b/common/ctsvc_record_person.c index b13dd88..ba3d4d3 100644 --- a/common/ctsvc_record_person.c +++ b/common/ctsvc_record_person.c @@ -138,6 +138,7 @@ static int __ctsvc_person_get_int(contacts_record_h record, unsigned int propert case CTSVC_PROPERTY_PERSON_DISPLAY_CONTACT_ID: *out = person->name_contact_id; break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PERSON_LINK_COUNT: *out = person->link_count; break; @@ -145,14 +146,13 @@ static int __ctsvc_person_get_int(contacts_record_h record, unsigned int propert *out = person->snippet_type; break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(person)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static int __ctsvc_person_get_double(contacts_record_h record, unsigned int property_id, double *out) { @@ -163,14 +163,12 @@ static int __ctsvc_person_get_double(contacts_record_h record, unsigned int prop break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(person)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_STOP */ static int __ctsvc_person_get_str_real(contacts_record_h record, unsigned int property_id, char **out_str, bool copy) { @@ -182,15 +180,18 @@ static int __ctsvc_person_get_str_real(contacts_record_h record, unsigned int pr case CTSVC_PROPERTY_PERSON_RINGTONE: *out_str = GET_STR(copy, person->ringtone_path); break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PERSON_IMAGE_THUMBNAIL: *out_str = GET_STR(copy, person->image_thumbnail_path); break; + /* LCOV_EXCL_STOP */ case CTSVC_PROPERTY_PERSON_VIBRATION: *out_str = GET_STR(copy, person->vibration); break; case CTSVC_PROPERTY_PERSON_MESSAGE_ALERT: *out_str = GET_STR(copy, person->message_alert); break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PERSON_STATUS: *out_str = GET_STR(copy, person->status); break; @@ -204,7 +205,6 @@ static int __ctsvc_person_get_str_real(contacts_record_h record, unsigned int pr *out_str = GET_STR(copy, person->snippet_string); break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(person)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ @@ -232,6 +232,7 @@ static int __ctsvc_person_get_bool(contacts_record_h record, unsigned int proper case CTSVC_PROPERTY_PERSON_IS_FAVORITE: *value = person->is_favorite; break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PERSON_HAS_PHONENUMBER: *value = person->has_phonenumber; break; @@ -239,14 +240,13 @@ static int __ctsvc_person_get_bool(contacts_record_h record, unsigned int proper *value = person->has_email; break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(company)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static int __ctsvc_person_set_int(contacts_record_h record, unsigned int property_id, int value, bool *is_dirty) { @@ -270,20 +270,19 @@ static int __ctsvc_person_set_int(contacts_record_h record, unsigned int propert person->snippet_type = value; break; default: - /* LCOV_EXCL_START */ ERR("This field(0x%0x) is not supported in value(person)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_STOP */ static int __ctsvc_person_set_str(contacts_record_h record, unsigned int property_id, const char *str, bool *is_dirty) { ctsvc_person_s *person = (ctsvc_person_s*)record; switch (property_id) { + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PERSON_DISPLAY_NAME: CHECK_DIRTY_STR(person->display_name, str, is_dirty); FREEandSTRDUP(person->display_name, str); @@ -296,6 +295,7 @@ static int __ctsvc_person_set_str(contacts_record_h record, unsigned int propert CHECK_DIRTY_STR(person->image_thumbnail_path, str, is_dirty); FREEandSTRDUP(person->image_thumbnail_path, str); break; + /* LCOV_EXCL_STOP */ case CTSVC_PROPERTY_PERSON_RINGTONE: CHECK_DIRTY_STR(person->ringtone_path, str, is_dirty); FREEandSTRDUP(person->ringtone_path, str); @@ -308,12 +308,12 @@ static int __ctsvc_person_set_str(contacts_record_h record, unsigned int propert CHECK_DIRTY_STR(person->message_alert, str, is_dirty); FREEandSTRDUP(person->message_alert, str); break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PERSON_SNIPPET_STRING: CHECK_DIRTY_STR(person->snippet_string, str, is_dirty); FREEandSTRDUP(person->snippet_string, str); break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(person)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ @@ -332,8 +332,8 @@ static int __ctsvc_person_set_bool(contacts_record_h record, unsigned int proper if (person->is_favorite != value) person->is_favorite = value; break; - default: /* LCOV_EXCL_START */ + default: ERR("This field(%d) is not supported in value(person)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ diff --git a/common/ctsvc_record_phonelog.c b/common/ctsvc_record_phonelog.c index bc0ab36..0367766 100644 --- a/common/ctsvc_record_phonelog.c +++ b/common/ctsvc_record_phonelog.c @@ -116,6 +116,7 @@ static int __ctsvc_phonelog_get_int(contacts_record_h record, unsigned int prope ctsvc_phonelog_s *phonelog = (ctsvc_phonelog_s*)record; switch (property_id) { + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PHONELOG_ID: *out = phonelog->id; break; @@ -125,9 +126,11 @@ static int __ctsvc_phonelog_get_int(contacts_record_h record, unsigned int prope case CTSVC_PROPERTY_PHONELOG_LOG_TIME: *out = phonelog->log_time; break; + /* LCOV_EXCL_STOP */ case CTSVC_PROPERTY_PHONELOG_LOG_TYPE: *out = phonelog->log_type; break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PHONELOG_EXTRA_DATA1: *out = phonelog->extra_data1; break; @@ -135,14 +138,13 @@ static int __ctsvc_phonelog_get_int(contacts_record_h record, unsigned int prope *out = phonelog->sim_slot_no; break; default: - /* LCOV_EXCL_START */ ERR("property_id(%d) is not supported in value(phonelog)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static int __ctsvc_phonelog_get_str_real(contacts_record_h record, unsigned int property_id, char **out_str, bool copy) { ctsvc_phonelog_s *phonelog = (ctsvc_phonelog_s*)record; @@ -155,10 +157,8 @@ static int __ctsvc_phonelog_get_str_real(contacts_record_h record, unsigned int *out_str = GET_STR(copy, phonelog->extra_data2); break; default: - /* LCOV_EXCL_START */ ERR("property_id(%d) is not supported in value(phonelog)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } @@ -172,12 +172,13 @@ static int __ctsvc_phonelog_get_str(contacts_record_h record, unsigned int prope { return __ctsvc_phonelog_get_str_real(record, property_id, out_str, true); } - + /* LCOV_EXCL_STOP */ static int __ctsvc_phonelog_set_int(contacts_record_h record, unsigned int property_id, int value, bool *is_dirty) { ctsvc_phonelog_s *phonelog = (ctsvc_phonelog_s*)record; switch (property_id) { + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PHONELOG_ID: CHECK_DIRTY_VAL(phonelog->id, value, is_dirty); phonelog->id = value; @@ -194,21 +195,27 @@ static int __ctsvc_phonelog_set_int(contacts_record_h record, unsigned int prope CHECK_DIRTY_VAL(phonelog->log_time, value, is_dirty); phonelog->log_time = value; break; + /* LCOV_EXCL_STOP */ case CTSVC_PROPERTY_PHONELOG_LOG_TYPE: if ((CONTACTS_PLOG_TYPE_NONE <= value && value <= CONTACTS_PLOG_TYPE_VIDEO_BLOCKED) || (CONTACTS_PLOG_TYPE_MMS_INCOMING <= value && value <= CONTACTS_PLOG_TYPE_MMS_BLOCKED) + /* LCOV_EXCL_START */ || (CONTACTS_PLOG_TYPE_EMAIL_RECEIVED <= value && value <= CONTACTS_PLOG_TYPE_EMAIL_SENT) + /* LCOV_EXCL_STOP */ ) { CHECK_DIRTY_VAL(phonelog->log_type, value, is_dirty); phonelog->log_type = value; } else { + /* LCOV_EXCL_START */ ERR("log type is in invalid range (%d)", value); return CONTACTS_ERROR_INVALID_PARAMETER; + /* LCOV_EXCL_STOP */ } break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_PHONELOG_EXTRA_DATA1: RETVM_IF(0 < phonelog->id, CONTACTS_ERROR_INVALID_PARAMETER, "property_id(%d) is a read-only value (phonelog)", property_id); @@ -222,14 +229,13 @@ static int __ctsvc_phonelog_set_int(contacts_record_h record, unsigned int prope phonelog->sim_slot_no = value; break; default: - /* LCOV_EXCL_START */ ERR("property_id(%d) is not supported in value(phonelog)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static int __ctsvc_phonelog_set_str(contacts_record_h record, unsigned int property_id, const char *str, bool *is_dirty) { ctsvc_phonelog_s *phonelog = (ctsvc_phonelog_s*)record; @@ -246,11 +252,9 @@ static int __ctsvc_phonelog_set_str(contacts_record_h record, unsigned int prope FREEandSTRDUP(phonelog->extra_data2, str); break; default: - /* LCOV_EXCL_START */ ERR("property_id(%d) is not supported in value(phonelog)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_STOP */ diff --git a/common/ctsvc_record_speeddial.c b/common/ctsvc_record_speeddial.c index 3c1f31e..2b411ca 100644 --- a/common/ctsvc_record_speeddial.c +++ b/common/ctsvc_record_speeddial.c @@ -119,6 +119,7 @@ static int __ctsvc_speeddial_get_int(contacts_record_h record, unsigned int prop case CTSVC_PROPERTY_SPEEDDIAL_DIAL_NUMBER: *out = speeddial->dial_number; break; + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_SPEEDDIAL_NUMBER_ID: *out = speeddial->number_id; break; @@ -129,14 +130,13 @@ static int __ctsvc_speeddial_get_int(contacts_record_h record, unsigned int prop *out = speeddial->person_id; break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(speeddial)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static int __ctsvc_speeddial_get_str_real(contacts_record_h record, unsigned int property_id, char **out_str, bool copy) { ctsvc_speeddial_s *speeddial = (ctsvc_speeddial_s*)record; @@ -155,14 +155,11 @@ static int __ctsvc_speeddial_get_str_real(contacts_record_h record, unsigned int *out_str = GET_STR(copy, speeddial->label); break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(speeddial)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - static int __ctsvc_speeddial_get_str_p(contacts_record_h record, unsigned int property_id, char **out_str) { return __ctsvc_speeddial_get_str_real(record, property_id, out_str, false); @@ -172,12 +169,13 @@ static int __ctsvc_speeddial_get_str(contacts_record_h record, unsigned int prop { return __ctsvc_speeddial_get_str_real(record, property_id, out_str, true); } - + /* LCOV_EXCL_STOP */ static int __ctsvc_speeddial_set_int(contacts_record_h record, unsigned int property_id, int value, bool *is_dirty) { ctsvc_speeddial_s *speeddial = (ctsvc_speeddial_s*)record; switch (property_id) { + /* LCOV_EXCL_START */ case CTSVC_PROPERTY_SPEEDDIAL_NUMBER_TYPE: CHECK_DIRTY_VAL(speeddial->number_type, value, is_dirty); speeddial->number_type = value; @@ -186,6 +184,7 @@ static int __ctsvc_speeddial_set_int(contacts_record_h record, unsigned int prop CHECK_DIRTY_VAL(speeddial->person_id, value, is_dirty); speeddial->person_id = value; break; + /* LCOV_EXCL_STOP */ case CTSVC_PROPERTY_SPEEDDIAL_DIAL_NUMBER: RETVM_IF(0 < speeddial->id, CONTACTS_ERROR_INVALID_PARAMETER, "property_id(%d) is a read-only value (speeddial)", property_id); @@ -196,15 +195,15 @@ static int __ctsvc_speeddial_set_int(contacts_record_h record, unsigned int prop CHECK_DIRTY_VAL(speeddial->number_id, value, is_dirty); speeddial->number_id = value; break; - default: /* LCOV_EXCL_START */ + default: ERR("This field(%d) is not supported in value(speeddial)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static int __ctsvc_speeddial_set_str(contacts_record_h record, unsigned int property_id, const char *str, bool *is_dirty) { ctsvc_speeddial_s *speeddial = (ctsvc_speeddial_s*)record; @@ -227,10 +226,9 @@ static int __ctsvc_speeddial_set_str(contacts_record_h record, unsigned int prop FREEandSTRDUP(speeddial->label, str); break; default: - /* LCOV_EXCL_START */ ERR("This field(%d) is not supported in value(speeddial)", property_id); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } + /* LCOV_EXCL_STOP */ \ No newline at end of file diff --git a/common/ctsvc_socket.c b/common/ctsvc_socket.c index 5920f8a..59affca 100644 --- a/common/ctsvc_socket.c +++ b/common/ctsvc_socket.c @@ -41,8 +41,10 @@ int ctsvc_socket_init(void) WARN("ctsvc_socket_init: socket ref count : %d", __ctsvc_conn_refcnt); if (0 < __ctsvc_conn_refcnt) { + /* LCOV_EXCL_START */ __ctsvc_conn_refcnt++; return CONTACTS_ERROR_NONE; + /* LCOV_EXCL_STOP */ } char sock_file[CTSVC_PATH_MAX_LEN] = {0}; @@ -88,9 +90,11 @@ int ctsvc_socket_init(void) void ctsvc_socket_final(void) { if (1 < __ctsvc_conn_refcnt) { + /* LCOV_EXCL_START */ DBG("socket ref count : %d", __ctsvc_conn_refcnt); __ctsvc_conn_refcnt--; return; + /* LCOV_EXCL_STOP */ } else if (__ctsvc_conn_refcnt < 1) { DBG("Please call connection API. socket ref count : %d", __ctsvc_conn_refcnt); return; @@ -100,7 +104,7 @@ void ctsvc_socket_final(void) close(__ctsvc_sockfd); __ctsvc_sockfd = -1; } - + /* LCOV_EXCL_START */ static inline int __ctsvc_safe_write(int fd, const char *buf, int buf_size) { int ret, writed = 0; @@ -196,7 +200,6 @@ static int __ctsvc_client_socket_response_handler(int fd, ctsvc_socket_msg_s *ms } break; - /* LCOV_EXCL_START */ default: WARN("Unknown Type(%d), ret=%d, attach_num= %d," "attach1 = %d, attach2 = %d, attach3 = %d, attach4 = %d", @@ -204,7 +207,6 @@ static int __ctsvc_client_socket_response_handler(int fd, ctsvc_socket_msg_s *ms msg->attach_sizes[0], msg->attach_sizes[1], msg->attach_sizes[2], msg->attach_sizes[3]); break; - /* LCOV_EXCL_STOP */ } } while (CTSVC_SOCKET_MSG_TYPE_REQUEST_RETURN_VALUE != msg->type); @@ -221,29 +223,24 @@ static void __ctsvc_remove_invalid_msg(int fd, int size) ret = read(fd, dummy, sizeof(dummy)); if (-1 == ret) { - /* LCOV_EXCL_START */ if (EINTR == errno) continue; else return; - /* LCOV_EXCL_STOP */ } size -= ret; } else { ret = read(fd, dummy, size); if (-1 == ret) { - /* LCOV_EXCL_START */ if (EINTR == errno) continue; else return; - /* LCOV_EXCL_STOP */ } size -= ret; } } } -/* LCOV_EXCL_START */ int ctsvc_request_sim_import(int sim_slot_no, contacts_sim_import_progress_cb callback, void *user_data) { diff --git a/common/ctsvc_vcard.c b/common/ctsvc_vcard.c index 538d528..d05c8ea 100644 --- a/common/ctsvc_vcard.c +++ b/common/ctsvc_vcard.c @@ -228,6 +228,7 @@ static int __ctsvc_vcard_append_str(char **buf, int *buf_size, int len, const ch while (*s) { switch (*s) { +/* LCOV_EXCL_START */ case '\r': if (*(s+1) && '\n' == *(s+1)) { s++; @@ -253,6 +254,7 @@ static int __ctsvc_vcard_append_str(char **buf, int *buf_size, int len, const ch } *r = 'n'; break; +/* LCOV_EXCL_STOP */ case ';': case ':': case ',': @@ -262,6 +264,7 @@ static int __ctsvc_vcard_append_str(char **buf, int *buf_size, int len, const ch *r = '\\'; r++; str_len++; +/* LCOV_EXCL_START */ if (*buf_size < str_len+len+1) { *buf_size = *buf_size * 2; if (NULL == (tmp = realloc(*buf, *buf_size))) { @@ -271,8 +274,10 @@ static int __ctsvc_vcard_append_str(char **buf, int *buf_size, int len, const ch r = (char *)(*buf+len+str_len); } } +/* LCOV_EXCL_STOP */ *r = *s; break; +/* LCOV_EXCL_START */ case 0xA1: if (*(s+1) && 0xAC == *(s+1)) { /* en/em backslash */ *r = '\\'; @@ -337,6 +342,7 @@ static int __ctsvc_vcard_append_str(char **buf, int *buf_size, int len, const ch *r = *s; } break; +/* LCOV_EXCL_STOP */ default: *r = *s; break; @@ -502,6 +508,7 @@ static inline const char* __ctsvc_get_img_suffix(int type) switch (type) { case CTSVC_VCARD_IMG_TIFF: return "tiff"; + /* LCOV_EXCL_START */ case CTSVC_VCARD_IMG_GIF: return "gif"; case CTSVC_VCARD_IMG_PNG: @@ -532,6 +539,7 @@ static inline const char* __ctsvc_get_img_suffix(int type) return "mpeg2"; case CTSVC_VCARD_IMG_AVI: return "avi"; + /* LCOV_EXCL_STOP */ case CTSVC_VCARD_IMG_JPEG: case CTSVC_VCARD_IMG_NONE: default: @@ -562,7 +570,7 @@ static inline int __ctsvc_vcard_get_image_type(const char *val) free(image_path); return CTSVC_VCARD_IMG_JPEG; } - + /* LCOV_EXCL_START */ result = strstr(image_path, "png"); if (result) { free(image_path); @@ -661,6 +669,7 @@ static inline int __ctsvc_vcard_get_image_type(const char *val) free(image_path); return CTSVC_VCARD_IMG_NONE; + /* LCOV_EXCL_STOP */ } static inline const char* __ctsvc_get_image_type_str(int type) @@ -668,6 +677,7 @@ static inline const char* __ctsvc_get_image_type_str(int type) switch (type) { case CTSVC_VCARD_IMG_TIFF: return "TIFF"; + /* LCOV_EXCL_START */ case CTSVC_VCARD_IMG_GIF: return "GIF"; case CTSVC_VCARD_IMG_PNG: @@ -698,6 +708,7 @@ static inline const char* __ctsvc_get_image_type_str(int type) return "MPEG2"; case CTSVC_VCARD_IMG_AVI: return "AVI"; + /* LCOV_EXCL_STOP */ case CTSVC_VCARD_IMG_JPEG: default: return "JPEG"; @@ -760,7 +771,7 @@ static inline int __ctsvc_vcard_put_company_logo(const char *path, char **buf, i } return len; } - + /* LCOV_EXCL_START */ static bool __ctsvc_vcard_is_valid_custom_label(char *label) { char *src = label; @@ -778,16 +789,20 @@ static bool __ctsvc_vcard_is_valid_custom_label(char *label) } return true; } - + /* LCOV_EXCL_STOP */ static inline int __ctsvc_vcard_put_company_type(int type, char *label, char **buf, int *buf_size, int len) { if (type == CONTACTS_COMPANY_TYPE_WORK) { + /* LCOV_EXCL_START */ CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE=WORK"); + /* LCOV_EXCL_STOP */ } else if (type == CONTACTS_COMPANY_TYPE_CUSTOM) { + /* LCOV_EXCL_START */ if (__ctsvc_vcard_is_valid_custom_label(label)) { CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE=X-"); CTSVC_VCARD_APPEND_STR(buf, buf_size, len, label); } + /* LCOV_EXCL_STOP */ } return len; } @@ -900,19 +915,22 @@ static inline int __ctsvc_vcard_put_postal_type(int type, char *label, char **bu type_str = "INTL"; else if (type == CONTACTS_ADDRESS_TYPE_HOME) type_str = "HOME"; + /* LCOV_EXCL_START */ else if (type == CONTACTS_ADDRESS_TYPE_WORK) type_str = "WORK"; else if (type == CONTACTS_ADDRESS_TYPE_POSTAL) type_str = "POSTAL"; else if (type == CONTACTS_ADDRESS_TYPE_PARCEL) type_str = "PARCEL"; - + /* LCOV_EXCL_STOP */ if (type == CONTACTS_ADDRESS_TYPE_CUSTOM) { + /* LCOV_EXCL_START */ if (__ctsvc_vcard_is_valid_custom_label(label)) { CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE=X-"); CTSVC_VCARD_APPEND_STR(buf, buf_size, len, label); } return len; + /* LCOV_EXCL_STOP */ } if (type_str) { @@ -1028,6 +1046,7 @@ static inline int __ctsvc_vcard_put_number_type(int type, char *label, char **bu { if (type & CONTACTS_NUMBER_TYPE_HOME) CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE=HOME"); + /* LCOV_EXCL_START */ if (type & CONTACTS_NUMBER_TYPE_MSG) CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE=MSG"); if (type & CONTACTS_NUMBER_TYPE_WORK) @@ -1067,6 +1086,7 @@ static inline int __ctsvc_vcard_put_number_type(int type, char *label, char **bu } return len; } + /* LCOV_EXCL_STOP */ return len; } @@ -1074,6 +1094,7 @@ static int __ctsvc_vcard_check_utf8(char c) { if ((c & 0xff) < (128 & 0xff)) return 1; + /* LCOV_EXCL_START */ else if ((c & (char)0xe0) == (char)0xc0) return 2; else if ((c & (char)0xf0) == (char)0xe0) @@ -1084,6 +1105,7 @@ static int __ctsvc_vcard_check_utf8(char c) return 5; else if ((c & (char)0xfe) == (char)0xfc) return 6; + /* LCOV_EXCL_STOP */ else return CONTACTS_ERROR_INVALID_PARAMETER; /* LCOV_EXCL_LINE */ } @@ -1098,6 +1120,7 @@ static void __ctsvc_vcard_get_clean_number_for_export(char *str, char *dest) while (*s) { char_len = __ctsvc_vcard_check_utf8(*s); + /* LCOV_EXCL_START */ if (3 == char_len) { if (*s == 0xef) { if (*(s+1) == 0xbc) { @@ -1140,6 +1163,7 @@ static void __ctsvc_vcard_get_clean_number_for_export(char *str, char *dest) } else { s += char_len; } + /* LCOV_EXCL_STOP */ } else if (1 == char_len) { switch (*s) { case '/': @@ -1154,40 +1178,52 @@ static void __ctsvc_vcard_get_clean_number_for_export(char *str, char *dest) r++; s++; break; + /* LCOV_EXCL_START */ case ',': + /* LCOV_EXCL_STOP */ case 'p': + /* LCOV_EXCL_START */ case 'P': *r = 'p'; r++; s++; break; case ';': + /* LCOV_EXCL_STOP */ case 'w': + /* LCOV_EXCL_START */ case 'W': *r = 'w'; r++; s++; break; case 'a' ... 'o': + /* LCOV_EXCL_STOP */ case 'q' ... 'v': case 'x' ... 'z': + /* LCOV_EXCL_START */ *r = *s - 0x20; r++; s++; break; case 'A' ... 'O': + /* LCOV_EXCL_STOP */ case 'Q' ... 'V': case 'X' ... 'Z': + /* LCOV_EXCL_START */ *r = *s; r++; s++; break; + /* LCOV_EXCL_STOP */ default: s++; break; } } else { + /* LCOV_EXCL_START */ s += char_len; + /* LCOV_EXCL_STOP */ } } *r = '\0'; @@ -1237,7 +1273,9 @@ static inline int __ctsvc_vcard_put_email_type(int type, char *label, char **buf char *type_str = NULL; if (CONTACTS_EMAIL_TYPE_HOME == type) { type_str = "HOME"; - } else if (CONTACTS_EMAIL_TYPE_WORK == type) { + } + /* LCOV_EXCL_START */ + else if (CONTACTS_EMAIL_TYPE_WORK == type) { type_str = "WORK"; } else if (CONTACTS_EMAIL_TYPE_MOBILE == type) { type_str = "CELL"; @@ -1248,7 +1286,7 @@ static inline int __ctsvc_vcard_put_email_type(int type, char *label, char **buf } return len; } - + /* LCOV_EXCL_STOP */ if (type_str) { CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE="); CTSVC_VCARD_APPEND_STR(buf, buf_size, len, type_str); @@ -1284,7 +1322,9 @@ static inline int __ctsvc_vcard_put_url_type(int type, char *label, char **buf, if (CONTACTS_URL_TYPE_HOME == type) { type_str = "HOME"; - } else if (CONTACTS_URL_TYPE_WORK == type) { + } + /* LCOV_EXCL_START */ + else if (CONTACTS_URL_TYPE_WORK == type) { type_str = "WORK"; } else if (CONTACTS_URL_TYPE_CUSTOM == type) { if (__ctsvc_vcard_is_valid_custom_label(label)) { @@ -1293,6 +1333,7 @@ static inline int __ctsvc_vcard_put_url_type(int type, char *label, char **buf, } return len; } + /* LCOV_EXCL_STOP */ if (type_str) { CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE="); CTSVC_VCARD_APPEND_STR(buf, buf_size, len, type_str); @@ -1449,6 +1490,7 @@ static inline int __ctsvc_vcard_put_relationship_type(int type, char *label, cha case CONTACTS_RELATIONSHIP_TYPE_ASSISTANT: type_str = "ASSISTANT"; break; + /* LCOV_EXCL_START */ case CONTACTS_RELATIONSHIP_TYPE_BROTHER: type_str = "BROTHER"; break; @@ -1493,6 +1535,7 @@ static inline int __ctsvc_vcard_put_relationship_type(int type, char *label, cha CTSVC_VCARD_APPEND_STR(buf, buf_size, len, ";TYPE=X-"); CTSVC_VCARD_APPEND_STR(buf, buf_size, len, label); } + /* LCOV_EXCL_STOP */ return len; } @@ -1589,6 +1632,7 @@ static bool _ctsvc_vcard_image_util_supported_jpeg_colorspace_cb( size = (uint64_t)size_decode; if (limit_size_of_photo < width || limit_size_of_photo < height) { /* need resize */ + /* LCOV_EXCL_START */ int resized_width; int resized_height; media_format_h fmt; @@ -1648,6 +1692,7 @@ static bool _ctsvc_vcard_image_util_supported_jpeg_colorspace_cb( width = resized_width; height = resized_height; + /* LCOV_EXCL_STOP */ } image_util_encode_h eh = NULL; @@ -1931,6 +1976,7 @@ static inline int __ctsvc_vcard_append_my_profile(ctsvc_my_profile_s *my_profile } if (my_profile->changed_time) { + /* LCOV_EXCL_START */ struct tm ts; gmtime_r((time_t *)&my_profile->changed_time, &ts); char temp[VCARD_ITEM_LENGTH] = {0}; @@ -1941,6 +1987,7 @@ static inline int __ctsvc_vcard_append_my_profile(ctsvc_my_profile_s *my_profile CTSVC_CRLF); CTSVC_VCARD_APPEND_STR(buf, buf_size, len, temp); + /* LCOV_EXCL_STOP */ } #if 0 @@ -2404,7 +2451,7 @@ static bool __ctsvc_vcard_has_unsupported_format(const char *row) char *end_of_raw = strchr(row, '\n'); bool has_xcustom = false; bool has_quoted = false; - + /* LCOV_EXCL_START */ if ((xcustom1 != NULL && xcustom1 < end_of_raw) ||(xcustom2 != NULL && xcustom2 < end_of_raw)) has_xcustom = true; @@ -2416,10 +2463,10 @@ static bool __ctsvc_vcard_has_unsupported_format(const char *row) INFO("This row has unsupported format"); return true; } - + /* LCOV_EXCL_STOP */ return false; } - + /* LCOV_EXCL_START */ static inline char* __ctsvc_vcard_pass_unsupported(char *vcard) { while (*vcard) { @@ -2430,7 +2477,7 @@ static inline char* __ctsvc_vcard_pass_unsupported(char *vcard) return NULL; } - + /* LCOV_EXCL_STOP */ static char* __ctsvc_strtok(char *val, char c) { char *before = NULL; @@ -2452,9 +2499,11 @@ static inline bool __ctsvc_vcard_check_base64_encoded(char *src) while (*tmp) { if ('B' == *tmp) { + /* LCOV_EXCL_START */ ret = strncmp(tmp, "BASE64", sizeof("BASE64") - 1); if (STRING_EQUAL == ret) return true; + /* LCOV_EXCL_STOP */ } else if (':' == *tmp || '\r' == *tmp) { break; } @@ -2471,11 +2520,13 @@ static inline int __ctsvc_vcard_check_quoted(char *src, int max, int *quoted) while (*src && max) { if ('Q' == *src) { + /* LCOV_EXCL_START */ ret = strncmp(src, "QUOTED-PRINTABLE", sizeof("QUOTED-PRINTABLE") - 1); if (STRING_EQUAL == ret) { *quoted = TRUE; return TRUE; } + /* LCOV_EXCL_STOP */ } else if (':' == *src) { break; } @@ -2507,7 +2558,7 @@ static inline int __ctsvc_vcard_remove_folding(char *folded_src) *result = '\0'; return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static inline int __ctsvc_vcard_hex_to_dec(char hex) { switch (hex) { @@ -2552,7 +2603,7 @@ static inline int __ctsvc_vcard_decode_quoted_val(char *val) *dest = '\0'; return dest - val; } - + /* LCOV_EXCL_STOP */ static inline char* __ctsvc_vcard_translate_charset(char *src, int len) { int ret; @@ -2586,7 +2637,7 @@ static inline char* __ctsvc_vcard_translate_charset(char *src, int len) enc[i] = '\0'; if (0 == strcasecmp("UTF-8", enc)) return NULL; - + /* LCOV_EXCL_START */ while (':' != *val) val++; @@ -2595,7 +2646,6 @@ static inline char* __ctsvc_vcard_translate_charset(char *src, int len) temp_size = (src_len+1) * sizeof(UChar); temp = malloc(temp_size); if (NULL == temp) { - /* LCOV_EXCL_START */ ERR("malloc() Fail"); return NULL; /* LCOV_EXCL_STOP */ @@ -2604,12 +2654,12 @@ static inline char* __ctsvc_vcard_translate_charset(char *src, int len) WARN_IF(U_FAILURE(err), "ucnv_open() Fail(%d), enc=%s", err, enc); /* LCOV_EXCL_LINE */ ucnv_toUChars(conv, temp, temp_size, val, src_len, &err); WARN_IF(U_FAILURE(err), "ucnv_toUChars() Fail(%d), enc=%s", err, enc); /* LCOV_EXCL_LINE */ + /* LCOV_EXCL_START */ ucnv_close(conv); dest_size = temp_size*2; dest = malloc(dest_size); if (NULL == dest) { - /* LCOV_EXCL_START */ ERR("malloc() Fail"); free(temp); return NULL; @@ -2667,6 +2717,7 @@ static char* __ctsvc_vcard_get_val(int ver, char *src, char **prefix, char **des quoted = FALSE; cursor = src; if (CTSVC_VCARD_VER_2_1 == ver) { + /* LCOV_EXCL_START */ while (*cursor) { if ('=' == *cursor && __ctsvc_vcard_check_quoted(src, cursor - src, "ed)) { if ('\r' == *(cursor+1) && '\n' == *(cursor+2)) @@ -2680,6 +2731,7 @@ static char* __ctsvc_vcard_get_val(int ver, char *src, char **prefix, char **des cursor++; } + /* LCOV_EXCL_STOP */ } else { while (*cursor) { if ('\r' == *cursor && '\n' == *(cursor+1) && ' ' != *(cursor+2)) @@ -2693,8 +2745,10 @@ static char* __ctsvc_vcard_get_val(int ver, char *src, char **prefix, char **des } if (src == cursor) { + /* LCOV_EXCL_START */ *dest = NULL; return NULL; + /* LCOV_EXCL_STOP */ } else { int len = 0; char temp = *cursor; @@ -2720,8 +2774,10 @@ static char* __ctsvc_vcard_get_val(int ver, char *src, char **prefix, char **des len = strlen(*dest); new_dest = __ctsvc_vcard_translate_charset(*dest, len); if (new_dest) { + /* LCOV_EXCL_START */ free(*dest); *dest = new_dest; + /* LCOV_EXCL_STOP */ } *cursor = temp; return (cursor + 1); @@ -2752,11 +2808,13 @@ static char* __ctsvc_vcard_remove_escape_char(char *str) if (*s == '\\' && *(s+1)) { char *n = (char*)(s+1); switch (*n) { + /* LCOV_EXCL_START */ case 'n': case 'N': *r = '\n'; s++; break; + /* LCOV_EXCL_STOP */ case ';': case ':': case ',': @@ -2766,6 +2824,7 @@ static char* __ctsvc_vcard_remove_escape_char(char *str) *r = *n; s++; break; + /* LCOV_EXCL_START */ case 0xA1: /* en/em backslash */ if (*(n+1) && 0xAC == *(n+1)) { *r = *n; @@ -2785,6 +2844,7 @@ static char* __ctsvc_vcard_remove_escape_char(char *str) default: *r = *s; break; + /* LCOV_EXCL_STOP */ } r++; s++; @@ -2825,8 +2885,9 @@ static inline int __ctsvc_vcard_get_display_name(ctsvc_list_s *name_list, char * WARN_IF(ret != CONTACTS_ERROR_NONE, "contacts_record_get_str_p is Fail(%d)", ret); if ((NULL == first_name || '\0' == *first_name) && (NULL == last_name || '\0' == *last_name)) + /* LCOV_EXCL_START */ contacts_record_set_str(name, _contacts_name.first, __ctsvc_vcard_remove_escape_char(temp)); - + /* LCOV_EXCL_STOP */ return CONTACTS_ERROR_NONE; } @@ -2870,7 +2931,9 @@ static inline int __ctsvc_vcard_get_name(ctsvc_list_s *name_list, char *val) RETVM_IF(ret < CONTACTS_ERROR_NONE, ret, "contacts_record_create() Fail(%d)", ret); contacts_list_add((contacts_list_h)name_list, name); } else { + /* LCOV_EXCL_START */ contacts_list_get_current_record_p((contacts_list_h)name_list, &name); + /* LCOV_EXCL_STOP */ } contacts_record_set_str(name, _contacts_name.first, NULL); /* remove FN */ @@ -3036,7 +3099,7 @@ static inline int __ctsvc_vcard_get_photo(contacts_record_h contact, ctsvc_list_ return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static inline void __ctsvc_vcard_get_event_type(contacts_record_h event, char *val) { int type = CONTACTS_EVENT_TYPE_OTHER; @@ -3047,10 +3110,8 @@ static inline void __ctsvc_vcard_get_event_type(contacts_record_h event, char *v while (temp) { lower = strdup(temp); if (NULL == lower) { - /* LCOV_EXCL_START */ ERR("strdup() Fail"); break; - /* LCOV_EXCL_STOP */ } lower_temp = lower; while (*lower_temp) { @@ -3069,7 +3130,7 @@ static inline void __ctsvc_vcard_get_event_type(contacts_record_h event, char *v } contacts_record_set_int(event, _contacts_event.type, type); } - + /* LCOV_EXCL_STOP */ static inline int __ctsvc_vcard_get_event(ctsvc_list_s *event_list, int type, char *prefix, char *val) { @@ -3115,10 +3176,12 @@ static inline int __ctsvc_vcard_get_event(ctsvc_list_s *event_list, int type, ch if (CTSVC_VCARD_VALUE_BDAY == type) contacts_record_set_int(event, _contacts_event.type, CONTACTS_EVENT_TYPE_BIRTH); + /* LCOV_EXCL_START */ else if (CTSVC_VCARD_VALUE_X_ANNIVERSARY == type) contacts_record_set_int(event, _contacts_event.type, CONTACTS_EVENT_TYPE_ANNIVERSARY); else if (CTSVC_VCARD_VALUE_X_TIZEN_EVENT == type) __ctsvc_vcard_get_event_type(event, prefix); + /* LCOV_EXCL_STOP */ contacts_list_add((contacts_list_h)event_list, event); return CONTACTS_ERROR_NONE; @@ -3146,12 +3209,16 @@ static inline void __ctsvc_vcard_get_company_type(contacts_record_h company, cha result = strstr(lower, "work"); if (result) + /* LCOV_EXCL_START */ type = CONTACTS_COMPANY_TYPE_WORK; + /* LCOV_EXCL_STOP */ result = strstr(lower, "x-"); if (result) { + /* LCOV_EXCL_START */ type = CONTACTS_COMPANY_TYPE_CUSTOM; contacts_record_set_str(company, _contacts_company.label, temp+(result-lower)+2); + /* LCOV_EXCL_STOP */ } free(lower); @@ -3254,8 +3321,10 @@ static inline int __ctsvc_vcard_get_company_logo(ctsvc_list_s *company_list, cha buf = g_base64_decode(temp+1, &size); if ((0 == size) || (NULL == buf)) { + /* LCOV_EXCL_START */ g_free(buf); return CONTACTS_ERROR_NONE; + /* LCOV_EXCL_STOP */ } gettimeofday(&tv, NULL); @@ -3491,6 +3560,7 @@ static inline bool __ctsvc_vcard_get_number_type(contacts_record_h number, char if (result) type |= CONTACTS_NUMBER_TYPE_PCS; result = strstr(lower, "x-"); if (result) { + /* LCOV_EXCL_START */ if (strstr(lower, "x-assistant")) { type |= CONTACTS_NUMBER_TYPE_ASSISTANT; } else if (strstr(lower, "x-radio")) { @@ -3503,6 +3573,7 @@ static inline bool __ctsvc_vcard_get_number_type(contacts_record_h number, char type = CONTACTS_NUMBER_TYPE_CUSTOM; contacts_record_set_str(number, _contacts_number.label, temp+(result-lower)+2); } + /* LCOV_EXCL_STOP */ } free(lower); @@ -3520,6 +3591,7 @@ static char* __ctsvc_vcard_get_clean_number_for_import(char *str) char *r = s; while (*s) { char_len = __ctsvc_vcard_check_utf8(*s); + /* LCOV_EXCL_START */ if (3 == char_len) { if (*s == 0xef) { if (*(s+1) == 0xbc) { @@ -3566,7 +3638,9 @@ static char* __ctsvc_vcard_get_clean_number_for_import(char *str) } else { s += char_len; } - } else if (1 == char_len) { + } + /* LCOV_EXCL_STOP */ + else if (1 == char_len) { switch (*s) { case '/': case '.': @@ -3582,6 +3656,7 @@ static char* __ctsvc_vcard_get_clean_number_for_import(char *str) r++; s++; break; + /* LCOV_EXCL_START */ case 'p': case 'P': *r = ','; @@ -3595,15 +3670,19 @@ static char* __ctsvc_vcard_get_clean_number_for_import(char *str) s++; break; case 'a' ... 'o': + /* LCOV_EXCL_STOP */ case 'q' ... 'v': case 'x' ... 'z': + /* LCOV_EXCL_START */ *r = *s - 0x20; r++; s++; break; case 'A' ... 'O': + /* LCOV_EXCL_STOP */ case 'Q' ... 'V': case 'X' ... 'Z': + /* LCOV_EXCL_START */ *r = *s; r++; s++; @@ -3611,9 +3690,12 @@ static char* __ctsvc_vcard_get_clean_number_for_import(char *str) default: s++; break; + /* LCOV_EXCL_STOP */ } } else { + /* LCOV_EXCL_START */ s += char_len; + /* LCOV_EXCL_STOP */ } } *r = '\0'; @@ -3674,8 +3756,10 @@ static inline bool __ctsvc_vcard_get_email_type(contacts_record_h email, char *v } else if (strstr(lower, "cell")) { type = CONTACTS_EMAIL_TYPE_MOBILE; } else if ((result = strstr(lower, "x-"))) { + /* LCOV_EXCL_START */ type = CONTACTS_EMAIL_TYPE_CUSTOM; contacts_record_set_str(email, _contacts_email.label, temp+(result-lower)+2); + /* LCOV_EXCL_STOP */ } free(lower); @@ -3742,8 +3826,10 @@ static inline bool __ctsvc_vcard_get_postal_type(contacts_record_h address, char if (result) type |= CONTACTS_ADDRESS_TYPE_WORK; result = strstr(lower, "x-"); if (result) { + /* LCOV_EXCL_START */ type = CONTACTS_ADDRESS_TYPE_CUSTOM; contacts_record_set_str(address, _contacts_address.label, temp+(result-lower)+2); + /* LCOV_EXCL_STOP */ } result = strstr(val, "pref"); if (result) pref = true; @@ -3854,6 +3940,7 @@ static inline int __ctsvc_vcard_get_messenger(ctsvc_list_s *messenger_list, int contacts_record_set_str(messenger, _contacts_messenger.im_id, __ctsvc_vcard_remove_escape_char(temp)); switch (type) { + /* LCOV_EXCL_START */ case CTSVC_VCARD_VALUE_X_MSN: contacts_record_set_int(messenger, _contacts_messenger.type, CONTACTS_MESSENGER_TYPE_WLM); break; @@ -3879,6 +3966,7 @@ static inline int __ctsvc_vcard_get_messenger(ctsvc_list_s *messenger_list, int case CTSVC_VCARD_VALUE_X_GOOGLE_TALK: contacts_record_set_int(messenger, _contacts_messenger.type, CONTACTS_MESSENGER_TYPE_GOOGLE); break; + /* LCOV_EXCL_STOP */ case CTSVC_VCARD_VALUE_X_TIZEN_MESSENGER: __ctsvc_vcard_get_messenger_type(messenger, prefix); break; @@ -3967,7 +4055,7 @@ static inline int __ctsvc_vcard_get_relationship(ctsvc_list_s *relationship_list return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static char* __ctsvc_vcard_decode_base64_val(char *val) { gsize size = 0; @@ -3985,11 +4073,9 @@ static char* __ctsvc_vcard_decode_base64_val(char *val) dest = calloc((src-val)+size+1, sizeof(char)); if (NULL == dest) { - /* LCOV_EXCL_START */ g_free(decoded_str); ERR("calloc() Fail"); return NULL; - /* LCOV_EXCL_STOP */ } snprintf(dest, (src-val)+1, "%s", val); @@ -3998,7 +4084,7 @@ static char* __ctsvc_vcard_decode_base64_val(char *val) return dest; } - + /* LCOV_EXCL_STOP */ static inline int __ctsvc_vcard_get_contact(int ver, char *vcard, contacts_record_h *record) { int type; @@ -4013,10 +4099,12 @@ static inline int __ctsvc_vcard_get_contact(int ver, char *vcard, contacts_recor bool base64_encoded = false; type = __ctsvc_vcard_check_content_type(&cursor); if (CTSVC_VCARD_VALUE_NONE == type || __ctsvc_vcard_has_unsupported_format(cursor)) { + /* LCOV_EXCL_START */ new_start = __ctsvc_vcard_pass_unsupported(cursor); if (new_start) { cursor = new_start; continue; + /* LCOV_EXCL_STOP */ } else { break; } @@ -4026,6 +4114,7 @@ static inline int __ctsvc_vcard_get_contact(int ver, char *vcard, contacts_recor base64_encoded = __ctsvc_vcard_check_base64_encoded(cursor); new_start = __ctsvc_vcard_get_val(ver, cursor, &prefix, &val); + /* LCOV_EXCL_START */ if (NULL == new_start) { if (prefix) free(prefix); @@ -4040,21 +4129,18 @@ static inline int __ctsvc_vcard_get_contact(int ver, char *vcard, contacts_recor free(prefix); continue; } - if (base64_encoded) { char *temp = __ctsvc_vcard_decode_base64_val(val); if (NULL == temp) { - /* LCOV_EXCL_START */ ERR("__ctsvc_vcard_decode_base64_val() Fail"); free(prefix); free(val); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } free(val); val = temp; } - + /* LCOV_EXCL_STOP */ switch (type) { case CTSVC_VCARD_VALUE_FN: __ctsvc_vcard_get_display_name(contact->name, val); @@ -4172,9 +4258,11 @@ static inline int __ctsvc_vcard_check_version(const char *src) case '\n': case '\r': return CTSVC_VCARD_VER_2_1; + /* LCOV_EXCL_START */ case ' ': src++; break; + /* LCOV_EXCL_STOP */ default: start = true; break; @@ -4262,6 +4350,7 @@ static inline void __ctsvc_vcard_make_contact_display_name(ctsvc_contact_s *cont if (reverse_lang_type == CTSVC_LANG_JAPANESE) { /* make temp_display name Prefix - Last - Middle - First - Suffix */ + /* LCOV_EXCL_START */ if (name->addition) { if (*temp_display) len += snprintf(temp_display + len, temp_display_len - len, " "); @@ -4273,6 +4362,7 @@ static inline void __ctsvc_vcard_make_contact_display_name(ctsvc_contact_s *cont len += snprintf(temp_display + len, temp_display_len - len, " "); len += snprintf(temp_display + len, temp_display_len - len, "%s", name->first); } + /* LCOV_EXCL_STOP */ } else { /* make temp_display name Prefix - Last - First -Middle - Suffix */ if (name->first) { @@ -4431,6 +4521,7 @@ static inline void __ctsvc_vcard_make_contact_display_name(ctsvc_contact_s *cont } contact->display_source_type = CONTACTS_DISPLAY_NAME_SOURCE_TYPE_NAME; } else { + /* LCOV_EXCL_START */ GList *cur; bool set_display_name = false; if (contact->company && contact->company->records) { @@ -4483,6 +4574,7 @@ static inline void __ctsvc_vcard_make_contact_display_name(ctsvc_contact_s *cont } } } + /* LCOV_EXCL_STOP */ } return; } @@ -4563,10 +4655,12 @@ static void __contacts_vcard_free_sub_vcard_info_list(GList *list) return; GList *cursor; + /* LCOV_EXCL_START */ for (cursor = list; cursor; cursor = cursor->next) { sub_vcard_info_s *vcard_info = cursor->data; free(vcard_info); } + /* LCOV_EXCL_STOP */ g_list_free(list); } @@ -4619,12 +4713,14 @@ static const char* __contacts_vcard_parse_get_vcard_object(const char *cursor, G vcard_cursor = __contacts_vcard_remove_line_break(vcard_cursor); pos_start = vcard_start_cursor; + /* LCOV_EXCL_START */ for (sub_vcard_cursor = sub_vcard_list; sub_vcard_cursor; sub_vcard_cursor = sub_vcard_cursor->next) { sub_vcard_info_s *sub_vcard_info = sub_vcard_cursor->data; const char *pos_end = sub_vcard_info->pos_start; vcard_len += (pos_end - pos_start); pos_start = sub_vcard_info->pos_end; } + /* LCOV_EXCL_STOP */ vcard_len += (vcard_cursor - pos_start); vcard_object = calloc(vcard_len + 1, sizeof(char)); if (NULL == vcard_object) { @@ -4637,6 +4733,7 @@ static const char* __contacts_vcard_parse_get_vcard_object(const char *cursor, G vcard_len = 0; pos_start = vcard_start_cursor; + /* LCOV_EXCL_START */ for (sub_vcard_cursor = sub_vcard_list; sub_vcard_cursor; sub_vcard_cursor = sub_vcard_cursor->next) { sub_vcard_info_s *sub_vcard_info = sub_vcard_cursor->data; const char *pos_end = sub_vcard_info->pos_start; @@ -4644,16 +4741,17 @@ static const char* __contacts_vcard_parse_get_vcard_object(const char *cursor, G vcard_len += (pos_end - pos_start); pos_start = sub_vcard_info->pos_end; } + /* LCOV_EXCL_STOP */ __contacts_vcard_free_sub_vcard_info_list(sub_vcard_list); memcpy(vcard_object+vcard_len, pos_start, vcard_cursor - pos_start); *plist_vcard_object = g_list_append(*plist_vcard_object, vcard_object); return vcard_cursor; } else if (STRING_EQUAL == strncmp(vcard_cursor, begin, strlen(begin))) { + /* LCOV_EXCL_START */ /* sub vcard */ sub_vcard_info_s *sub_vcard_info = calloc(1, sizeof(sub_vcard_info_s)); if (NULL == sub_vcard_info) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); __contacts_vcard_free_sub_vcard_info_list(sub_vcard_list); return NULL; @@ -4666,6 +4764,7 @@ static const char* __contacts_vcard_parse_get_vcard_object(const char *cursor, G sub_vcard_list = g_list_append(sub_vcard_list, sub_vcard_info); continue; + /* LCOV_EXCL_STOP */ } new_line = false; } @@ -4675,10 +4774,11 @@ static const char* __contacts_vcard_parse_get_vcard_object(const char *cursor, G vcard_cursor = __contacts_vcard_remove_line_break(vcard_cursor); } } - + /* LCOV_EXCL_START */ __contacts_vcard_free_sub_vcard_info_list(sub_vcard_list); return vcard_cursor; + /* LCOV_EXCL_STOP */ } EXPORT_API int contacts_vcard_parse_to_contacts(const char *vcard_stream, contacts_list_h *out_contacts) @@ -4775,8 +4875,8 @@ EXPORT_API int contacts_vcard_parse_to_contact_foreach(const char *vcard_file_na while (fgets(line, sizeof(line), file)) { if (0 == len) if (STRING_EQUAL != strncmp(line, "BEGIN:VCARD", strlen("BEGIN:VCARD"))) + /* LCOV_EXCL_START */ continue; - if (buf_size - len <= strlen(line)) { char *new_stream; buf_size += sizeof(line) * 2; @@ -4784,7 +4884,6 @@ EXPORT_API int contacts_vcard_parse_to_contact_foreach(const char *vcard_file_na if (new_stream) { stream = new_stream; } else { - /* LCOV_EXCL_START */ free(stream); fclose(file); return CONTACTS_ERROR_OUT_OF_MEMORY; diff --git a/server/ctsvc_localize_ch.c b/server/ctsvc_localize_ch.c index a97ab78..3985bf4 100644 --- a/server/ctsvc_localize_ch.c +++ b/server/ctsvc_localize_ch.c @@ -2125,7 +2125,7 @@ static const char* const pinyin_spell_table[] = { "xie", "yu"}; #define DUOYINZI_SEPERATOR '|' - + /* LCOV_EXCL_START */ static inline int __ctsvc_get_pinyinspell(UChar src, char spell[CHINESE_DUOYINZI_MAX_COUNT][CHINESE_PINYIN_SPELL_MAX_LEN]) { int offset, len, i, j; @@ -2147,12 +2147,13 @@ static inline int __ctsvc_get_pinyinspell(UChar src, char spell[CHINESE_DUOYINZI return count; } - + /* LCOV_EXCL_STOP */ static inline bool __ctsvc_is_chinese(const UChar *src) { if (CHINESE_UNICODE_START <= *src && *src <= CHINESE_UNICODE_END) + /* LCOV_EXCL_START */ return true; - + /* LCOV_EXCL_STOP */ return false; } @@ -2198,6 +2199,7 @@ bool ctsvc_has_chinese(const char *src) * the size can be too many. So, we limit to support total 3 multiple pinyin spell. * If exceeded 3 length, then we will pick first one. */ + /* LCOV_EXCL_START */ int ctsvc_convert_chinese_to_pinyin(const char *src, pinyin_name_s **name, int *size) { UChar temp_result[strlen(src)+1]; @@ -2214,10 +2216,8 @@ int ctsvc_convert_chinese_to_pinyin(const char *src, pinyin_name_s **name, int * u_strFromUTF8(temp_result, array_sizeof(temp_result), NULL, src, -1, &status); if (U_FAILURE(status)) { - /* LCOV_EXCL_START */ ERR("u_strFromUTF8 Fail(%s)", u_errorName(status)); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } len = u_strlen(temp_result); @@ -2315,4 +2315,4 @@ void ctsvc_pinyin_free(pinyin_name_s *pinyinname, int size) } free(pinyinname); } - + /* LCOV_EXCL_STOP */ \ No newline at end of file diff --git a/server/ctsvc_localize_jp.c b/server/ctsvc_localize_jp.c index 524deaa..9eb9cc7 100644 --- a/server/ctsvc_localize_jp.c +++ b/server/ctsvc_localize_jp.c @@ -77,7 +77,7 @@ static inline bool is_japanese(UChar src) return false; } } - + /* LCOV_EXCL_START */ int ctsvc_convert_japanese_to_hiragana_unicode(UChar *src, UChar *dest, int dest_size) { int i, j = 0, len = 0; @@ -143,74 +143,60 @@ int ctsvc_convert_japanese_to_hiragana(const char *src, char **dest) u_strFromUTF8(NULL, 0, &size, src, strlen(src), &status); if (U_FAILURE(status) && status != U_BUFFER_OVERFLOW_ERROR) { - /* LCOV_EXCL_START */ ERR("u_strFromUTF8 to get the dest length Fail(%s)", u_errorName(status)); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } status = U_ZERO_ERROR; tmp_result = calloc(1, sizeof(UChar) * (size + 1)); if (NULL == tmp_result) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } u_strFromUTF8(tmp_result, size + 1, NULL, src, -1, &status); if (U_FAILURE(status)) { - /* LCOV_EXCL_START */ ERR("u_strFromUTF8 Fail(%s)", u_errorName(status)); free(tmp_result); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } result = calloc(1, sizeof(UChar) * (size + 1)); if (NULL == result) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); free(tmp_result); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } ctsvc_convert_japanese_to_hiragana_unicode(tmp_result, result, size + 1); u_strToUTF8(NULL, 0, &size, result, -1, &status); if (U_FAILURE(status) && status != U_BUFFER_OVERFLOW_ERROR) { - /* LCOV_EXCL_START */ ERR("u_strToUTF8 to get the dest length Fail(%s)", u_errorName(status)); free(tmp_result); free(result); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } status = U_ZERO_ERROR; *dest = calloc(1, sizeof(char)*(size+1)); if (NULL == *dest) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); free(tmp_result); free(result); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } u_strToUTF8(*dest, size + 1, &size, result, -1, &status); if (U_FAILURE(status)) { - /* LCOV_EXCL_START */ ERR("u_strToUTF8 Fail(%s)", u_errorName(status)); free(tmp_result); free(result); free(*dest); *dest = NULL; return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } free(tmp_result); free(result); return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_STOP */ \ No newline at end of file diff --git a/server/ctsvc_localize_kor.c b/server/ctsvc_localize_kor.c index e089a65..2971b90 100644 --- a/server/ctsvc_localize_kor.c +++ b/server/ctsvc_localize_kor.c @@ -90,7 +90,7 @@ static const char hangul_compatibility_jongseong[] = { static const char hangul_jamo_jongseong[] = { 0xAA, 0xAC, 0xAD, 0xB0, 0xB1, 0xB2, 0xB3, 0xB4, 0xB5, /* to jongseong 0x11A8~0x11FF */ 0xC7, 0xC8, 0xCC, 0xCE, 0xD3, 0xD7, 0xD9, 0xDD, 0xDF, 0xF1, 0xF2, 0x00}; - + /* LCOV_EXCL_START */ static inline bool is_chosung(UChar src) { int unicode_value1 = 0; @@ -113,7 +113,7 @@ static inline bool is_chosung(UChar src) return false; } - + /* LCOV_EXCL_STOP */ bool ctsvc_is_hangul(UChar src) { if ((0x1100 == (src & 0xFF00)) /* korean -Hangul Jamo*/ diff --git a/server/ctsvc_normalize.c b/server/ctsvc_normalize.c index d8b2e68..b8d4117 100644 --- a/server/ctsvc_normalize.c +++ b/server/ctsvc_normalize.c @@ -189,19 +189,17 @@ static int __ctsvc_normalize_str(const char *src, char **dest, bool is_index) u_strFromUTF8(tmp_result, tmp_size + 1, &tmp_size, src, -1, &status); if (status == U_BUFFER_OVERFLOW_ERROR) { + /* LCOV_EXCL_START */ status = U_ZERO_ERROR; free(tmp_result); tmp_result = calloc(1, sizeof(UChar) * (tmp_size + 1)); if (NULL == tmp_result) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } u_strFromUTF8(tmp_result, tmp_size + 1, NULL, src, -1, &status); if (U_FAILURE(status)) { - /* LCOV_EXCL_START */ ERR("u_strFromUTF8()Fail(%s)", u_errorName(status)); free(tmp_result); return CONTACTS_ERROR_SYSTEM; @@ -226,20 +224,18 @@ static int __ctsvc_normalize_str(const char *src, char **dest, bool is_index) upper_size = u_strToUpper(tmp_upper, tmp_size+1, tmp_result, -1, NULL, &status); if (status == U_BUFFER_OVERFLOW_ERROR) { + /* LCOV_EXCL_START */ status = U_ZERO_ERROR; free(tmp_upper); tmp_upper = calloc(1, sizeof(UChar) * (upper_size + 1)); if (NULL == tmp_upper) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); free(tmp_result); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } u_strFromUTF8(tmp_upper, upper_size + 1, NULL, src, -1, &status); if (U_FAILURE(status)) { - /* LCOV_EXCL_START */ ERR("u_strFromUTF8()Fail(%s)", u_errorName(status)); free(tmp_result); free(tmp_upper); @@ -267,21 +263,19 @@ static int __ctsvc_normalize_str(const char *src, char **dest, bool is_index) size = unorm_normalize(tmp_upper, -1, UNORM_NFD, 0, result, size+1, &status); if (status == U_BUFFER_OVERFLOW_ERROR) { + /* LCOV_EXCL_START */ status = U_ZERO_ERROR; free(result); result = calloc(1, sizeof(UChar)*(size + 1)); if (NULL == result) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); free(tmp_result); free(tmp_upper); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } unorm_normalize(tmp_upper, -1, UNORM_NFD, 0, result, size+1, &status); if (U_FAILURE(status)) { - /* LCOV_EXCL_START */ ERR("unorm_normalize() Fail(%s)", u_errorName(status)); free(tmp_result); free(tmp_upper); @@ -330,9 +324,11 @@ static int __ctsvc_normalize_str(const char *src, char **dest, bool is_index) } if (replaced) { + /* LCOV_EXCL_START */ temp_result[j] = 0x0; free(result); result = temp_result; + /* LCOV_EXCL_STOP */ } else { free(temp_result); } @@ -388,13 +384,16 @@ static int __ctsvc_convert_halfwidth_ascii_and_symbol(const char *src, UChar *de for (i = 0; i < size; i++) { /* FF00 ~ FF60: Fullwidth ASCII variants */ if (CTSVC_COMPARE_BETWEEN((UChar)0xFF00, dest[i], (UChar)0xFF60)) { + /* LCOV_EXCL_START */ int unicode_value1 = 0; int unicode_value2 = 0; unicode_value1 = 0x0; unicode_value2 = (0xFF & dest[i]) + 0x20; dest[i] = unicode_value1 << 8 | unicode_value2; + /* LCOV_EXCL_STOP */ } else if (CTSVC_COMPARE_BETWEEN((UChar)0xFFE0, dest[i], (UChar)0xFFE6)) { /* FFE0~FFE6: Fullwidth symbol variants */ + /* LCOV_EXCL_START */ if (dest[i] == (UChar)0xFFE0) dest[i] = (UChar)0x00A2; else if (dest[i] == (UChar)0xFFE1) @@ -409,6 +408,7 @@ static int __ctsvc_convert_halfwidth_ascii_and_symbol(const char *src, UChar *de dest[i] = (UChar)0x00A5; else if (dest[i] == (UChar)0xFFE6) dest[i] = (UChar)0x20A9; + /* LCOV_EXCL_STOP */ /* else */ } /* else */ @@ -467,7 +467,7 @@ int ctsvc_normalize_str(const char *src, char **dest) ret = __ctsvc_normalize_str(temp, dest, false); return ret; } - + /* LCOV_EXCL_START */ static void __ctsvc_convert_japanese_group_letter(char *dest) { int i, size, dest_len; @@ -492,7 +492,7 @@ static void __ctsvc_convert_japanese_group_letter(char *dest) RETM_IF(U_FAILURE(status), "u_strToUTF8() Fail(%s)", u_errorName(status)); } - + /* LCOV_EXCL_STOP */ static bool __ctsvc_check_range_out_index(const char src[]) { if (src[0] == 0xe2 && src[1] == 0x80 && src[2] == 0xa6) @@ -529,7 +529,9 @@ int ctsvc_normalize_index(const char *src, char **dest) } if (ret == CTSVC_LANG_JAPANESE) + /* LCOV_EXCL_START */ __ctsvc_convert_japanese_group_letter(*dest); + /* LCOV_EXCL_STOP */ return ret; } diff --git a/server/ctsvc_number_utils.c b/server/ctsvc_number_utils.c index f0ae609..b4e669d 100644 --- a/server/ctsvc_number_utils.c +++ b/server/ctsvc_number_utils.c @@ -317,7 +317,6 @@ char* ctsvc_get_network_cc(bool reload) /* LCOV_EXCL_START */ ERR("tel_get_property_string Fail(%d)", ret); return NULL; - /* LCOV_EXCL_STOP */ } if (temp) { @@ -340,7 +339,7 @@ static void _numutil_network_cc_changed(TapiHandle *handle, const char *noti_id, { ctsvc_get_network_cc(true); } - + /* LCOV_EXCL_STOP */ void* ctsvc_init_tapi_handle_for_cc() { if (handle_for_cc) @@ -359,7 +358,7 @@ void* ctsvc_init_tapi_handle_for_cc() return handle_for_cc; } - + /* LCOV_EXCL_START */ void ctsvc_deinit_tapi_handle_for_cc() { if (handle_for_cc) { @@ -369,7 +368,6 @@ void ctsvc_deinit_tapi_handle_for_cc() } handle_for_cc = NULL; } - static inline int _numutil_has_country_code(const char *src, int len) { int ret = 0; @@ -398,9 +396,7 @@ static inline int _numutil_has_country_code(const char *src, int len) ret += 1; break; default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); - /* LCOV_EXCL_STOP */ } break; case 3: @@ -420,9 +416,7 @@ static inline int _numutil_has_country_code(const char *src, int len) ret += 1; break; default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); - /* LCOV_EXCL_STOP */ } break; case 4: @@ -442,9 +436,7 @@ static inline int _numutil_has_country_code(const char *src, int len) ret += 1; break; default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); - /* LCOV_EXCL_STOP */ } break; case 5: @@ -464,9 +456,7 @@ static inline int _numutil_has_country_code(const char *src, int len) ret += 1; break; default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); - /* LCOV_EXCL_STOP */ } break; case 6: @@ -486,9 +476,7 @@ static inline int _numutil_has_country_code(const char *src, int len) ret += 1; break; default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); - /* LCOV_EXCL_STOP */ } break; case 8: @@ -508,9 +496,7 @@ static inline int _numutil_has_country_code(const char *src, int len) ret += 1; break; default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); - /* LCOV_EXCL_STOP */ } break; case 9: @@ -530,17 +516,13 @@ static inline int _numutil_has_country_code(const char *src, int len) ret += 1; break; default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); - /* LCOV_EXCL_STOP */ } break; case 0: default: - /* LCOV_EXCL_START */ ERR("The parameter(src:%s) has invalid character set", src); return 0; - /* LCOV_EXCL_STOP */ } return ret; @@ -773,7 +755,7 @@ static int _numutil_has_ip_and_cc(const char*number, int len, int *index) } return ret; } - + /* LCOV_EXCL_STOP */ int ctsvc_normalize_number(const char *src, char *dest, int dest_size, bool replace_alphabet) { @@ -792,19 +774,25 @@ int ctsvc_normalize_number(const char *src, char *dest, int dest_size, d_pos = 0; s_pos = 0; if (src[s_pos] == '+') + /* LCOV_EXCL_START */ temp[d_pos++] = src[s_pos++]; + /* LCOV_EXCL_STOP */ while (src[s_pos] != 0) { if (src[s_pos] == '+' || src[s_pos] == ';') + /* LCOV_EXCL_START */ s_pos++; + /* LCOV_EXCL_STOP */ temp[d_pos++] = src[s_pos++]; } temp[d_pos] = 0; ret = phone_number_get_normalized_number(temp, &normalized_out); if (PHONE_NUMBER_ERROR_NONE == ret) { + /* LCOV_EXCL_START */ d_pos = strlen(normalized_out); memcpy(dest, normalized_out, d_pos+1); free(normalized_out); + /* LCOV_EXCL_STOP */ } else { memcpy(dest, temp, d_pos+1); } @@ -840,6 +828,7 @@ int ctsvc_clean_number(const char *src, char *dest, int dest_size, bool replace_ if (char_len == 3) { /* fullwidth -> halfwidth */ + /* LCOV_EXCL_START */ if (src[s_pos] != 0xef) { s_pos += char_len; continue; @@ -864,9 +853,12 @@ int ctsvc_clean_number(const char *src, char *dest, int dest_size, bool replace_ if (0x81 <= src[s_pos+2] && src[s_pos+2] <= 0x9a) temp[pos++] = src[s_pos+2] - 0x40; /* ef bd 81 : 'a' ~ ef bd 9a : 'z' */ } + /* LCOV_EXCL_STOP */ } else if (char_len == 1) { if (0x41 <= src[s_pos] && src[s_pos] <= 0x5a) /* 'A' ~ 'Z' */ + /* LCOV_EXCL_START */ temp[pos++] = src[s_pos]; + /* LCOV_EXCL_STOP */ else if (0x61 <= src[s_pos] && src[s_pos] <= 0x7a) /* 'a' ~ 'z' */ temp[pos++] = src[s_pos] - 0x20; else @@ -883,7 +875,9 @@ int ctsvc_clean_number(const char *src, char *dest, int dest_size, bool replace_ dest[d_pos++] = temp[pos]; else if (temp[pos] == '+' || temp[pos] == '#' || temp[pos] == '*' || temp[pos] == ';' || temp[pos] == ',') + /* LCOV_EXCL_START */ dest[d_pos++] = temp[pos]; + /* LCOV_EXCL_STOP */ pos++; } dest[d_pos] = 0; @@ -905,12 +899,16 @@ static int _numutil_minmatch_number(const char *src, char *dest, int dest_size, RETV_IF(dest_size <= strlen(src), CONTACTS_ERROR_INVALID_PARAMETER); if ('+' == src[0]) { + /* LCOV_EXCL_START */ len = _numutil_has_country_code(&src[1], strlen(src)-1); temp_number = src + len +1; + /* LCOV_EXCL_STOP */ } else if ('0' == src[0]) { temp_number = src+1; } else if (cc && cc[0] == '7' && src[0] == '8') { + /* LCOV_EXCL_START */ temp_number = src+1; + /* LCOV_EXCL_STOP */ } else { temp_number = src; } @@ -940,7 +938,9 @@ static int _numutil_minmatch_number(const char *src, char *dest, int dest_size, dest[len-i-1] = c; } } else { + /* LCOV_EXCL_START */ memcpy(dest, src, strlen(src)+1); + /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; @@ -996,7 +996,7 @@ static bool _numutil_is_phonenumber_fullwidth(const char *keyword) char_len = ctsvc_check_utf8(keyword[i]); if (char_len != UTF8_FULLWIDTH_LENGTH || str_len-i < UTF8_FULLWIDTH_LENGTH) return false; - + /* LCOV_EXCL_START */ if (keyword[i] != 0xef) return false; @@ -1007,6 +1007,7 @@ static bool _numutil_is_phonenumber_fullwidth(const char *keyword) continue; else if (0x8b == keyword[i+2]) /* ef bc 8b : '+' */ continue; + /* LCOV_EXCL_STOP */ else return false; } @@ -1019,6 +1020,7 @@ bool ctsvc_is_phonenumber(const char *src) } /* numbers are cleaned number or normalized number */ + /* LCOV_EXCL_START */ static bool _numutil_number_compare(const char *number1, const char *number2) { int len1; @@ -1112,7 +1114,6 @@ static bool _numutil_number_compare(const char *number1, const char *number2) return false; } - /* When querying _NUMBER_COMPARE_, this function will be called. */ void ctsvc_db_phone_number_equal_callback(sqlite3_context *context, int argc, sqlite3_value **argv) @@ -1122,11 +1123,9 @@ void ctsvc_db_phone_number_equal_callback(sqlite3_context *context, char *number2; if (argc < 4) { - /* LCOV_EXCL_START */ sqlite3_result_int(context, 0); ERR("argc invalid"); return; - /* LCOV_EXCL_STOP */ } number1 = (char*)sqlite3_value_text(argv[0]); @@ -1136,4 +1135,4 @@ void ctsvc_db_phone_number_equal_callback(sqlite3_context *context, return; #endif } - + /* LCOV_EXCL_STOP */ diff --git a/server/ctsvc_schema_recovery.c b/server/ctsvc_schema_recovery.c index 697aad7..29b1e72 100644 --- a/server/ctsvc_schema_recovery.c +++ b/server/ctsvc_schema_recovery.c @@ -38,7 +38,7 @@ static inline bool __ctsvc_server_db_file_exist(void) close(fd); return true; } - + /* LCOV_EXCL_START */ static inline int __ctsvc_server_remake_db_file() { int ret; @@ -50,17 +50,15 @@ static inline int __ctsvc_server_remake_db_file() ret = sqlite3_exec(db, schema_query, NULL, 0, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("remake contacts DB file is Fail : %s", errmsg); sqlite3_free(errmsg); - /* LCOV_EXCL_STOP */ } ctsvc_server_db_close(); return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_STOP */ static int __integrity_callback(void *user_data, int ncol, char **cols, char **name) { CTS_FN_CALL; @@ -69,8 +67,10 @@ static int __integrity_callback(void *user_data, int ncol, char **cols, char **n RETV_IF(NULL == cols, CONTACTS_ERROR_DB); if (0 != strcmp(cols[0], "ok")) { + /* LCOV_EXCL_START */ ERR("integrity_check Fail(%s)", cols[0]); return CONTACTS_ERROR_DB; + /* LCOV_EXCL_STOP */ } return 0; @@ -180,7 +180,7 @@ static bool __ctsvc_server_check_db() return __ctsvc_server_check_db_table(db); } - + /* LCOV_EXCL_START */ static void __ctsvc_server_backup_file() { int i; @@ -214,16 +214,18 @@ static void __ctsvc_server_backup_file() ERR("Unable to remove file"); } } - + /* LCOV_EXCL_STOP */ int ctsvc_server_check_schema(void) { if (false == __ctsvc_server_db_file_exist()) { + /* LCOV_EXCL_START */ ERR("Need to remake db files"); __ctsvc_server_remake_db_file(); } else if (false == __ctsvc_server_check_db()) { ERR("Need to backup and remake db files"); __ctsvc_server_backup_file(); __ctsvc_server_remake_db_file(); + /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; diff --git a/server/ctsvc_server_phonelog.c b/server/ctsvc_server_phonelog.c index d43cc20..abc5b05 100644 --- a/server/ctsvc_server_phonelog.c +++ b/server/ctsvc_server_phonelog.c @@ -49,19 +49,19 @@ int ctsvc_phone_log_reset_statistics_by_sim(int sim_slot_no) int sim_info_id; RETVM_IF(false == ctsvc_server_have_telephony_feature(), CONTACTS_ERROR_NOT_SUPPORTED, "Telephony feature disabled"); + /* LCOV_EXCL_START */ sim_info_id = ctsvc_server_sim_get_info_id_by_sim_slot_no(sim_slot_no); if (sim_info_id <= 0) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_sim_get_info_id_by_sim_slot_no() Fail(%d)", sim_info_id); return CONTACTS_ERROR_NO_DATA; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "DELETE FROM "CTS_TABLE_PHONELOG_STAT " WHERE sim_id = %d", sim_info_id); return ctsvc_query_exec(query); + /* LCOV_EXCL_STOP */ } - + /* LCOV_EXCL_START */ int ctsvc_phone_log_delete(contacts_phone_log_delete_e op, ...) { int ret; @@ -97,27 +97,23 @@ int ctsvc_phone_log_delete(contacts_phone_log_delete_e op, ...) extra_data1, CONTACTS_PLOG_TYPE_EMAIL_RECEIVED, CONTACTS_PLOG_TYPE_EMAIL_SENT); break; default: - /* LCOV_EXCL_START */ ERR("the operation is not proper (op : %d)", op); return CONTACTS_ERROR_INVALID_PARAMETER; - /* LCOV_EXCL_STOP */ } ret = ctsvc_begin_trans(); RETVM_IF(ret, ret, "ctsvc_begin_trans() Fail(%d)", ret); ret = ctsvc_query_exec(query); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_query_exec() Fail(%d)", ret); ctsvc_end_trans(false); return ret; - /* LCOV_EXCL_STOP */ } ctsvc_set_phonelog_noti(); ret = ctsvc_end_trans(true); return ret; } - + /* LCOV_EXCL_STOP */ void ctsvc_db_phone_log_delete_callback(sqlite3_context *context, int argc, sqlite3_value **argv) { @@ -125,8 +121,10 @@ void ctsvc_db_phone_log_delete_callback(sqlite3_context *context, int phone_log_id; if (argc < 1) { + /* LCOV_EXCL_START */ sqlite3_result_null(context); return; + /* LCOV_EXCL_STOP */ } phone_log_id = sqlite3_value_int(argv[0]); @@ -254,7 +252,9 @@ int ctsvc_db_phone_log_update_person_id(const char *number, int old_person_id, "OR (minmatch = ? AND _NUMBER_COMPARE_(normal_num, ?, NULL, NULL))) "); bind_text = g_slist_append(bind_text, strdup(minmatch)); bind_text = g_slist_append(bind_text, strdup(normal_num)); - } else { + } + /* LCOV_EXCL_START */ + else { len += snprintf(query+len, sizeof(query)-len, ") "); } } else { @@ -263,6 +263,7 @@ int ctsvc_db_phone_log_update_person_id(const char *number, int old_person_id, } else { len += snprintf(query+len, sizeof(query)-len, ") "); } + /* LCOV_EXCL_STOP */ } if (0 < old_person_id) @@ -324,8 +325,10 @@ int ctsvc_db_phone_log_update_person_id(const char *number, int old_person_id, /* CASE : number is inserted (contact insert/update) => update person_id of phone logs from NULL */ if (number && old_person_id <= 0 && 0 < candidate_person_id) { + /* LCOV_EXCL_START */ __ctsvc_db_phone_log_find_person_id(address, normal_address, minmatch_address, candidate_person_id, &number_type); + /* LCOV_EXCL_STOP */ new_person_id = candidate_person_id; } else if (number && old_person_id <= 0) { /* CASE : phonelog insert without person_id */ @@ -334,7 +337,7 @@ int ctsvc_db_phone_log_update_person_id(const char *number, int old_person_id, minmatch_address, -1, &number_type); if (new_person_id <= 0) continue; - + /* LCOV_EXCL_START */ if (found_person_id) *found_person_id = new_person_id; } else if (number && 0 < old_person_id) { @@ -376,12 +379,15 @@ int ctsvc_db_phone_log_update_person_id(const char *number, int old_person_id, ctsvc_stmt_bind_int(update_log, 3, phonelog_id); ctsvc_stmt_step(update_log); ctsvc_stmt_reset(update_log); + /* LCOV_EXCL_STOP */ } ctsvc_stmt_finalize(get_log); ctsvc_stmt_finalize(update_log); if (need_noti) + /* LCOV_EXCL_START */ ctsvc_set_phonelog_noti(); + /* LCOV_EXCL_STOP */ if (bind_text) { for (cursor = bind_text; cursor; cursor = cursor->next) diff --git a/server/ctsvc_server_service.c b/server/ctsvc_server_service.c index 43483fc..d0bd923 100644 --- a/server/ctsvc_server_service.c +++ b/server/ctsvc_server_service.c @@ -81,25 +81,32 @@ int ctsvc_disconnect() if (1 == thread_connection) { ctsvc_db_deinit(); - } else if (thread_connection <= 0) { + } + /* LCOV_EXCL_START */ + else if (thread_connection <= 0) { DBG("System : please call contacts_connect_on_thread(), thread_connection count is (%d)", thread_connection); ctsvc_mutex_unlock(CTS_MUTEX_CONNECTION); return CONTACTS_ERROR_INVALID_PARAMETER; } + /* LCOV_EXCL_STOP */ thread_connection--; if (1 == ctsvc_connection) { + /* LCOV_EXCL_START */ ctsvc_inotify_close(); ctsvc_deregister_vconf(); ctsvc_view_uri_deinit(); ctsvc_db_plugin_deinit(); ctsvc_deinit_tapi_handle_for_cc(); + /* LCOV_EXCL_STOP */ } else if (1 < ctsvc_connection) { DBG("System : connection count is %d", ctsvc_connection); } else { + /* LCOV_EXCL_START */ DBG("System : please call contacts_connect(), connection count is (%d)", ctsvc_connection); ctsvc_mutex_unlock(CTS_MUTEX_CONNECTION); return CONTACTS_ERROR_INVALID_PARAMETER; + /* LCOV_EXCL_STOP */ } ctsvc_connection--; ctsvc_mutex_unlock(CTS_MUTEX_CONNECTION); diff --git a/server/ctsvc_server_setting.c b/server/ctsvc_server_setting.c index b8e7588..dab399f 100644 --- a/server/ctsvc_server_setting.c +++ b/server/ctsvc_server_setting.c @@ -49,8 +49,10 @@ int ctsvc_setting_get_name_display_order(contacts_name_display_order_e *order) int ret; if (name_display_order < 0) { + /* LCOV_EXCL_START */ ret = vconf_get_int(CTSVC_VCONF_DISPLAY_ORDER, &name_display_order); RETVM_IF(ret < 0, CONTACTS_ERROR_SYSTEM, "System : vconf_get_int() Fail(%d)", ret); + /* LCOV_EXCL_STOP */ } *order = name_display_order; @@ -79,8 +81,10 @@ int ctsvc_setting_get_name_sorting_order(contacts_name_sorting_order_e *order) { int ret; if (name_sorting_order < 0) { + /* LCOV_EXCL_START */ ret = vconf_get_int(CTSVC_VCONF_SORTING_ORDER, &name_sorting_order); RETVM_IF(ret < 0, CONTACTS_ERROR_SYSTEM, "System : vconf_get_int(sort order) Fail(%d)", ret); + /* LCOV_EXCL_STOP */ } *order = name_sorting_order; @@ -145,8 +149,10 @@ int ctsvc_register_vconf(void) /* display order */ ret = vconf_get_int(CTSVC_VCONF_DISPLAY_ORDER, &name_display_order); if (ret < 0) { + /* LCOV_EXCL_START */ ERR("vconf_get_int() Fail(%d)", ret); name_display_order = CONTACTS_NAME_DISPLAY_ORDER_FIRSTLAST; + /* LCOV_EXCL_STOP */ } ret = vconf_notify_key_changed(CTSVC_VCONF_DISPLAY_ORDER, ctsvc_vconf_display_order_cb, NULL); @@ -155,8 +161,10 @@ int ctsvc_register_vconf(void) /* sorting order */ ret = vconf_get_int(CTSVC_VCONF_SORTING_ORDER, &name_sorting_order); if (ret < 0) { + /* LCOV_EXCL_START */ ERR("vconf_get_int() Fail(%d)", ret); name_sorting_order = CONTACTS_NAME_SORTING_ORDER_FIRSTLAST; + /* LCOV_EXCL_STOP */ } ret = vconf_notify_key_changed(CTSVC_VCONF_SORTING_ORDER, ctsvc_vconf_sorting_order_cb, NULL); @@ -165,8 +173,10 @@ int ctsvc_register_vconf(void) /* phonenumber min match digit */ ret = vconf_get_int(CTSVC_VCONF_PHONENUMBER_MIN_MATCH_DIGIT, &phonenumber_min_match_digit); if (ret < 0) { + /* LCOV_EXCL_START */ ERR("vconf_get_int() Fail(%d)", ret); phonenumber_min_match_digit = 8; + /* LCOV_EXCL_STOP */ } ret = vconf_get_int(ctsvc_get_default_sort_vconfkey(), &primary_sort); @@ -179,7 +189,7 @@ int ctsvc_register_vconf(void) return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ void ctsvc_deregister_vconf(void) { int ret; @@ -193,26 +203,28 @@ void ctsvc_deregister_vconf(void) ret = vconf_ignore_key_changed(ctsvc_get_default_sort_vconfkey(), ctsvc_vconf_sort_change_cb); RETM_IF(ret < 0, "vconf_ignore_key_changed(default_lang) Fail(%d)", ret); } - + /* LCOV_EXCL_STOP */ int ctsvc_get_phonenumber_min_match_digit(void) { int ret = 0; if (phonenumber_min_match_digit <= 0) { + /* LCOV_EXCL_START */ ret = vconf_get_int(CTSVC_VCONF_PHONENUMBER_MIN_MATCH_DIGIT, &phonenumber_min_match_digit); if (ret < 0) { ERR("vconf_get_int() Fail(%d)", ret); phonenumber_min_match_digit = 8; } + /* LCOV_EXCL_STOP */ } return phonenumber_min_match_digit; } - + /* LCOV_EXCL_START */ void ctsvc_set_phonenumber_min_match_digit(int min) { phonenumber_min_match_digit = min; } - + /* LCOV_EXCL_STOP */ const char* ctsvc_get_default_sort_vconfkey(void) { @@ -222,10 +234,12 @@ const char* ctsvc_get_default_sort_vconfkey(void) int ctsvc_get_primary_sort(void) { if (primary_sort < 0) { + /* LCOV_EXCL_START */ int ret; ret = vconf_get_int(ctsvc_get_default_sort_vconfkey(), &primary_sort); WARN_IF(ret < 0, "vconf_get_int() Fail(%d)", ret); ctsvc_set_sort_memory(primary_sort); + /* LCOV_EXCL_STOP */ } return primary_sort; } diff --git a/server/ctsvc_server_sim.c b/server/ctsvc_server_sim.c index eeac0d3..ee9ef51 100644 --- a/server/ctsvc_server_sim.c +++ b/server/ctsvc_server_sim.c @@ -123,7 +123,7 @@ static TapiHandle* __ctsvc_server_sim_get_tapi_handle(ctsvc_sim_info_s *info) return info->handle; } - + /* LCOV_EXCL_START */ static inline void __ctsvc_server_sim_set_return_data(void *data) { greturn_data = data; @@ -158,12 +158,9 @@ static sim_contact_s * __ctsvc_server_sim_record_clone(TelSimPbRecord_t *sim_rec { sim_contact_s *record = calloc(1, sizeof(sim_contact_s)); if (NULL == record) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); return NULL; - /* LCOV_EXCL_STOP */ } -/* LCOV_EXCL_START */ record->sim_index = sim_record->index; record->name = SAFE_STRDUP((char*)sim_record->name); record->nickname = SAFE_STRDUP((char*)sim_record->sne); @@ -178,7 +175,7 @@ static sim_contact_s * __ctsvc_server_sim_record_clone(TelSimPbRecord_t *sim_rec return record; } - + /* LCOV_EXCL_STOP */ static inline int __ctsvc_server_sim_record_add_num(contacts_record_h *record, char *number) { int ret; @@ -372,13 +369,11 @@ static int __ctsvc_server_sim_insert_record_to_db(sim_contact_s *record) /* LCOV_EXCL_START */ ERR("__ctsvc_server_sim_ctsvc_record_clone() Fail(%d)", ret); return CONTACTS_ERROR_INTERNAL; - /* LCOV_EXCL_STOP */ } ret = ctsvc_db_insert_record(contact, NULL); contacts_record_destroy(contact, true); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_db_insert_record() Fail(%d)", ret); return ret; /* LCOV_EXCL_STOP */ @@ -386,7 +381,7 @@ static int __ctsvc_server_sim_insert_record_to_db(sim_contact_s *record) return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static void __ctsvc_server_sim_import_contact_cb(TapiHandle *handle, int result, void *data, void *user_data) { CTS_FN_CALL; @@ -397,24 +392,20 @@ static void __ctsvc_server_sim_import_contact_cb(TapiHandle *handle, int result, ctsvc_sim_info_s *info; if (NULL == sim_info) { - /* LCOV_EXCL_START */ ERR("sim_info is NULL, result = %d", access_rt); ret = ctsvc_server_socket_return(__ctsvc_server_sim_get_return_data(), CONTACTS_ERROR_SYSTEM, 0, NULL); WARN_IF(CONTACTS_ERROR_NONE != ret, "ctsvc_server_socket_return() Fail(%d)", ret); __ctsvc_server_sim_set_return_data(NULL); return; - /* LCOV_EXCL_STOP */ } info = __ctsvc_server_sim_get_info_by_tapi_handle(handle); if (NULL == info) { - /* LCOV_EXCL_START */ ERR("__ctsvc_server_sim_get_info_by_tapi_handle() Fail"); ret = ctsvc_server_socket_return(__ctsvc_server_sim_get_return_data(), CONTACTS_ERROR_INTERNAL, 0, NULL); WARN_IF(CONTACTS_ERROR_NONE != ret, "ctsvc_server_socket_return() Fail(%d)", ret); __ctsvc_server_sim_set_return_data(NULL); return; - /* LCOV_EXCL_STOP */ } if (sim_import_cancel) { @@ -434,7 +425,6 @@ static void __ctsvc_server_sim_import_contact_cb(TapiHandle *handle, int result, DBG("TAPI_SIM_PB_INVALID_INDEX : start_index = %d", start_index); start_index++; if (info->file_record[TAPI_PB_3G_NAME].index_max < start_index) { - /* LCOV_EXCL_START */ ERR("start_index is invalid start_index = %d, total = %d", start_index, info->file_record[TAPI_PB_3G_NAME].index_max); ret = ctsvc_server_socket_return(__ctsvc_server_sim_get_return_data(), CONTACTS_ERROR_INTERNAL, 0, NULL); @@ -442,32 +432,27 @@ static void __ctsvc_server_sim_import_contact_cb(TapiHandle *handle, int result, __ctsvc_server_sim_init_import_info(info); __ctsvc_server_sim_set_return_data(NULL); return; - /* LCOV_EXCL_STOP */ } ret = tel_read_sim_pb_record(handle, info->sim_type, start_index, __ctsvc_server_sim_import_contact_cb, (void*)start_index); if (TAPI_API_SUCCESS != ret) { - /* LCOV_EXCL_START */ ERR("SIM phonebook access Fail(%d) start_indext(%d)", access_rt, start_index); ret = ctsvc_server_socket_return(__ctsvc_server_sim_get_return_data(), CONTACTS_ERROR_SYSTEM, 0, NULL); WARN_IF(CONTACTS_ERROR_NONE != ret, "ctsvc_server_socket_return() Fail(%d)", ret); __ctsvc_server_sim_init_import_info(info); __ctsvc_server_sim_set_return_data(NULL); return; - /* LCOV_EXCL_STOP */ } return; } if (TAPI_SIM_PB_SUCCESS != access_rt) { - /* LCOV_EXCL_START */ ERR("SIM phonebook access Fail(%d)", access_rt); ret = ctsvc_server_socket_return(__ctsvc_server_sim_get_return_data(), CONTACTS_ERROR_SYSTEM, 0, NULL); WARN_IF(CONTACTS_ERROR_NONE != ret, "ctsvc_server_socket_return() Fail(%d)", ret); __ctsvc_server_sim_init_import_info(info); __ctsvc_server_sim_set_return_data(NULL); return; - /* LCOV_EXCL_STOP */ } switch (sim_info->phonebook_type) { @@ -486,14 +471,12 @@ static void __ctsvc_server_sim_import_contact_cb(TapiHandle *handle, int result, case TAPI_SIM_PB_FDN: case TAPI_SIM_PB_SDN: default: - /* LCOV_EXCL_START */ ERR("Unknown storage type(%d)", sim_info->phonebook_type); ret = ctsvc_server_socket_return(__ctsvc_server_sim_get_return_data(), CONTACTS_ERROR_SYSTEM, 0, NULL); WARN_IF(CONTACTS_ERROR_NONE != ret, "ctsvc_server_socket_return() Fail(%d)", ret); __ctsvc_server_sim_init_import_info(info); __ctsvc_server_sim_set_return_data(NULL); return; - /* LCOV_EXCL_STOP */ } if (sim_info->next_index && CTSVC_TAPI_SIM_PB_MAX != sim_info->next_index) { @@ -502,14 +485,12 @@ static void __ctsvc_server_sim_import_contact_cb(TapiHandle *handle, int result, sim_info->next_index, __ctsvc_server_sim_import_contact_cb, NULL); if (TAPI_API_SUCCESS != ret) { - /* LCOV_EXCL_START */ ERR("tel_read_sim_pb_record() Fail(%d)", ret); ret = ctsvc_server_socket_return(__ctsvc_server_sim_get_return_data(), CONTACTS_ERROR_SYSTEM, 0, NULL); WARN_IF(CONTACTS_ERROR_NONE != ret, "ctsvc_server_socket_return() Fail(%d)", ret); __ctsvc_server_sim_init_import_info(info); __ctsvc_server_sim_set_return_data(NULL); return; - /* LCOV_EXCL_STOP */ } } else { if (info->file_record[TAPI_PB_3G_NAME].used_count == info->imported_cnt) { @@ -551,11 +532,9 @@ int ctsvc_server_sim_import_contact(void *data, int sim_slot_no) ret = tel_read_sim_pb_record(__ctsvc_server_sim_get_tapi_handle(info), info->sim_type, 1, __ctsvc_server_sim_import_contact_cb, NULL); if (TAPI_API_SUCCESS != ret) { - /* LCOV_EXCL_START */ ERR("tel_read_sim_pb_record = %d", ret); __ctsvc_server_sim_set_return_data(NULL); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; @@ -597,11 +576,9 @@ static void __ctsvc_server_sim_sdn_read_cb(TapiHandle *handle, int result, void __ctsvc_server_sim_sdn_read_cb, info); RETM_IF(TAPI_API_SUCCESS != ret, "tel_read_sim_pb_record() Fail(%d)", ret); } - /* LCOV_EXCL_START */ ERR("SIM phonebook access Fail(%d)", access_rt); ctsvc_server_trim_memory(); return; - /* LCOV_EXCL_STOP */ } switch (sim_info->phonebook_type) { @@ -613,11 +590,9 @@ static void __ctsvc_server_sim_sdn_read_cb(TapiHandle *handle, int result, void case TAPI_SIM_PB_3GSIM: case TAPI_SIM_PB_FDN: default: - /* LCOV_EXCL_START */ ERR("Not SDN type(%d)", sim_info->phonebook_type); ctsvc_server_trim_memory(); return; - /* LCOV_EXCL_STOP */ } if (sim_info->next_index && CTSVC_TAPI_SIM_PB_MAX != sim_info->next_index) { @@ -625,11 +600,9 @@ static void __ctsvc_server_sim_sdn_read_cb(TapiHandle *handle, int result, void ret = tel_read_sim_pb_record(handle, sim_info->phonebook_type, sim_info->next_index, __ctsvc_server_sim_sdn_read_cb, info); if (TAPI_API_SUCCESS != ret) { - /* LCOV_EXCL_START */ ERR("tel_read_sim_pb_record() Fail(%d)", ret); ctsvc_server_trim_memory(); return; - /* LCOV_EXCL_STOP */ } } else { ctsvc_server_trim_memory(); @@ -650,16 +623,12 @@ static void __ctsvc_server_sim_sdn_count_cb(TapiHandle *handle, int result, void if (0 < ps->UsedRecordCount) { ret = tel_read_sim_pb_record(info->handle, TAPI_SIM_PB_SDN, 1, __ctsvc_server_sim_sdn_read_cb, info); if (TAPI_API_SUCCESS != ret) { - /* LCOV_EXCL_START */ ERR("tel_read_sim_pb_record() Fail(%d)", ret); ctsvc_server_trim_memory(); return; - /* LCOV_EXCL_STOP */ } } else { - /* LCOV_EXCL_START */ ERR("ps->UsedRecordCount:0. No SDN!!!!"); - /* LCOV_EXCL_STOP */ } ctsvc_server_trim_memory(); } @@ -676,47 +645,37 @@ static int __ctsvc_server_sim_sdn_read(ctsvc_sim_info_s *info) ret = tel_get_sim_init_info(info->handle, &sim_status, &card_changed); if (TAPI_API_SUCCESS != ret) { - /* LCOV_EXCL_START */ ERR("tel_get_sim_init_info() Fail(%d)", ret); DBG("sim_status = %d, card_changed = %d", sim_status, card_changed); ctsvc_server_trim_memory(); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } if (TAPI_SIM_STATUS_CARD_NOT_PRESENT == sim_status || TAPI_SIM_STATUS_CARD_REMOVED == sim_status) { ret = ctsvc_server_delete_sdn_contact(info->sim_slot_no); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_delete_sdn_contact() Fail(%d)", ret); ctsvc_server_trim_memory(); return ret; - /* LCOV_EXCL_STOP */ } } else if (TAPI_SIM_STATUS_SIM_INIT_COMPLETED == sim_status) { ret = ctsvc_server_delete_sdn_contact(info->sim_slot_no); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_delete_sdn_contact() Fail(%d)", ret); ctsvc_server_trim_memory(); return ret; - /* LCOV_EXCL_STOP */ } ret = tel_get_sim_pb_count(info->handle, TAPI_SIM_PB_SDN, __ctsvc_server_sim_sdn_count_cb, info); if (TAPI_API_SUCCESS != ret) { - /* LCOV_EXCL_START */ ERR("tel_get_sim_pb_meta_info() Fail(%d)", ret); ctsvc_server_trim_memory(); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } } else { - /* LCOV_EXCL_START */ ERR("sim_status Fail(%d)", sim_status); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; @@ -772,10 +731,8 @@ static void __ctsvc_server_sim_get_meta_info_cb(TapiHandle *handle, int result, info->file_record[CTSVC_2GSIM_NAME].text_max, info->file_record[CTSVC_2GSIM_NAME].used_count); } else { - /* LCOV_EXCL_START */ ERR("sim_type [%d]error ", info->sim_type); return; - /* LCOV_EXCL_STOP */ } if (false == info->initialized) { @@ -819,10 +776,8 @@ static int __ctsvc_server_sim_get_type_and_iccid(ctsvc_sim_info_s *info) } else if (TAPI_SIM_CARD_TYPE_GSM == cardtype) { info->sim_type = TAPI_SIM_PB_ADN; } else { - /* LCOV_EXCL_START */ ERR("cardtype(%d)is invalid!!!", cardtype); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } /* set iccid : unique info of SIM */ @@ -849,10 +804,8 @@ static int __ctsvc_server_sim_init_meta_info(ctsvc_sim_info_s *info) } else if (TAPI_SIM_PB_ADN == info->sim_type) { ret = tel_get_sim_pb_meta_info(info->handle, info->sim_type, __ctsvc_server_sim_get_meta_info_cb, info); } else { - /* LCOV_EXCL_START */ ERR("info->sim_type is invalid(%d) stop sim init !!!", info->sim_type); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } RETVM_IF(TAPI_API_SUCCESS != ret, CONTACTS_ERROR_SYSTEM, @@ -882,9 +835,7 @@ static void __ctsvc_server_sim_noti_pb_status(TapiHandle *handle, const char *no info->initialized = false; __ctsvc_server_sim_init_import_info(info); } else { - /* LCOV_EXCL_START */ ERR("This noti did not control !!!"); - /* LCOV_EXCL_STOP */ } } @@ -930,10 +881,8 @@ static void __ctsvc_server_sim_status_events_cb(TapiHandle *handle, const char * ctsvc_sim_info_s *info = (ctsvc_sim_info_s *)user_data; if (TAPI_SIM_STATUS_SIM_INIT_COMPLETED != status) { - /* LCOV_EXCL_START */ ERR("sim is not ready (%d)", status); return; - /* LCOV_EXCL_STOP */ } INFO("sim is Ready"); @@ -970,11 +919,9 @@ static int __ctsvc_server_sim_init_info() int card_changed = 0; ctsvc_sim_info_s *info = calloc(1, sizeof(ctsvc_sim_info_s)); if (NULL == info) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); g_strfreev(cp_name); return CONTACTS_ERROR_OUT_OF_MEMORY; - /* LCOV_EXCL_STOP */ } info->cp_name = strdup(cp_name[cp_index]); INFO("SIM cp_name[%d] : %s", cp_index, info->cp_name); @@ -994,24 +941,18 @@ static int __ctsvc_server_sim_init_info() __ctsvc_server_sim_get_tapi_handle(info); if (NULL == info->handle) { - /* LCOV_EXCL_START */ ERR("__ctsvc_server_sim_get_tapi_handle() Fail"); continue; - /* LCOV_EXCL_STOP */ } __ctsvc_sim_info = g_slist_append(__ctsvc_sim_info, (void*)info); cp_index++; ret = tel_get_sim_init_info(info->handle, &status, &card_changed); if (TAPI_API_SUCCESS != ret || TAPI_SIM_STATUS_SIM_INIT_COMPLETED != status) { - /* LCOV_EXCL_START */ ERR("tel_get_sim_init_info() ret(%d), status(%d)", ret, status); ret = tel_register_noti_event(info->handle, TAPI_NOTI_SIM_STATUS, __ctsvc_server_sim_status_events_cb, info); - /* LCOV_EXCL_STOP */ if (TAPI_API_SUCCESS != ret) - /* LCOV_EXCL_START */ ERR("tel_register_noti_event() Fail(%d)", ret); - /* LCOV_EXCL_STOP */ } else { __ctsvc_server_sim_get_info(info); } @@ -1028,10 +969,8 @@ static void __ctsvc_server_telephony_ready_cb(keynode_t *key, void *data) vconf_get_bool(VCONFKEY_TELEPHONY_READY, &bReady); if (0 == bReady) { - /* LCOV_EXCL_START */ ERR("telephony is not ready "); return; - /* LCOV_EXCL_STOP */ } INFO("telephony is Ready"); @@ -1042,7 +981,7 @@ static void __ctsvc_server_telephony_ready_cb(keynode_t *key, void *data) __ctsvc_server_sim_init_info(); ctsvc_server_start_timeout(); } - + /* LCOV_EXCL_STOP */ int ctsvc_server_sim_init() { @@ -1055,10 +994,10 @@ int ctsvc_server_sim_init() vconf_notify_key_changed(VCONFKEY_TELEPHONY_READY, __ctsvc_server_telephony_ready_cb, NULL); __ctsvc_tapi_cb = true; return CONTACTS_ERROR_NONE; - /* LCOV_EXCL_STOP */ } return __ctsvc_server_sim_init_info(); + /* LCOV_EXCL_STOP */ } int ctsvc_server_sim_final(void) @@ -1071,6 +1010,7 @@ int ctsvc_server_sim_final(void) vconf_ignore_key_changed(VCONFKEY_TELEPHONY_READY, __ctsvc_server_telephony_ready_cb); for (info_cursor = __ctsvc_sim_info; info_cursor; info_cursor = info_cursor->next) { +/* LCOV_EXCL_START */ ctsvc_sim_info_s *info = info_cursor->data; free(info->cp_name); free(info->sim_unique_id); @@ -1086,7 +1026,7 @@ int ctsvc_server_sim_final(void) free(info); } +/* LCOV_EXCL_STOP */ return CONTACTS_ERROR_NONE; } -/* LCOV_EXCL_STOP */ diff --git a/server/ctsvc_server_socket.c b/server/ctsvc_server_socket.c index 7bbfd6f..0f25b69 100644 --- a/server/ctsvc_server_socket.c +++ b/server/ctsvc_server_socket.c @@ -57,7 +57,7 @@ struct client_info { /* key : socket_fd, value : struct client_info* */ static GHashTable *_client_info_table = NULL; - +/* LCOV_EXCL_START */ static inline int __ctsvc_server_socket_safe_write(int fd, char *buf, int buf_size) { int ret, writed = 0; @@ -161,11 +161,9 @@ static void __ctsvc_server_socket_import_sim(GIOChannel *src, int size) if (0 < size) { g_io_channel_read_chars(src, receiver, size, &len, &gerr); if (gerr) { - /* LCOV_EXCL_START */ ERR("g_io_channel_read_chars() Fail(%s)", gerr->message); g_error_free(gerr); return; - /* LCOV_EXCL_STOP */ } DBG("Receiver = %s(%d), read_size = %d", receiver, len, size); } @@ -179,10 +177,8 @@ static void __ctsvc_server_socket_import_sim(GIOChannel *src, int size) } if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_sim_import_contact() Fail(%d)", ret); ctsvc_server_socket_return(src, ret, 0, NULL); - /* LCOV_EXCL_STOP */ } } @@ -198,11 +194,9 @@ static void __ctsvc_server_socket_get_sim_init_status(GIOChannel *src, int size) if (0 < size) { g_io_channel_read_chars(src, receiver, size, &len, &gerr); if (gerr) { - /* LCOV_EXCL_START */ ERR("g_io_channel_read_chars() Fail(%s)", gerr->message); g_error_free(gerr); return; - /* LCOV_EXCL_STOP */ } DBG("Receiver = %s(%d), read_size = %d", receiver, len, size); } @@ -216,10 +210,8 @@ static void __ctsvc_server_socket_get_sim_init_status(GIOChannel *src, int size) } if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_socket_get_sim_init_status() Fail(%d)", ret); ctsvc_server_socket_return(src, ret, 0, NULL); - /* LCOV_EXCL_STOP */ } } @@ -251,13 +243,11 @@ static void __ctsvc_server_socket_read_flush(GIOChannel *src, int size) g_io_channel_read_chars(src, receiver, size, &len, &gerr); if (gerr) { - /* LCOV_EXCL_START */ ERR("g_io_channel_read_chars() Fail(%s)", gerr->message); g_error_free(gerr); - /* LCOV_EXCL_STOP */ } } - + /* LCOV_EXCL_STOP */ static cynara *_cynara = NULL; static int _ctsvc_server_initialize_cynara() @@ -293,7 +283,7 @@ static void _ctsvc_server_finalize_cynara() /* LCOV_EXCL_STOP */ } } - + /* LCOV_EXCL_START */ static bool _ctsvc_server_check_privilege(struct client_info *info, const char *privilege) { RETV_IF(NULL == info, false); @@ -306,7 +296,7 @@ static bool _ctsvc_server_check_privilege(struct client_info *info, const char * return false; } - + /* LCOV_EXCL_STOP */ static gboolean __ctsvc_server_socket_request_handler(GIOChannel *src, GIOCondition condition, gpointer data) { @@ -327,7 +317,7 @@ static gboolean __ctsvc_server_socket_request_handler(GIOChannel *src, GIOCondit close(fd); return FALSE; } - + /* LCOV_EXCL_START */ ret = __ctsvc_server_socket_safe_read(fd, (char *)&msg, sizeof(msg)); RETVM_IF(-1 == ret, TRUE, "__ctsvc_server_socket_safe_read() Fail(errno = %d)", errno); @@ -336,12 +326,10 @@ static gboolean __ctsvc_server_socket_request_handler(GIOChannel *src, GIOCondit bool have_telephony_feature = false; have_telephony_feature = ctsvc_server_have_telephony_feature(); if (false == have_telephony_feature) { - /* LCOV_EXCL_START */ ERR("Telephony feature disabled"); __ctsvc_server_socket_read_flush(src, msg.attach_sizes[0]); /* sim_id */ ctsvc_server_socket_return(src, CONTACTS_ERROR_NOT_SUPPORTED, 0, NULL); return TRUE; - /* LCOV_EXCL_STOP */ } ctsvc_mutex_lock(CTS_MUTEX_SOCKET_CLIENT_INFO); @@ -358,23 +346,19 @@ static gboolean __ctsvc_server_socket_request_handler(GIOChannel *src, GIOCondit switch (msg.type) { case CTSVC_SOCKET_MSG_TYPE_REQUEST_IMPORT_SIM: if (false == have_write_permission) { - /* LCOV_EXCL_START */ ERR("write permission denied"); __ctsvc_server_socket_read_flush(src, msg.attach_sizes[0]); /* sim_id */ ctsvc_server_socket_return(src, CONTACTS_ERROR_PERMISSION_DENIED, 0, NULL); return TRUE; - /* LCOV_EXCL_STOP */ } __ctsvc_server_socket_import_sim(src, msg.attach_sizes[0]); break; case CTSVC_SOCKET_MSG_TYPE_REQUEST_SIM_INIT_COMPLETE: if (false == have_read_permission) { - /* LCOV_EXCL_START */ ERR("read permission denied"); __ctsvc_server_socket_read_flush(src, msg.attach_sizes[0]); /* sim_id */ ctsvc_server_socket_return(src, CONTACTS_ERROR_PERMISSION_DENIED, 0, NULL); return TRUE; - /* LCOV_EXCL_STOP */ } __ctsvc_server_socket_get_sim_init_status(src, msg.attach_sizes[0]); break; @@ -383,11 +367,10 @@ static gboolean __ctsvc_server_socket_request_handler(GIOChannel *src, GIOCondit break; default: - /* LCOV_EXCL_START */ ERR("Unknown request type(%d)", msg.type); break; - /* LCOV_EXCL_STOP */ } + /* LCOV_EXCL_STOP */ return TRUE; } @@ -513,7 +496,9 @@ int ctsvc_server_socket_init(void) wrn = snprintf(addr.sun_path, sizeof(addr.sun_path), "%s", sock_file); if (wrn < 0){ + /* LCOV_EXCL_START */ WARN("Error writing in sock file (%s)", sock_file); + /* LCOV_EXCL_STOP */ } sockfd = socket(PF_UNIX, SOCK_STREAM, 0); RETVM_IF(-1 == sockfd, CONTACTS_ERROR_SYSTEM, "socket() Fail(errno = %d)", errno); diff --git a/server/ctsvc_server_sqlite.c b/server/ctsvc_server_sqlite.c index 2b81ed2..e087cc6 100644 --- a/server/ctsvc_server_sqlite.c +++ b/server/ctsvc_server_sqlite.c @@ -156,7 +156,7 @@ int ctsvc_server_db_open(sqlite3 **db, int *sqlite_error) *db = server_db; return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ int ctsvc_server_db_close(void) { if (server_db) { @@ -181,10 +181,8 @@ int ctsvc_server_begin_trans(void) } if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec() Fail(%d)", ret); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } return CONTACTS_ERROR_NONE; } @@ -226,10 +224,8 @@ int ctsvc_server_end_trans(bool success) } while ((SQLITE_BUSY == ret || SQLITE_LOCKED == ret) && i < CTS_COMMIT_TRY_MAX); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(ROLLBACK) Fail(%d) : DB lock", ret); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } else { INFO("rollback end"); return CONTACTS_ERROR_NONE; @@ -254,104 +250,88 @@ int ctsvc_server_update_sort(int prev_sort_primary, int prev_sort_secondary, int CTS_TABLE_CONTACTS, prev_sort_primary, CTSVC_SORT_PRIMARY); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE %s SET display_name_language=%d WHERE display_name_language = %d", CTS_TABLE_CONTACTS, prev_sort_secondary, CTSVC_SORT_SECONDARY); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE %s SET display_name_language=%d WHERE display_name_language=%d", CTS_TABLE_CONTACTS, CTSVC_SORT_PRIMARY, new_sort_primary); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE %s SET display_name_language=%d WHERE display_name_language =%d", CTS_TABLE_CONTACTS, CTSVC_SORT_SECONDARY, new_sort_secondary); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE %s SET reverse_display_name_language=%d WHERE reverse_display_name_language = %d", CTS_TABLE_CONTACTS, prev_sort_primary, CTSVC_SORT_PRIMARY); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE %s SET reverse_display_name_language=%d WHERE reverse_display_name_language = %d", CTS_TABLE_CONTACTS, prev_sort_secondary, CTSVC_SORT_SECONDARY); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE %s SET reverse_display_name_language=%d WHERE reverse_display_name_language = %d", CTS_TABLE_CONTACTS, CTSVC_SORT_PRIMARY, new_sort_primary); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE %s SET reverse_display_name_language=%d WHERE reverse_display_name_language = %d", CTS_TABLE_CONTACTS, CTSVC_SORT_SECONDARY, new_sort_secondary); ret = sqlite3_exec(db, query, NULL, NULL, &errmsg); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_exec(%s) Fail(%d, %s)", query, ret, errmsg); sqlite3_free(errmsg); ctsvc_server_end_trans(false); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } ret = ctsvc_server_set_default_sort(new_sort_primary); @@ -391,11 +371,9 @@ int ctsvc_server_insert_sdn_contact(const char *name, const char *number, ret = sqlite3_prepare_v2(db, query, strlen(query), &stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2(%s) Fail(%s)", query, sqlite3_errmsg(db)); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } sqlite3_bind_text(stmt, 1, name, strlen(name), SQLITE_STATIC); @@ -404,12 +382,10 @@ int ctsvc_server_insert_sdn_contact(const char *name, const char *number, ret = sqlite3_step(stmt); if (SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step() Fail(%d)", ret); sqlite3_finalize(stmt); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } sqlite3_finalize(stmt); ctsvc_server_db_close(); @@ -434,20 +410,16 @@ int ctsvc_server_delete_sdn_contact(int sim_slot_no) ret = sqlite3_prepare_v2(db, query, strlen(query), &stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2(%s) Fail(%s)", query, sqlite3_errmsg(db)); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } ret = sqlite3_step(stmt); if (SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step() Fail(%d)", ret); sqlite3_finalize(stmt); ctsvc_server_db_close(); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } sqlite3_finalize(stmt); @@ -467,21 +439,17 @@ int ctsvc_server_update_collation() ret = ctsvc_server_db_open(&db, NULL); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_db_open() Fail(%d)", ret); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return ret; - /* LCOV_EXCL_STOP */ } ret = ctsvc_server_begin_trans(); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_begin_trans() Fail(%d)", ret); ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return ret; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), @@ -489,13 +457,11 @@ int ctsvc_server_update_collation() "FROM "CTS_TABLE_CONTACTS" WHERE deleted = 0"); ret = sqlite3_prepare_v2(db, query, strlen(query), &stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2(%s) Fail(%s)", query, sqlite3_errmsg(db)); ctsvc_server_end_trans(false); ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), @@ -503,14 +469,12 @@ int ctsvc_server_update_collation() "WHERE contact_id = ?"); ret = sqlite3_prepare_v2(db, query, strlen(query), &update_stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2(%s) Fail(%s)", query, sqlite3_errmsg(db)); sqlite3_finalize(stmt); ctsvc_server_end_trans(false); ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } while (SQLITE_ROW == (ret = sqlite3_step(stmt))) { @@ -540,7 +504,6 @@ int ctsvc_server_update_collation() free(reverse_sortkey); if (SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step(%s) Fail(%d, %s)", query, ret, sqlite3_errmsg(db)); sqlite3_finalize(stmt); sqlite3_finalize(update_stmt); @@ -548,7 +511,6 @@ int ctsvc_server_update_collation() ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } sqlite3_reset(update_stmt); @@ -556,7 +518,6 @@ int ctsvc_server_update_collation() } if (SQLITE_ROW != ret && SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step() Fail(%d)", ret); sqlite3_finalize(update_stmt); sqlite3_finalize(stmt); @@ -564,7 +525,6 @@ int ctsvc_server_update_collation() ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } sqlite3_finalize(update_stmt); @@ -614,10 +574,8 @@ int ctsvc_server_get_sim_id(const char *unique_id, int *sim_id) "WHERE unique_id = '%s'", unique_id); ret = ctsvc_query_get_first_int_result(query, &id); if (CONTACTS_ERROR_NONE != ret && CONTACTS_ERROR_NO_DATA != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_query_get_first_int_result() Fail(%d)", ret); return ret; - /* LCOV_EXCL_STOP */ } if (CONTACTS_ERROR_NO_DATA == ret) { @@ -625,10 +583,8 @@ int ctsvc_server_get_sim_id(const char *unique_id, int *sim_id) "INSERT INTO "CTS_TABLE_SIM_INFO" (unique_id) VALUES('%s')", unique_id); ret = ctsvc_query_exec(query); if (ret != CONTACTS_ERROR_NONE) { - /* LCOV_EXCL_START */ ERR("ctsvc_query_exec() Fail(%d)", ret); return ret; - /* LCOV_EXCL_STOP */ } id = ctsvc_db_get_last_insert_id(); } @@ -658,11 +614,9 @@ static int __ctsvc_server_db_get_contact_data(sqlite3 *db, int id, ctsvc_contact ret = sqlite3_step(stmt); if (SQLITE_ROW != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step() Fail(%d)", ret); sqlite3_finalize(stmt); return ret; - /* LCOV_EXCL_STOP */ } do { @@ -705,10 +659,8 @@ static int __ctsvc_server_get_contact_id_list(sqlite3* db, GList** contact_id_li int ret = sqlite3_prepare_v2(db, query, strlen(query), &stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2(%s) Fail(%s)", query, sqlite3_errmsg(db)); return CONTACTS_ERROR_SYSTEM; - /* LCOV_EXCL_STOP */ } int contact_id = 0; @@ -723,12 +675,10 @@ static int __ctsvc_server_get_contact_id_list(sqlite3* db, GList** contact_id_li } if (SQLITE_ROW != ret && SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step() Fail(%d)", ret); g_list_free(list); sqlite3_finalize(stmt); return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } list = g_list_reverse(list); @@ -755,32 +705,26 @@ int ctsvc_server_update_sort_name() ret = ctsvc_server_db_open(&db, NULL); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_db_open() Fail(%d)", ret); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return ret; - /* LCOV_EXCL_STOP */ } ret = __ctsvc_server_get_contact_id_list(db, &contact_id_list); if (ret != CONTACTS_ERROR_NONE) { - /* LCOV_EXCL_START */ ERR("__ctsvc_server_get_contact_id_list() Fail(%d)", ret); ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return ret; - /* LCOV_EXCL_STOP */ } ret = ctsvc_server_begin_trans(); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_begin_trans() Fail(%d)", ret); g_list_free(contact_id_list); ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return ret; - /* LCOV_EXCL_STOP */ } /* Update sort_name, sortkey, display_name_language of contact table */ @@ -792,11 +736,9 @@ int ctsvc_server_update_sort_name() "WHERE contact_id = ?"); ret = sqlite3_prepare_v2(db, query, strlen(query), &update_stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_propare_v2(%s) Fail(%s)", query, sqlite3_errmsg(db)); ret = CONTACTS_ERROR_DB; goto DATA_FREE; - /* LCOV_EXCL_STOP */ } /* Update name of search_index table */ @@ -805,11 +747,9 @@ int ctsvc_server_update_sort_name() CTS_TABLE_SEARCH_INDEX); ret = sqlite3_prepare_v2(db, query, strlen(query), &search_name_stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_propare_v2(%s) Fail(%s)", query, sqlite3_errmsg(db)); ret = CONTACTS_ERROR_DB; goto DATA_FREE; - /* LCOV_EXCL_STOP */ } int count = 0; @@ -824,12 +764,10 @@ int ctsvc_server_update_sort_name() contacts_record_create(_contacts_contact._uri, (contacts_record_h*)&contact); ret = __ctsvc_server_db_get_contact_data(db, contact_id, contact); if (SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step(%s) Fail(%d, %s)", query, ret, sqlite3_errmsg(db)); contacts_record_destroy((contacts_record_h)contact, true); ret = CONTACTS_ERROR_DB; goto DATA_FREE; - /* LCOV_EXCL_STOP */ } /* update sort_name, sortkey, display_name_language(sort group) */ @@ -849,12 +787,10 @@ int ctsvc_server_update_sort_name() ret = sqlite3_step(update_stmt); if (SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step(%s) Fail(%d, %s)", query, ret, sqlite3_errmsg(db)); contacts_record_destroy((contacts_record_h)contact, true); ret = CONTACTS_ERROR_DB; goto DATA_FREE; - /* LCOV_EXCL_STOP */ } sqlite3_reset(update_stmt); @@ -866,12 +802,10 @@ int ctsvc_server_update_sort_name() ret = sqlite3_step(search_name_stmt); free(search_name); if (SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step(%s) Fail(%d, %s)", query, ret, sqlite3_errmsg(db)); contacts_record_destroy((contacts_record_h)contact, true); ret = CONTACTS_ERROR_DB; goto DATA_FREE; - /* LCOV_EXCL_STOP */ } sqlite3_reset(search_name_stmt); } @@ -886,7 +820,6 @@ int ctsvc_server_update_sort_name() usleep(100*1000); ret = ctsvc_server_begin_trans(); if (CONTACTS_ERROR_NONE != ret) { - /* LCOV_EXCL_START */ ERR("ctsvc_server_begin_trans() Fail(%d)", ret); g_list_free(contact_id_list); if (update_stmt) @@ -896,7 +829,6 @@ int ctsvc_server_update_sort_name() ctsvc_server_db_close(); ctsvc_db_set_status(CONTACTS_DB_STATUS_NORMAL); return ret; - /* LCOV_EXCL_STOP */ } } } @@ -945,4 +877,4 @@ DATA_FREE: return ret; } - + /* LCOV_EXCL_STOP */ \ No newline at end of file diff --git a/server/ctsvc_server_update.c b/server/ctsvc_server_update.c index 5fa20e5..28f8c3d 100644 --- a/server/ctsvc_server_update.c +++ b/server/ctsvc_server_update.c @@ -37,7 +37,7 @@ #define CTSVC_SCHEMA_VERSION 104 #ifdef ENABLE_LOG_FEATURE - + /* LCOV_EXCL_START */ static int __ctsvc_server_find_person_id_of_phonelog(sqlite3 *__db, char *normal_num, char *minmatch, int person_id, int *find_number_type) { @@ -65,7 +65,6 @@ static int __ctsvc_server_find_person_id_of_phonelog(sqlite3 *__db, char *normal ret = sqlite3_prepare_v2(__db, query, sizeof(query), &stmt, NULL); if (stmt == NULL) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2 Fail(%d)", ret); if (bind_text) { for (cursor = bind_text; cursor; cursor = cursor->next, i++) @@ -73,7 +72,6 @@ static int __ctsvc_server_find_person_id_of_phonelog(sqlite3 *__db, char *normal g_slist_free(bind_text); } return CONTACTS_ERROR_DB; - /* LCOV_EXCL_STOP */ } if (bind_text) { @@ -126,10 +124,8 @@ static void __ctsvc_server_number_info_update(sqlite3 *__db) "WHERE log_type < %d", CONTACTS_PLOG_TYPE_EMAIL_RECEIVED); ret = sqlite3_prepare_v2(__db, query, strlen(query), &stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2() Fail(%s)", sqlite3_errmsg(__db)); return; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), @@ -137,11 +133,9 @@ static void __ctsvc_server_number_info_update(sqlite3 *__db) "minmatch=?, person_id=?, number_type=? WHERE id = ?"); ret = sqlite3_prepare_v2(__db, query, strlen(query), &update_stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2() Fail(%s)", sqlite3_errmsg(__db)); sqlite3_finalize(stmt); return; - /* LCOV_EXCL_STOP */ } while (SQLITE_ROW == (ret = sqlite3_step(stmt))) { @@ -191,21 +185,17 @@ static void __ctsvc_server_number_info_update(sqlite3 *__db) "WHERE datatype = %d", CONTACTS_DATA_TYPE_NUMBER); ret = sqlite3_prepare_v2(__db, query, strlen(query), &stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2() Fail(%s)", sqlite3_errmsg(__db)); return; - /* LCOV_EXCL_STOP */ } snprintf(query, sizeof(query), "UPDATE "CTS_TABLE_DATA" SET data4=?, data5=?, data6=? WHERE id = ?"); ret = sqlite3_prepare_v2(__db, query, strlen(query), &update_stmt, NULL); if (SQLITE_OK != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_prepare_v2() Fail(%s)", sqlite3_errmsg(__db)); sqlite3_finalize(stmt); return; - /* LCOV_EXCL_STOP */ } while (SQLITE_ROW == (ret = sqlite3_step(stmt))) { @@ -228,10 +218,8 @@ static void __ctsvc_server_number_info_update(sqlite3 *__db) sqlite3_bind_int(update_stmt, 4, id); ret = sqlite3_step(update_stmt); if (SQLITE_DONE != ret) { - /* LCOV_EXCL_START */ ERR("sqlite3_step() Fail(%d, %s)", ret, sqlite3_errmsg(__db)); break; - /* LCOV_EXCL_STOP */ } sqlite3_reset(update_stmt); } @@ -241,7 +229,7 @@ static void __ctsvc_server_number_info_update(sqlite3 *__db) return; } - + /* LCOV_EXCL_STOP */ static int __ctsvc_server_get_db_version(sqlite3 *db, int *version) { int ret; @@ -292,6 +280,7 @@ int ctsvc_server_db_update(void) } /* Tizen 2.3a Releae 0.9.114.7 ('14/5) ----------------------------------- */ + /* LCOV_EXCL_START */ if (old_version <= 100) { ret = sqlite3_exec(__db, "CREATE INDEX name_lookup_idx1 ON name_lookup(contact_id);", NULL, 0, &errmsg); if (SQLITE_OK != ret) { @@ -497,5 +486,6 @@ int ctsvc_server_db_update(void) db_util_close(__db); return CONTACTS_ERROR_NONE; + /* LCOV_EXCL_STOP */ } diff --git a/server/ctsvc_server_utils.c b/server/ctsvc_server_utils.c index 1379b0d..5394089 100644 --- a/server/ctsvc_server_utils.c +++ b/server/ctsvc_server_utils.c @@ -67,7 +67,7 @@ inline int ctsvc_server_set_default_sort(int sort) ctsvc_set_sort_memory(sort); return CONTACTS_ERROR_NONE; } - + /* LCOV_EXCL_START */ static void* __ctsvc_server_update_language_on_thread(void *data) { int ret; @@ -85,7 +85,6 @@ static void* __ctsvc_server_update_language_on_thread(void *data) old_secondary_sort = ctsvc_get_secondary_sort(); if (old_primary_sort < 0 || old_secondary_sort < 0) { - /* LCOV_EXCL_START */ ERR("old_primary_sort (%d), old_secondary_sort(%d)", old_primary_sort, old_secondary_sort); free(info->langset); free(info->new_langset); @@ -93,7 +92,6 @@ static void* __ctsvc_server_update_language_on_thread(void *data) ret = vconf_notify_key_changed(VCONFKEY_LANGSET, __ctsvc_server_change_language_cb, NULL); WARN_IF(ret < 0, "vconf_notify_key_changed(%s) Fail(%d)", VCONFKEY_LANGSET, ret); return NULL; - /* LCOV_EXCL_STOP */ } ctsvc_server_stop_timeout(); @@ -156,10 +154,8 @@ static void __ctsvc_server_change_language_cb(keynode_t *key, void *data) new_langset = vconf_keynode_get_str(key); if (NULL == new_langset) { - /* LCOV_EXCL_START */ ERR("vconf_keynode_get_str() Fail"); return; - /* LCOV_EXCL_STOP */ } langset = ctsvc_get_langset(); INFO("%s --> %s", langset, new_langset); @@ -167,10 +163,8 @@ static void __ctsvc_server_change_language_cb(keynode_t *key, void *data) if (STRING_EQUAL != strcmp(langset, new_langset)) { cts_language_update_thread_info_s *info = calloc(1, sizeof(cts_language_update_thread_info_s)); if (NULL == info) { - /* LCOV_EXCL_START */ ERR("calloc() Fail"); return; - /* LCOV_EXCL_STOP */ } ret = vconf_ignore_key_changed(VCONFKEY_LANGSET, __ctsvc_server_change_language_cb); @@ -183,7 +177,7 @@ static void __ctsvc_server_change_language_cb(keynode_t *key, void *data) pthread_create(&worker, NULL, __ctsvc_server_update_language_on_thread, info); } } - + /* LCOV_EXCL_STOP */ void ctsvc_server_final_configuration(void) { int ret = -1; @@ -207,10 +201,12 @@ int ctsvc_server_init_configuration(void) ret = vconf_get_int(ctsvc_get_default_sort_vconfkey(), &sort_type); if (ret < 0 || sort_type == CTSVC_SORT_OTHERS) { + /* LCOV_EXCL_START */ ERR("vconf_get_int(%s) Fail(%d)", ctsvc_get_default_sort_vconfkey(), ret); sort_type = ctsvc_get_sort_type_from_language(system_language); if (sort_type == CTSVC_SORT_OTHERS) sort_type = CTSVC_SORT_WESTERN; + /* LCOV_EXCL_STOP_ */ } ctsvc_server_set_default_sort(sort_type); diff --git a/server/ctsvc_utils_string.c b/server/ctsvc_utils_string.c index db0c9c2..bba0638 100644 --- a/server/ctsvc_utils_string.c +++ b/server/ctsvc_utils_string.c @@ -155,7 +155,7 @@ static int __ctsvc_normalize_str_to_unicode(const char *src, int src_size, UChar } #define SMALL_BUFFER_SIZE 10 - + /* LCOV_EXCL_START */ static bool __ctsvc_compare_pinyin_letter(const char *haystack, int haystack_lang, const char *needle, int needle_lang, int *h_len, int *n_len) { pinyin_name_s *pinyinname = NULL; @@ -184,10 +184,8 @@ static bool __ctsvc_compare_pinyin_letter(const char *haystack, int haystack_lan ret = ctsvc_convert_chinese_to_pinyin(temp, &pinyinname, &size); if (ret != CONTACTS_ERROR_NONE) { - /* LCOV_EXCL_START */ ERR("ctsvc_convert_chinese_to_pinyin Fail(%d)", ret); return false; - /* LCOV_EXCL_STOP */ } for (k = 0; k < size; k++) { @@ -228,13 +226,14 @@ static bool __ctsvc_compare_pinyin_letter(const char *haystack, int haystack_lan return match; } - + /* LCOV_EXCL_STOP */ static bool __ctsvc_compare_unicode_letter(const UChar* haystack, int haystack_lang, const UChar *needle, int needle_lang) { int i, j; bool ret = false; switch (haystack_lang) { + /* LCOV_EXCL_START */ case CTSVC_LANG_NUMBER: case CTSVC_LANG_OTHERS: { @@ -252,6 +251,7 @@ static bool __ctsvc_compare_unicode_letter(const UChar* haystack, int haystack_l } return ret; } + /* LCOV_EXCL_STOP */ case CTSVC_LANG_ENGLISH: { switch (needle_lang) { @@ -259,13 +259,17 @@ static bool __ctsvc_compare_unicode_letter(const UChar* haystack, int haystack_l for (i = 0, j = 0; i < u_strlen(haystack) && j < u_strlen(needle);) { if (CTSVC_COMPARE_BETWEEN(CTSVC_COMBINING_DIACRITICAL_MARKS_START, haystack[i], CTSVC_COMBINING_DIACRITICAL_MARKS_END)) { + /* LCOV_EXCL_START */ i++; continue; + /* LCOV_EXCL_STOP */ } if (CTSVC_COMPARE_BETWEEN(CTSVC_COMBINING_DIACRITICAL_MARKS_START, needle[j], CTSVC_COMBINING_DIACRITICAL_MARKS_END)) { + /* LCOV_EXCL_START */ j++; continue; + /* LCOV_EXCL_STOP */ } if (haystack[i] == needle[j]) { @@ -283,6 +287,7 @@ static bool __ctsvc_compare_unicode_letter(const UChar* haystack, int haystack_l return false; } } + /* LCOV_EXCL_START */ break; case CTSVC_LANG_KOREAN: { @@ -340,6 +345,7 @@ static bool __ctsvc_compare_unicode_letter(const UChar* haystack, int haystack_l } return false; + /* LCOV_EXCL_STOP */ } int ctsvc_utils_string_strstr(const char *haystack, const char *needle, int *len) @@ -388,8 +394,10 @@ int ctsvc_utils_string_strstr(const char *haystack, const char *needle, int *len match_len = h_len_temp; if (temp_needle[j] == '\0') { + /* LCOV_EXCL_START */ *len = match_len; return match_start; + /* LCOV_EXCL_STOP */ } } h_len = h_len_temp; @@ -411,11 +419,13 @@ int ctsvc_utils_string_strstr(const char *haystack, const char *needle, int *len h_len = h_len_temp; continue; } else { + /* LCOV_EXCL_START */ j = 0; matching = false; match_start = -1; match_len = 0; i -= h_len; + /* LCOV_EXCL_STOP */ } } @@ -443,21 +453,28 @@ static bool _get_modified_number(char *temp, char *keyword, int *out_len_keyword if ('\0' == *cursor_keyword) break; if ('\0' == *cursor_number) { + /* LCOV_EXCL_START */ if (true == is_start) /* ended even if keyword is remained */ is_start = false; + /* LCOV_EXCL_STOP */ break; } if (*cursor_number < '0' || '9' < *cursor_number) { + /* LCOV_EXCL_START */ if (true == is_start) len_keyword++; cursor_number++; continue; + /* LCOV_EXCL_STOP */ } if (*cursor_keyword < '0' || '9' < *cursor_keyword) { + /* LCOV_EXCL_START */ cursor_keyword++; continue; + /* LCOV_EXCL_STOP */ } if (*cursor_keyword != *cursor_number) { + /* LCOV_EXCL_START */ if (true == is_start) { is_start = false; cursor_keyword = keyword; @@ -466,6 +483,7 @@ static bool _get_modified_number(char *temp, char *keyword, int *out_len_keyword cursor_number++; } continue; + /* LCOV_EXCL_STOP */ } if (false == is_start) start_needle = cursor_number; @@ -483,7 +501,7 @@ static bool _get_modified_number(char *temp, char *keyword, int *out_len_keyword return true; } - + /* LCOV_EXCL_START */ static char *_strchr_with_nth(char *end_pointer, char *s, char *delimiters, int nth) { RETV_IF(NULL == delimiters, s); @@ -535,7 +553,7 @@ static char *_strrchr_with_nth(char *start_pointer, char *s, char *delimiters, i } return cursor + 1; } - + /* LCOV_EXCL_STOP */ char *ctsvc_utils_get_modified_str(char *temp, bool is_snippet, const char *keyword, const char *start_match, const char *end_match, int token_number) { @@ -581,7 +599,7 @@ char *ctsvc_utils_get_modified_str(char *temp, bool is_snippet, const char *keyw DBG("NUMBER"); is_modified = _get_modified_number(temp, (char *)keyword, &len_keyword, &len_offset); break; - + /* LCOV_EXCL_START */ default: DBG("DATA"); pos_start = strcasestr(temp, keyword); @@ -591,10 +609,13 @@ char *ctsvc_utils_get_modified_str(char *temp, bool is_snippet, const char *keyw len_keyword = strlen(keyword); len_offset = strlen(temp) - strlen(pos_start);; break; + /* LCOV_EXCL_STOP */ } if (false == is_modified) + /* LCOV_EXCL_START */ return strdup(temp); + /* LCOV_EXCL_STOP */ len_full = strlen(temp) + strlen(start_match) + strlen(end_match) + 1; @@ -622,6 +643,7 @@ char *ctsvc_utils_get_modified_str(char *temp, bool is_snippet, const char *keyw if (token_number < 1) return mod_temp; + /* LCOV_EXCL_START */ char *cursor = NULL; cursor = _strchr_with_nth(mod_temp + len_full, keyword_start, " \n", token_number); *cursor = '\0'; @@ -629,6 +651,7 @@ char *ctsvc_utils_get_modified_str(char *temp, bool is_snippet, const char *keyw strncpy(mod_temp, cursor, len_full); return mod_temp; + /* LCOV_EXCL_STOP */ } -- 2.7.4