commit a41b4570d6f22636ec65bbb94907bf5861765673
author Szymon Janc <szymon.janc@codecoup.pl> Fri Mar 15 10:44:50 2024 +0100
committer Szymon Janc <szymon.janc@codecoup.pl> Thu Mar 21 08:52:42 2024 +0100
tree cc9b9ab70ef7131b94600ac9cdb7633aa9fd7f8f
parent 3d6bfda185a6574c19652a3ab0ca998abdfaa85f

nimble/host: Fix possible memory leak in ble_eatt_prepare_rx_sdu

ble_l2cap_recv_ready consumes om buffer only on success.

nimble/host/src/ble_eatt.c

diff --git a/nimble/host/src/ble_eatt.c b/nimble/host/src/ble_eatt.c
index 721c946..2ef62a8 100644
--- a/nimble/host/src/ble_eatt.c
+++ b/nimble/host/src/ble_eatt.c
@@ -148,6 +148,12 @@
     }
 
     rc = ble_l2cap_recv_ready(chan, om);
+    if (rc) {
+        BLE_EATT_LOG_ERROR("eatt: Failed to supply RX SDU conn_handle 0x%04x (status=%d)\n",
+                            chan->conn_handle, rc);
+        os_mbuf_free_chain(om);
+    }
+
     return rc;
 }