Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GameINI: fix Summoner: Goddess Reborn sound system crash during transitions #13250

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

Charlese2
Copy link
Contributor

@Charlese2 Charlese2 commented Dec 28, 2024

Summoner: Goddess Reborn will reinitialize the sound system buffers during transitions while sounds are still being played. There are still pointers in the sound list to sounds in those buffers when the buffers get cleared, so the game crashes with a null pointer dereference if an Audio Interface DMA interrupt is triggered while in that state.

This patch will clean up the pending sound list during sound system reinitialization before continuing to the sound system initialization call previously at 0x8017E338 (On GS2E78).

I would need to work with other people to patch the other regions of the game. If they need patching that is.

@JMC47
Copy link
Contributor

JMC47 commented Dec 29, 2024

@dolphin-emu-bot rebuild

@JMC47
Copy link
Contributor

JMC47 commented Jan 17, 2025

For anyone looking at this in the future. The issue is both a game bug that can happen on console, but happens much more often in Dolphin due to timings not being the same as console.

@JMC47 JMC47 merged commit f65465b into dolphin-emu:master Jan 17, 2025
13 checks passed
@Charlese2 Charlese2 deleted the fix-sound-system-crash branch January 22, 2025 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants