From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Andrei Emeltchenko To: linux-bluetooth@vger.kernel.org Subject: [PATCH 01/17] HDP: Fix checking always constant error code Date: Mon, 16 Jun 2014 10:57:36 +0300 Message-Id: <1402905472-17643-1-git-send-email-Andrei.Emeltchenko.news@gmail.com> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: From: Andrei Emeltchenko sdp_set_access_protos() always returns 0, there is no sense to check for error code. Fixes compile warnings. --- profiles/health/hdp_util.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/profiles/health/hdp_util.c b/profiles/health/hdp_util.c index 58770b5..ff427a6 100644 --- a/profiles/health/hdp_util.c +++ b/profiles/health/hdp_util.c @@ -362,7 +362,7 @@ static gboolean set_sdp_services_uuid(sdp_record_t *record, HdpRole role) static gboolean register_service_protocols(struct hdp_adapter *adapter, sdp_record_t *sdp_record) { - gboolean ret; + gboolean ret = TRUE; uuid_t l2cap_uuid, mcap_c_uuid; sdp_list_t *l2cap_list, *proto_list = NULL, *mcap_list = NULL; sdp_list_t *access_proto_list = NULL; @@ -425,11 +425,7 @@ 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