From: patchwork-bot+bluetooth@kernel.org
To: Pauli Virtanen <pav@iki.fi>
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH BlueZ 1/2] shared/bap: clean up requests for a stream before freeing it
Date: Tue, 16 Apr 2024 15:40:28 +0000 [thread overview]
Message-ID: <171328202895.2661.5369436299401809718.git-patchwork-notify@kernel.org> (raw)
In-Reply-To: <d52ddf4759720a2879677fca0129d3fd1a32dda0.1712951445.git.pav@iki.fi>
Hello:
This series was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:
On Fri, 12 Apr 2024 22:55:55 +0300 you wrote:
> Cancel stream's queued requests before freeing the stream.
>
> As the callbacks may do some cleanup on error, be sure to call them
> before removing the requests.
>
> Fixes:
> =======================================================================
> ERROR: AddressSanitizer: heap-use-after-free on address 0x60d000013430
> READ of size 8 at 0x60d000013430 thread T0
> #0 0x89cb9f in stream_stop_complete src/shared/bap.c:1211
> #1 0x89c997 in bap_req_complete src/shared/bap.c:1192
> #2 0x8a105f in bap_process_queue src/shared/bap.c:1474
> #3 0x93c93f in timeout_callback src/shared/timeout-glib.c:25
> ...
> freed by thread T0 here:
> #1 0x89b744 in bap_stream_free src/shared/bap.c:1105
> #2 0x89bac8 in bap_stream_detach src/shared/bap.c:1122
> #3 0x89dbfc in bap_stream_state_changed src/shared/bap.c:1261
> #4 0x8a2169 in bap_ucast_set_state src/shared/bap.c:1554
> #5 0x89e0d5 in stream_set_state src/shared/bap.c:1291
> #6 0x8a78b6 in bap_ucast_release src/shared/bap.c:1927
> #7 0x8d45bb in bt_bap_stream_release src/shared/bap.c:5516
> #8 0x8ba63f in remove_streams src/shared/bap.c:3538
> #9 0x7f23d0 in queue_foreach src/shared/queue.c:207
> #10 0x8bb875 in bt_bap_remove_pac src/shared/bap.c:3593
> #11 0x47416c in media_endpoint_destroy profiles/audio/media.c:185
> =======================================================================
>
> [...]
Here is the summary with links:
- [BlueZ,1/2] shared/bap: clean up requests for a stream before freeing it
(no matching commit)
- [BlueZ,2/2] bap: cancel stream operation before freeing setup
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=d3a6a6459cbd
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
prev parent reply other threads:[~2024-04-16 15:40 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-12 19:55 [PATCH BlueZ 1/2] shared/bap: clean up requests for a stream before freeing it Pauli Virtanen
2024-04-12 19:55 ` [PATCH BlueZ 2/2] bap: cancel stream operation before freeing setup Pauli Virtanen
2024-04-12 22:04 ` [BlueZ,1/2] shared/bap: clean up requests for a stream before freeing it bluez.test.bot
2024-04-16 15:16 ` [PATCH BlueZ 1/2] " Luiz Augusto von Dentz
2024-04-16 15:40 ` patchwork-bot+bluetooth [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=171328202895.2661.5369436299401809718.git-patchwork-notify@kernel.org \
--to=patchwork-bot+bluetooth@kernel.org \
--cc=linux-bluetooth@vger.kernel.org \
--cc=pav@iki.fi \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.