From: Anjali Nijhara Date: Mon, 11 Sep 2023 05:30:44 +0000 (+0530) Subject: Add LCOV macro for coverage X-Git-Tag: accepted/tizen/unified/20230912.013617^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F82%2F298582%2F1;p=platform%2Fcore%2Fapi%2Fnsd.git Add LCOV macro for coverage Change-Id: I8cfdbd83f4d7e42a07499c4afe9c111faf6e94b5 --- diff --git a/src/dns-sd/dns-sd.c b/src/dns-sd/dns-sd.c index c3abd43..2c96b78 100644 --- a/src/dns-sd/dns-sd.c +++ b/src/dns-sd/dns-sd.c @@ -176,6 +176,7 @@ static dnssd_handle_s *__dnssd_get_struct_from_handle( return NULL; } +//LCOV_EXCL_START static resolve_reply_data *__dnssd_check_resolve_reply_data_validity( resolve_reply_data* data) { @@ -194,6 +195,7 @@ static resolve_reply_data *__dnssd_check_resolve_reply_data_validity( __DNSSD_LOG_FUNC_EXIT__; return NULL; } +//LCOV_EXCL_STOP static bool __dnssd_check_permission() { @@ -208,28 +210,28 @@ static bool __dnssd_check_permission() cynara *p_cynara = NULL; if (CYNARA_API_SUCCESS != cynara_initialize(&p_cynara, NULL)) { - DNSSD_LOGE("Failed to initialize cynara structure"); - return false; + DNSSD_LOGE("Failed to initialize cynara structure"); //LCOV_EXCL_LINE + return false; //LCOV_EXCL_LINE } memset(smack_label, 0x00, SMACK_LABEL_LEN + 1); fd = fopen("/proc/self/attr/current", "r"); if (fd == NULL) { - DNSSD_LOGE("Failed to open /proc/self/attr/current"); - cynara_finish(p_cynara); - return false; + DNSSD_LOGE("Failed to open /proc/self/attr/current"); //LCOV_EXCL_LINE + cynara_finish(p_cynara); //LCOV_EXCL_LINE + return false; //LCOV_EXCL_LINE } ret = fread(smack_label, SMACK_LABEL_LEN, 1, fd); if (ret == 0 && ferror(fd)) { - DNSSD_LOGE("Failed to read /proc/self/attr/current"); - if (fclose(fd) != 0) - DNSSD_LOGE("Failed to fclose"); - cynara_finish(p_cynara); - return false; + DNSSD_LOGE("Failed to read /proc/self/attr/current"); //LCOV_EXCL_LINE + if (fclose(fd) != 0) //LCOV_EXCL_LINE + DNSSD_LOGE("Failed to fclose"); //LCOV_EXCL_LINE + cynara_finish(p_cynara); //LCOV_EXCL_LINE + return false; //LCOV_EXCL_LINE } if (fclose(fd) != 0) - DNSSD_LOGE("Failed to fclose"); + DNSSD_LOGE("Failed to fclose"); //LCOV_EXCL_LINE snprintf(uid, sizeof(uid), "%d", getuid()); @@ -254,16 +256,16 @@ static gboolean __dnssd_process_result(GIOCondition condition, DNSServiceRef sd_ DNSServiceErrorType err; if (sd_ref == NULL) { - DNSSD_LOGE("DNSServiceRef is NULL"); - return FALSE; + DNSSD_LOGE("DNSServiceRef is NULL"); //LCOV_EXCL_LINE + return FALSE; //LCOV_EXCL_LINE } switch (condition) { case G_IO_IN: err = DNSServiceProcessResult(sd_ref); if (err) { - DNSSD_LOGE("DNSServiceProcessResult returned %d", err); - return FALSE; + DNSSD_LOGE("DNSServiceProcessResult returned %d", err); //LCOV_EXCL_LINE + return FALSE; //LCOV_EXCL_LINE } return TRUE; //LCOV_EXCL_START @@ -284,14 +286,15 @@ static gboolean __dnssd_process_result(GIOCondition condition, DNSServiceRef sd_ return FALSE; } +//LCOV_EXCL_START static gboolean __dnssd_handle_io_events(GIOChannel *source, GIOCondition condition, gpointer data) { dnssd_handle_s *handle = NULL; if (data == NULL) { - DNSSD_LOGE("data is NULL"); - return FALSE; + DNSSD_LOGE("data is NULL"); //LCOV_EXCL_LINE + return FALSE; //LCOV_EXCL_LINE } handle = (dnssd_handle_s*)data; @@ -302,6 +305,7 @@ static gboolean __dnssd_handle_io_events(GIOChannel *source, return __dnssd_process_result(condition, handle->sd_ref); } +//LCOV_EXCL_STOP static gboolean __dnssd_resolve_data_io_events(GIOChannel *source, GIOCondition condition, gpointer data) @@ -356,8 +360,8 @@ static void __dnssd_handle_add_event_handler(guint *watch_id, dnssd_handle_s *ha int dns_sd_fd = DNSServiceRefSockFD(handle->sd_ref); if (dns_sd_fd < 0) { - DNSSD_LOGE("Invalid socket fd"); - return; + DNSSD_LOGE("Invalid socket fd"); //LCOV_EXCL_LINE + return; //LCOV_EXCL_LINE } *watch_id = __dnssd_watch_fd(dns_sd_fd, __dnssd_handle_io_events, handle); @@ -367,8 +371,8 @@ static void __dnssd_resolve_data_add_event_handler(guint *watch_id, resolve_repl { int dns_sd_fd = DNSServiceRefSockFD(data->sd_ref); if (dns_sd_fd < 0) { - DNSSD_LOGE("Invalid socket fd"); - return; + DNSSD_LOGE("Invalid socket fd"); //LCOV_EXCL_LINE + return; //LCOV_EXCL_LINE } *watch_id = __dnssd_watch_fd(dns_sd_fd, __dnssd_resolve_data_io_events, data); @@ -382,10 +386,10 @@ EXPORT_API int dnssd_initialize(void) DNSSD_LOCK; if (__dnssd_is_init() == true) { - DNSSD_LOGE("Already initialized"); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_OPERATION; + DNSSD_LOGE("Already initialized"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_OPERATION; //LCOV_EXCL_LINE } __dnssd_set_initialized(true); @@ -435,7 +439,7 @@ EXPORT_API int dnssd_create_local_service(const char *service_type, local_handle = (dnssd_handle_s *)g_try_malloc0(REG_SIZE); if (local_handle == NULL) { DNSSD_LOGE("Failed to Allocate Memory"); //LCOV_EXCL_LINE - DNSSD_UNLOCK; + DNSSD_UNLOCK; //LCOV_EXCL_LINE __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE return DNSSD_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } @@ -490,9 +494,9 @@ static int __get_valid_registered_handle(dnssd_service_h dnssd_service, dnssd_ha } if (local_handle->op_type != DNSSD_TYPE_REGISTER) { - DNSSD_LOGD("DNSSD service is not a local service"); - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_SERVICE_NOT_FOUND; + DNSSD_LOGD("DNSSD service is not a local service"); //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_SERVICE_NOT_FOUND; //LCOV_EXCL_LINE } *handle = local_handle; @@ -675,11 +679,11 @@ EXPORT_API int dnssd_service_get_interface(dnssd_service_h dnssd_service, char * reg = GET_REG_DATA_P(local_handle); if_index = reg->if_index; } else { - *interface = NULL; - DNSSD_LOGE("Invalid DNS SD service"); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_PARAMETER; + *interface = NULL; //LCOV_EXCL_LINE + DNSSD_LOGE("Invalid DNS SD service"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (if_index > 0) { @@ -694,8 +698,8 @@ EXPORT_API int dnssd_service_get_interface(dnssd_service_h dnssd_service, char * } } else { - DNSSD_LOGD("No Interface (all interfaces is used)"); - *interface = NULL; + DNSSD_LOGD("No Interface (all interfaces is used)"); //LCOV_EXCL_LINE + *interface = NULL; //LCOV_EXCL_LINE } DNSSD_UNLOCK; @@ -732,10 +736,10 @@ EXPORT_API int dnssd_service_add_txt_record(dnssd_service_h local_service, } if (length > 255) { - DNSSD_LOGE("Invalid length"); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_PARAMETER; + DNSSD_LOGE("Invalid length"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (length == 0 || value == NULL) @@ -859,7 +863,7 @@ EXPORT_API int dnssd_service_set_record(dnssd_service_h local_service, unsigned if (ret != kDNSServiceErr_NoError) { DNSSD_LOGE("Failed to Add Record for DNS Service, error[%s]", //LCOV_EXCL_LINE dnssd_error_to_string(ret)); //LCOV_EXCL_LINE - DNSSD_UNLOCK; + DNSSD_UNLOCK; //LCOV_EXCL_LINE __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE return DNSSD_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE } @@ -905,7 +909,7 @@ EXPORT_API int dnssd_service_unset_record(dnssd_service_h local_service, dnssd_err = DNSServiceGetProperty(kDNSServiceProperty_DaemonVersion, &version, &size); if (dnssd_err != kDNSServiceErr_NoError) - DNSSD_LOGD("Daemon is running ver. %d.%d", version / 10000, version / 100 % 100); + DNSSD_LOGD("Daemon is running ver. %d.%d", version / 10000, version / 100 % 100); //LCOV_EXCL_LINE else DNSSD_LOGE("Daemon is not running"); @@ -1031,10 +1035,10 @@ EXPORT_API int dnssd_register_local_service(dnssd_service_h local_service, } if (register_cb == NULL) { - DNSSD_LOGE("No callback provided"); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_PARAMETER; + DNSSD_LOGE("No callback provided"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } reg = GET_REG_DATA_P(local_handle); @@ -1095,10 +1099,10 @@ EXPORT_API int dnssd_deregister_local_service(dnssd_service_h local_service) } if (local_handle->sd_ref == NULL) { - DNSSD_LOGE("Invalid DNS SD Client"); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_PARAMETER; + DNSSD_LOGE("Invalid DNS SD Client"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } reg = GET_REG_DATA_P(local_handle); @@ -1966,11 +1970,11 @@ EXPORT_API int dnssd_service_get_name(dnssd_service_h dnssd_service, char **serv reg = GET_REG_DATA_P(local_handle); *service_name = g_strdup(reg->service_name); } else { - *service_name = NULL; - DNSSD_LOGD("Invalid DNS SD service"); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_PARAMETER; + *service_name = NULL; //LCOV_EXCL_LINE + DNSSD_LOGD("Invalid DNS SD service"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } DNSSD_LOGD("Service Name %s", *service_name); @@ -1980,6 +1984,7 @@ EXPORT_API int dnssd_service_get_name(dnssd_service_h dnssd_service, char **serv return DNSSD_ERROR_NONE; } +//LCOV_EXCL_START static void _extract_ipv4_address(dnssd_found_data_s *found_data, char **ip_v4_address) { if (!found_data) @@ -2024,6 +2029,7 @@ static void _extract_ipv6_address(dnssd_found_data_s *found_data, char **ip_v6_a addr[11], addr[12], addr[13], addr[14], addr[15]); } +//LCOV_EXCL_STOP EXPORT_API int dnssd_service_get_ip(dnssd_service_h dnssd_service, char **ip_v4_address, char **ip_v6_address) @@ -2100,11 +2106,11 @@ EXPORT_API int dnssd_service_get_port(dnssd_service_h dnssd_service, int *port) reg = GET_REG_DATA_P(local_handle); *port = reg->port; } else { - *port = 0; - DNSSD_LOGD("Invalid DNS SD service"); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_PARAMETER; + *port = 0; //LCOV_EXCL_LINE + DNSSD_LOGD("Invalid DNS SD service"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } DNSSD_LOGD("Port %d", *port); @@ -2170,7 +2176,7 @@ EXPORT_API int dnssd_create_remote_service(const char *service_type, remote_handle = (dnssd_handle_s *)g_try_malloc0(FOUND_SIZE); if (remote_handle == NULL) { DNSSD_LOGE("Failed to Allocate Memory"); //LCOV_EXCL_LINE - DNSSD_UNLOCK; + DNSSD_UNLOCK; //LCOV_EXCL_LINE __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE return DNSSD_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } @@ -2188,12 +2194,12 @@ EXPORT_API int dnssd_create_remote_service(const char *service_type, if (interface && strlen(interface) != 0) { unsigned int if_index; if ((if_index = if_nametoindex(interface)) == 0) { - DNSSD_LOGE("Invalid interface name"); - g_free(remote_handle->service_type); - g_free(remote_handle); - DNSSD_UNLOCK; - __DNSSD_LOG_FUNC_EXIT__; - return DNSSD_ERROR_INVALID_PARAMETER; + DNSSD_LOGE("Invalid interface name"); //LCOV_EXCL_LINE + g_free(remote_handle->service_type); //LCOV_EXCL_LINE + g_free(remote_handle); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + __DNSSD_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } found->if_index = if_index; } else { @@ -2231,9 +2237,9 @@ EXPORT_API int dnssd_destroy_remote_service(dnssd_service_h dnssd_service) if (remote_handle->op_type != DNSSD_TYPE_FOUND && remote_handle->op_type != DNSSD_TYPE_FOUND_NOT_RESOLVED) { - DNSSD_LOGE("Invalid handle type"); - DNSSD_UNLOCK; - return DNSSD_ERROR_INVALID_PARAMETER; + DNSSD_LOGE("Invalid handle type"); //LCOV_EXCL_LINE + DNSSD_UNLOCK; //LCOV_EXCL_LINE + return DNSSD_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } DNSSD_LOGD("Destroy handle: [%p]->[%u]", remote_handle, dnssd_service); diff --git a/src/ssdp/ssdp.c b/src/ssdp/ssdp.c index f6fadc1..93c38eb 100644 --- a/src/ssdp/ssdp.c +++ b/src/ssdp/ssdp.c @@ -454,19 +454,19 @@ EXPORT_API int ssdp_initialize() GError *gerror = NULL; if (g_is_gssdp_init) { - SSDP_LOGE("gssdp already initialized"); - SSDP_UNLOCK; - return SSDP_ERROR_NONE; + SSDP_LOGE("gssdp already initialized"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + return SSDP_ERROR_NONE; //LCOV_EXCL_LINE } g_gssdp_client = gssdp_client_new(NULL, NULL, &gerror); if (gerror) { - SSDP_LOGE("Error creating the GSSDP client: %s\n", - gerror->message); + SSDP_LOGE("Error creating the GSSDP client: %s\n", //LCOV_EXCL_LINE + gerror->message); //LCOV_EXCL_LINE g_error_free(gerror); //LCOV_EXCL_LINE - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; - return SSDP_ERROR_OPERATION_FAILED; + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return SSDP_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE } if (g_gssdp_client == NULL) { @@ -525,16 +525,16 @@ EXPORT_API int ssdp_create_local_service(const char *target, ssdp_service_h *ssd SSDP_LOCK; if (!g_is_gssdp_init) { - SSDP_LOGE("gssdp not initialized"); - SSDP_UNLOCK; + SSDP_LOGE("gssdp not initialized"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE return SSDP_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } if (target == NULL) { - SSDP_LOGE("target is NULL"); - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; - return SSDP_ERROR_INVALID_PARAMETER; + SSDP_LOGE("target is NULL"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return SSDP_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (ssdp_service == NULL) { @@ -546,18 +546,18 @@ EXPORT_API int ssdp_create_local_service(const char *target, ssdp_service_h *ssd service = (ssdp_service_s*)g_try_malloc0(sizeof(ssdp_service_s)); if (!service) { - SSDP_LOGE("Failed to get memory for gssdp service structure"); - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; + SSDP_LOGE("Failed to get memory for gssdp service structure"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE return SSDP_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } service->target = g_strndup(target, strlen(target)); if (!service->target) { - SSDP_LOGE("Failed to get memory for gssdp service type"); + SSDP_LOGE("Failed to get memory for gssdp service type"); //LCOV_EXCL_LINE g_free(service); //LCOV_EXCL_LINE - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE return SSDP_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } @@ -590,8 +590,8 @@ EXPORT_API int ssdp_destroy_local_service(ssdp_service_h ssdp_service) SSDP_LOGD("SSDP service ID [%u]", ssdp_service); if (!g_is_gssdp_init) { - SSDP_LOGE("gssdp not initialized"); - SSDP_UNLOCK; + SSDP_LOGE("gssdp not initialized"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE return SSDP_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } @@ -628,16 +628,16 @@ EXPORT_API int ssdp_service_set_usn(ssdp_service_h local_service, const char* us SSDP_LOCK; if (local_service == 0 || usn == NULL) { - SSDP_LOGE("Invalid parameter"); - SSDP_UNLOCK; - return SSDP_ERROR_INVALID_PARAMETER; + SSDP_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + return SSDP_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } SSDP_LOGD("SSDP service ID [%u]", local_service); SSDP_LOGD("USN [%s]", usn); if (!g_is_gssdp_init) { - SSDP_LOGE("gssdp not initialized"); - SSDP_UNLOCK; + SSDP_LOGE("gssdp not initialized"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE return SSDP_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } @@ -649,9 +649,9 @@ EXPORT_API int ssdp_service_set_usn(ssdp_service_h local_service, const char* us } if (service->origin == SSDP_SERVICE_STATE_REGISTERED) { - SSDP_LOGE("Already registered"); - SSDP_UNLOCK; - return SSDP_ERROR_INVALID_PARAMETER; + SSDP_LOGE("Already registered"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + return SSDP_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (__ssdp_find_local_service_with_usn(g_ssdp_local_services, usn) != NULL) { @@ -678,16 +678,16 @@ EXPORT_API int ssdp_service_set_url(ssdp_service_h local_service, const char *ur SSDP_LOCK; if (local_service == 0 || url == NULL) { - SSDP_LOGE("Invalid parameter"); - SSDP_UNLOCK; - return SSDP_ERROR_INVALID_PARAMETER; + SSDP_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + return SSDP_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } SSDP_LOGD("SSDP service ID [%u]", local_service); SSDP_LOGD("Location [%s]", url); if (!g_is_gssdp_init) { - SSDP_LOGE("gssdp not initialized"); - SSDP_UNLOCK; + SSDP_LOGE("gssdp not initialized"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE return SSDP_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } @@ -699,9 +699,9 @@ EXPORT_API int ssdp_service_set_url(ssdp_service_h local_service, const char *ur } if (service->origin == SSDP_SERVICE_STATE_REGISTERED) { - SSDP_LOGE("Already registered"); - SSDP_UNLOCK; - return SSDP_ERROR_INVALID_PARAMETER; + SSDP_LOGE("Already registered"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + return SSDP_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (__ssdp_find_local_service_with_url(g_ssdp_local_services, url) != NULL) { @@ -755,9 +755,9 @@ EXPORT_API int ssdp_service_get_target(ssdp_service_h ssdp_service, char **targe target_local = g_strndup(service->target, strlen(service->target)); if (!target_local) { - SSDP_LOGE("Failed to get memory for gssdp service type"); - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; + SSDP_LOGE("Failed to get memory for gssdp service type"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE return SSDP_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } @@ -785,8 +785,8 @@ EXPORT_API int ssdp_service_get_usn(ssdp_service_h ssdp_service, char **usn) SSDP_LOGD("SSDP service ID [%u]", ssdp_service); if (!g_is_gssdp_init) { - SSDP_LOGE("gssdp not initialized"); - SSDP_UNLOCK; + SSDP_LOGE("gssdp not initialized"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE return SSDP_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } @@ -876,21 +876,21 @@ EXPORT_API int ssdp_register_local_service(ssdp_service_h local_service, SSDP_LOCK; if (__ssdp_check_permission() == false) { - SSDP_LOGE("Permission denied"); - SSDP_UNLOCK; - return SSDP_ERROR_PERMISSION_DENIED; + SSDP_LOGE("Permission denied"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + return SSDP_ERROR_PERMISSION_DENIED; //LCOV_EXCL_LINE } if (local_service == 0) { - SSDP_LOGE("Invalid parameter"); - SSDP_UNLOCK; - return SSDP_ERROR_INVALID_PARAMETER; + SSDP_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + return SSDP_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } SSDP_LOGD("SSDP service ID [%u]", local_service); if (!g_is_gssdp_init) { - SSDP_LOGE("gssdp not initialized"); - SSDP_UNLOCK; + SSDP_LOGE("gssdp not initialized"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE return SSDP_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } @@ -908,11 +908,11 @@ EXPORT_API int ssdp_register_local_service(ssdp_service_h local_service, } if (service->resource_group != NULL) { - SSDP_LOGE("Resource group is not NULL\ninit first"); - gssdp_resource_group_remove_resource(service->resource_group, + SSDP_LOGE("Resource group is not NULL\ninit first"); //LCOV_EXCL_LINE + gssdp_resource_group_remove_resource(service->resource_group, //LCOV_EXCL_LINE service->resource_id); - service->resource_group = NULL; - service->resource_id = 0; + service->resource_group = NULL; //LCOV_EXCL_LINE + service->resource_id = 0; //LCOV_EXCL_LINE } if (g_gssdp_client == NULL) { @@ -1052,19 +1052,19 @@ GLIST_ITER_END() if (browser == NULL) { browser = (ssdp_service_s*)g_try_malloc0(sizeof(ssdp_service_s)); if (!browser) { - SSDP_LOGE("Failed to get memory for gssdp service structure"); - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; - return SSDP_ERROR_OUT_OF_MEMORY; + SSDP_LOGE("Failed to get memory for gssdp service structure"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return SSDP_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } browser->target = g_strndup(target, strlen(target)); if (!browser->target) { - SSDP_LOGE("Failed to get memory for gssdp service type"); - g_free(browser); - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; - return SSDP_ERROR_OUT_OF_MEMORY; + SSDP_LOGE("Failed to get memory for gssdp service type"); //LCOV_EXCL_LINE + g_free(browser); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return SSDP_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } /* Create browser handle */ @@ -1076,10 +1076,10 @@ GLIST_ITER_END() g_gssdp_client, browser->target); if (browser->resource_browser == NULL) { - SSDP_LOGE("Failed to create service browser\n"); - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; - return SSDP_ERROR_OPERATION_FAILED; + SSDP_LOGE("Failed to create service browser\n"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return SSDP_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE } /* Connect signals */ @@ -1096,10 +1096,10 @@ GLIST_ITER_END() g_ssdp_local_services = g_list_append(g_ssdp_local_services, browser); } else { if (!(gssdp_resource_browser_rescan(browser->resource_browser))) { - SSDP_LOGE("Failed to request rescan"); - SSDP_UNLOCK; - __SSDP_LOG_FUNC_EXIT__; - return SSDP_ERROR_OPERATION_FAILED; + SSDP_LOGE("Failed to request rescan"); //LCOV_EXCL_LINE + SSDP_UNLOCK; //LCOV_EXCL_LINE + __SSDP_LOG_FUNC_EXIT__; //LCOV_EXCL_LINE + return SSDP_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE } }