Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Chengyi Zhao <zhaochengyi@uniontech.com>
To:     linux-bluetooth@vger.kernel.org
Cc:     Chengyi <zhaochengyi@uniontech.com>
Subject: [PATCH BlueZ 1/1] avdtp: Free discover when send_request returns error
Date:   Tue, 21 Jun 2022 12:31:33 +0800
Message-Id: <20220621043133.7720-2-zhaochengyi@uniontech.com>
In-Reply-To: <20220621043133.7720-1-zhaochengyi@uniontech.com>
References: <20220621043133.7720-1-zhaochengyi@uniontech.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Feedback-ID: bizesmtp:uniontech.com:qybgforeign:qybgforeign9
Precedence: bulk

From: Chengyi <zhaochengyi@uniontech.com>

When send_request returns an error, session->discover should
be released here, so that the next time the program enters
avdtp_discover, the -EBUSY error will not be returned.
---
 profiles/audio/avdtp.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/profiles/audio/avdtp.c b/profiles/audio/avdtp.c
index bc7afad81..0d2213ea2 100644
--- a/profiles/audio/avdtp.c
+++ b/profiles/audio/avdtp.c
@@ -3429,6 +3429,9 @@ int avdtp_discover(struct avdtp *session, avdtp_discover_cb_t cb,
 	if (err == 0) {
 		session->discover->cb = cb;
 		session->discover->user_data = user_data;
+	} else if (session->discover) {
+		g_free(session->discover);
+		session->discover = NULL;
 	}
 
 	return err;
-- 
2.20.1