#3654 ELVIRA1: Amiga crashes (paula.h:108)

[bloody_noob]

Subject:
Elvira - The Mistress of Dark causes a crash just at the beginning when you enter the castle and the guard (to your left) gets you for a chatter (That is before the Intro where you talk to Elvira for the first time).

Version:
ScummVM 0.11.1 (Mar 12 2008 14:17:57)
scummvm-20080316_unstripped (daily-build)

Language:
German, english

Version:
Amiga ( http://ubuntuusers.de/paste/113675/#code-l24 )

System:
Ubuntu 7.10 Gutsy Gibbon (self-compiled with GCC)
WindowsXPsp2 (pre-compiled)

Error:
File 333.out not found
File 393.out not found
File 723.out not found
File 383.out not found
(These files do not exist on my original german copies)

./sound/mods/paula.h:108: void Audio::Paula::setChannelData(uint8, const int8*, const int8*, uint32, uint32, int32): Assertion `0 <= offset && offset < 32768' failed.

Code near line 108 in paula.h:
00106 // For now, we only support 32k samples, as we use 16bit fixed point arithmetics.
00107 // If this ever turns out to be a problem, we can still enhance this code.
00108 assert(0 <= offset && offset < 32768);

[Max Horn]

Logged In: YES
user_id=12935
Originator: NO

My guess would be that the missing files are harmless/normal. The real issue is that it crashes in the Paula sound chip emulation code, which is used for MOD playback.

Kirben, any ideas?

[bloody_noob]

Logged In: YES
user_id=2037966
Originator: YES

Iam not sure it may help but Elvira2 and Waxworks (Amiga) seem to work properly. I didn`t test it very far but at least I was able to walk around and do some fighting without a crash.

[kirben]

Module

[kirben]

Logged In: YES
user_id=34715
Originator: NO

Those file not found messages can be ignored, as not every zone uses sound files (*3.out).

That assert is triggered by an offset of 1985310, but I'm not sure what it causing that exact value.

I have attached the module, in case it can be used for testing. The module is packed, and can be manually unpacked by using extract_agos.
File Added: 8tune

[Gregory Montoir]

[Gregory Montoir]

Logged In: YES
user_id=230314
Originator: NO

possible fix attached (Paula::setChannelData doesn't take a frac_t as its last arg).
File Added: 1915657.diff

[kirben]

Logged In: YES
user_id=34715
Originator: NO

Thanks, the patch solved the problem, and I committed patch to ScummVM 0.12.0svn.