Bug 2747 - OpenAL causes crash on MacOS X
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Sound
Version: 1.33 SVN
Hardware: Macintosh All
: P2 major
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2006-06-15 20:57 EDT by Tony J. White
Modified: 2007-05-21 11:26:49 EDT
0 users

See Also:

Attachments
check value from qalSourceUnqueueBuffers() (376 bytes, patch)
2006-06-15 20:58 EDT, Tony J. White

Description Tony J. White 2006-06-15 20:57:23 EDT 
As reported on the mailing list, the OpenAL music processing causes under
certain circumstances on MacOS X ppc.

It happens when a call to qalBufferData() fails causesing qalGetError() to
return AL_INVALID_OPERATION, yet still adding NULL to the queue.  The next
time qalSourceUnqueueBuffer() is called it pulls the NULL value and tries
to process it  causing the crash.

The real fix would be to find out exactly why qalBufferData() is failing, but
this patch provides a bandaid fix to check that the unqueued value is not
NULL before processing it.
Comment 1 Tony J. White 2006-06-15 20:58:14 EDT 
Created attachment 937 [details]
check value from qalSourceUnqueueBuffers()
Comment 2 Tony J. White 2006-08-02 02:13:39 EDT 
Just an update in case anyone else is working on this, it happens on both intel and ppc and seems to be related to a the background music loop when there are a lot of other game sounds being processed.

The attached patch is worthless.
Comment 3 Tony J. White 2006-08-02 22:36:09 EDT 
This should no longer cause crashing after revision 832.

However, it will still print out a big red ERROR: whenever qalBufferData() fails on processing background music.  This seems to happen at seemingly random times with Apple's OpenAL implementation.  It's starting to look like this bug might belong to Apple's OpenAL.
Comment 4 Tony J. White 2006-08-02 23:18:05 EDT 
http://opensource.creative.com/pipermail/openal/2005-October/008893.html

Changed MAX_SRC to 64 on Mac in revision 833.

This seems to clear up the problem.
Comment 5 Ryan C. Gordon 2007-05-21 11:26:49 EDT 
Setting a QA contact on all ioquake3 bugs, even resolved ones. Sorry if you get a flood of email from this, it should only happen once. Apologies for the incovenience.

--ryan.