Bluetooth: CCP: Check conn type before access by index

Ensure that the connection type of the provided bt_conn is
an LE connection.

Signed-off-by: Emil Gydesen <[email protected]>

Author: Thalley

subsys/bluetooth/audio/ccp_call_control_client.c

@@ -55,6 +55,8 @@ static struct bt_ccp_call_control_client clients[CONFIG_BT_MAX_CONN];
 
 static struct bt_ccp_call_control_client *get_client_by_conn(const struct bt_conn *conn)
 {
+	__ASSERT(bt_conn_is_type(conn, BT_CONN_TYPE_LE), "Invalid connection type for %p", conn);
+
 	return &clients[bt_conn_index(conn)];
 }
 
@@ -176,6 +178,11 @@ int bt_ccp_call_control_client_discover(struct bt_conn *conn,
 		return -EINVAL;
 	}
 
+	if (!bt_conn_is_type(conn, BT_CONN_TYPE_LE)) {
+		LOG_DBG("Invalid connection type for %p", conn);
+		return -EINVAL;
+	}
+
 	client = get_client_by_conn(conn);
 	if (atomic_test_and_set_bit(client->flags, CCP_CALL_CONTROL_CLIENT_FLAG_BUSY)) {
 		return -EBUSY;

tests/bluetooth/audio/mocks/src/conn.c

@@ -7,6 +7,7 @@
 #include <stdint.h>
 
 #include <errno.h>
+#include <stdbool.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ -76,3 +77,8 @@ void mock_bt_conn_disconnected(struct bt_conn *conn, uint8_t err)
 		}
 	}
 }
+
+bool bt_conn_is_type(const struct bt_conn *conn, enum bt_conn_type type)
+{
+	return true;
+}