From 2f1ea3485bb802a8ffa5f3d3a16d8f34dbd11f42 Mon Sep 17 00:00:00 2001 From: Andrei Emeltchenko Date: Fri, 4 Jul 2014 16:43:56 +0300 Subject: [PATCH] android/gatt: Add missing error check Fixes clang warning: ... android/gatt.c:2097:4: warning: Value stored to 'srvc_search_success' is never read srvc_search_success = search_dev_for_srvc(conn, NULL); ... --- android/gatt.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/android/gatt.c b/android/gatt.c index 3019cd567..04972a4e3 100644 --- a/android/gatt.c +++ b/android/gatt.c @@ -2093,9 +2093,13 @@ static void handle_client_search_service(const void *buf, uint16_t len) } } else { /* Refresh service cache if only partial search was performed */ - if (conn->device->partial_srvc_search) + if (conn->device->partial_srvc_search) { srvc_search_success = search_dev_for_srvc(conn, NULL); - else + if (!srvc_search_success) { + status = HAL_STATUS_FAILED; + goto reply; + } + } else queue_foreach(conn->device->services, send_client_primary_notify, INT_TO_PTR(cmd->conn_id)); -- 2.47.3