Skip to content

catch every exception from audio context initialization#2625

Merged
yaRnMcDonuts merged 1 commit intomasterfrom
badaudiocrash
Feb 28, 2026
Merged

catch every exception from audio context initialization#2625
yaRnMcDonuts merged 1 commit intomasterfrom
badaudiocrash

Conversation

@riccardobl
Copy link
Member

@riccardobl riccardobl commented Feb 22, 2026

Sometimes the audio context can throw different exception if the platform does not have an audio device at all,
eg.

java.lang.NullPointerException
	at org.lwjgl.system.Checks.check(Checks.java:188)
Checks.java:188
	at org.lwjgl.openal.ALC10.nalcCreateContext(ALC10.java:102)
ALC10.java:102
	at org.lwjgl.openal.ALC10.alcCreateContext(ALC10.java:113)
ALC10.java:113
	at com.jme3.audio.lwjgl.LwjglALC.createALC(LwjglALC.java:58)
LwjglALC.java:58
	at com.jme3.audio.openal.ALAudioRenderer.initOpenAL(ALAudioRenderer.java:130)
ALAudioRenderer.java:130
	at com.jme3.audio.openal.ALAudioRenderer.initialize(ALAudioRenderer.java:303)
ALAudioRenderer.java:303
	at com.jme3.app.LegacyApplication.initAudio(LegacyApplication.java:307)
LegacyApplication.java:307
	at com.jme3.app.LegacyApplication.initialize(LegacyApplication.java:670)
LegacyApplication.java:670
	at com.jme3.app.SimpleApplication.initialize(SimpleApplication.java:253)
SimpleApplication.java:253
	at com.jme3.system.lwjgl.LwjglWindow.initInThread(LwjglWindow.java:710)
LwjglWindow.java:710
	at com.jme3.system.lwjgl.LwjglWindow.run(LwjglWindow.java:813)
LwjglWindow.java:813
	at java.base/java.lang.Thread.run(Thread.java:1583)

on software rendered fedora.

This is currently causing the engine to crash unless the audio renderer is disabled manually.
This pr changes the code to catch every exception and disable the audio context,

@riccardobl riccardobl changed the title catch every exception from audio context catch every exception from audio context initialization Feb 22, 2026
@yaRnMcDonuts yaRnMcDonuts added this to the v3.10.0 milestone Feb 28, 2026
@yaRnMcDonuts yaRnMcDonuts merged commit 0302fd0 into master Feb 28, 2026
17 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.

3 participants