0002860: Crash when choosing cubic or spline sound resampler

Notes
(0015770) Klofkac (reporter) 2016-10-06 07:42	Additional research showed that libSDL is not the culprit. From the crashlog I found the thread which caught the signal 11 was thread that just ran function FMOD_Resampler_Cubic().

(0015771) Edward-san (developer) 2016-10-06 09:42 edited on: 2016-10-06 09:43	It's indeed FMOD specific. It was reported in zdoom forum here and fixed and backported to zandronum-zdoom-sync here, but there's a problem: the fix is done for fmod 4.26, which is more recent than 4.24.16, actually used by zandronum. I wonder if it could be better to update our fmod version to 4.26.36...

(0015851) Torr Samaho (administrator) 2016-10-09 18:57	The "fix" just removes the crashing resamplers when FMOD 4.26 is used. We can easily extend the removal to 4.24.

(0016248) Edward-san (developer) 2016-11-21 14:16	Finally I extended the fix to 4.24 (zdoom pull request here, but I did it too late because of the 'FMOD_STUDIO' macro condition present in the actual zdoom code. Would it be possible to exceptionally backport the fix to 3.0?

(0016283) Torr Samaho (administrator) 2016-11-22 07:04	We'd just need to drop "FMOD_STUDIO \|\| ", right? Then it's surely simply enough to backport it.

(0016312) Edward-san (developer) 2016-11-24 10:27	(G)ZDoom added it, so it's possible to backport it now.

(0016330) Torr Samaho (administrator) 2016-11-26 18:06	Backported.

(0016820) Decay (reporter) 2017-02-07 23:37	I asked Eric to test this in the latest build on his linux machine <Eric>okay, got zandronum installed and running, although opengl isn't working very well <Decay>cool <Decay>http://zandronum.com/tracker/view.php?id=2860#bugnotes simple ticket <Eric>neither option crashes in the latest alpha <Eric>i'm not really sure what i'm supposed to be listening for to see if they are working, but the game doesn't crash when you switch to them and restart sound

**Issue Community Support**
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why.
Supporters:	No one explicitly supports this issue yet.
Opponents:	No one explicitly opposes this issue yet.

Issue Community Support

This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.

Supporters:

No one explicitly supports this issue yet.

Opponents:

No one explicitly opposes this issue yet.

Issue History
Date Modified	Username	Field	Change
2016-10-06 07:36	Klofkac	New Issue
2016-10-06 07:36	Klofkac	File Added: zandronum-crash.7z
2016-10-06 07:42	Klofkac	Note Added: 0015770
2016-10-06 09:42	Edward-san	Note Added: 0015771
2016-10-06 09:42	Edward-san	Assigned To	=> Torr Samaho
2016-10-06 09:42	Edward-san	Status	new => assigned
2016-10-06 09:43	Edward-san	Note Edited: 0015771	View Revisions
2016-10-09 18:57	Torr Samaho	Note Added: 0015851
2016-10-09 18:59	Torr Samaho	Target Version	=> 3.0
2016-11-21 14:16	Edward-san	Note Added: 0016248
2016-11-22 07:04	Torr Samaho	Note Added: 0016283
2016-11-24 10:27	Edward-san	Note Added: 0016312
2016-11-26 18:06	Torr Samaho	Note Added: 0016330
2016-11-26 18:06	Torr Samaho	Status	assigned => needs testing
2017-02-07 20:19	Decay	Note Added: 0016818
2017-02-07 23:14	Decay	Note Deleted: 0016818
2017-02-07 23:37	Decay	Note Added: 0016820
2017-02-07 23:40	Ru5tK1ng	Status	needs testing => resolved
2017-02-07 23:40	Ru5tK1ng	Resolution	open => fixed
2017-02-07 23:40	Ru5tK1ng	Fixed in Version	=> 3.0
2018-09-30 21:48	Blzut3	Status	resolved => closed

Relationships