Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-1650-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
To: linux-bluetooth@vger.kernel.org
Subject: [PATCH BlueZ v3 3/3] shared/bap: Make bt_bap_ref_safe check if the instance is attached
Date: Tue,  6 Feb 2024 17:35:13 -0500
Message-ID: <20240206223513.2783330-3-luiz.dentz@gmail.com>
In-Reply-To: <20240206223513.2783330-1-luiz.dentz@gmail.com>
References: <20240206223513.2783330-1-luiz.dentz@gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

This makes bt_bap_ref_safe check that bap instance is part of sessions
list which means it is attached/valid.
---
 src/shared/bap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/shared/bap.c b/src/shared/bap.c
index 4722ed1a1b27..804352db9c41 100644
--- a/src/shared/bap.c
+++ b/src/shared/bap.c
@@ -1033,7 +1033,7 @@ static void stream_notify_metadata(struct bt_bap_stream *stream)
 
 static struct bt_bap *bt_bap_ref_safe(struct bt_bap *bap)
 {
-	if (!bap || !bap->ref_count)
+	if (!bap || !bap->ref_count || !queue_find(sessions, NULL, bap))
 		return NULL;
 
 	return bt_bap_ref(bap);
-- 
2.43.0