From c5966b25c1f2f5fb22cfaf37c2a85daea1d0f65e Mon Sep 17 00:00:00 2001 From: Marcin Kraglak Date: Wed, 9 Jul 2014 10:53:52 +0200 Subject: [PATCH] android/gatt: Fix sending incorrect frames Don't send pdu if encoding failed. It cause errors when trying send notification larger than MTU. Length passed to g_attrib_send was 0 which caused errors on ATT. --- android/gatt.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/android/gatt.c b/android/gatt.c index 2ee2d055f..cccf23e01 100644 --- a/android/gatt.c +++ b/android/gatt.c @@ -4884,9 +4884,14 @@ static void handle_server_send_indication(const void *buf, uint16_t len) (uint8_t *)cmd->value, cmd->len, pdu, mtu); - g_attrib_send(conn->device->attrib, 0, pdu, length, NULL, NULL, NULL); - - status = HAL_STATUS_SUCCESS; + if (length == 0) { + error("gatt: Failed to encode indication"); + status = HAL_STATUS_FAILED; + } else { + g_attrib_send(conn->device->attrib, 0, pdu, length, NULL, NULL, + NULL); + status = HAL_STATUS_SUCCESS; + } reply: ipc_send_rsp(hal_ipc, HAL_SERVICE_ID_GATT, -- 2.47.3