Skip to content

Fix audio issues with multiple users in the channel (#3251)#3256

Merged
quinchs merged 1 commit intodiscord-net:devfrom
Archivelit:fix/dave-multi-audio-session-users
Mar 22, 2026
Merged

Fix audio issues with multiple users in the channel (#3251)#3256
quinchs merged 1 commit intodiscord-net:devfrom
Archivelit:fix/dave-multi-audio-session-users

Conversation

@Archivelit
Copy link
Copy Markdown
Contributor

@Archivelit Archivelit commented Mar 22, 2026

Description

Current library crashes with NullReferenceException when processing a commit with an empty payload.

When commit fails it goes to the it allocates empty DaveAllocatedSpan and calls keyPackage.ToMemory() what led to NullReferenceException after several calls.

This patch adds check for empty payload and process it without commiting changes, preventing the crash.

Changes

Related Issues

Tests

Changes was tested locally via my own bot. It worked reliably. Changes from #3253 did not help me to fix the issue with multiple voice users

Add check for empty payload before commit
Incorporated patch from #3523 for multi-user voice session
@Misha-133 Misha-133 requested a review from quinchs March 22, 2026 13:16
@quinchs quinchs merged commit 6527e71 into discord-net:dev Mar 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Dave audio issues with multiple users in the channel

2 participants