linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH BlueZ] AVDTP: Fix rejecting AVDTP_START if starting flag is set
@ 2012-06-16  8:47 Luiz Augusto von Dentz
  2012-06-18  8:41 ` Johan Hedberg
  0 siblings, 1 reply; 2+ messages in thread
From: Luiz Augusto von Dentz @ 2012-06-16  8:47 UTC (permalink / raw)
  To: linux-bluetooth

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

This is now handled by checking if the command collided.
---
 audio/avdtp.c |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/audio/avdtp.c b/audio/avdtp.c
index eb56c7c..041abc3 100644
--- a/audio/avdtp.c
+++ b/audio/avdtp.c
@@ -1795,9 +1795,8 @@ static gboolean avdtp_start_cmd(struct avdtp *session, uint8_t transaction,
 
 		stream = sep->stream;
 
-		/* Also reject start cmd if we already initiated start */
-		if (sep->state != AVDTP_STATE_OPEN ||
-						stream->starting == TRUE) {
+		/* Also reject start cmd if state is not open */
+		if (sep->state != AVDTP_STATE_OPEN) {
 			err = AVDTP_BAD_STATE;
 			goto failed;
 		}
-- 
1.7.10.2


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH BlueZ] AVDTP: Fix rejecting AVDTP_START if starting flag is set
  2012-06-16  8:47 [PATCH BlueZ] AVDTP: Fix rejecting AVDTP_START if starting flag is set Luiz Augusto von Dentz
@ 2012-06-18  8:41 ` Johan Hedberg
  0 siblings, 0 replies; 2+ messages in thread
From: Johan Hedberg @ 2012-06-18  8:41 UTC (permalink / raw)
  To: Luiz Augusto von Dentz; +Cc: linux-bluetooth

Hi Luiz,

On Sat, Jun 16, 2012, Luiz Augusto von Dentz wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
> 
> This is now handled by checking if the command collided.
> ---
>  audio/avdtp.c |    5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)

Applied. Thanks.

Johan

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-06-18  8:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-06-16  8:47 [PATCH BlueZ] AVDTP: Fix rejecting AVDTP_START if starting flag is set Luiz Augusto von Dentz
2012-06-18  8:41 ` Johan Hedberg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).