From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> sdp_set_access_protos() always returns 0, there is no sense to check for error code. Fixes compile warnings. --- profiles/health/hdp_util.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/profiles/health/hdp_util.c b/profiles/health/hdp_util.c index 47c464e..75ae439 100644 --- a/profiles/health/hdp_util.c +++ b/profiles/health/hdp_util.c @@ -425,11 +425,8 @@ static gboolean register_service_protocols(struct hdp_adapter *adapter, goto end; } - if (sdp_set_access_protos(sdp_record, access_proto_list) < 0) { - ret = FALSE; - goto end; - } ret = TRUE; + sdp_set_access_protos(sdp_record, access_proto_list); end: if (l2cap_list != NULL) -- 1.8.3.2 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html