Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3338 closed defect (fixed)

COMI: iMUSE-related crash on startup

Reported by: salty-horse Owned by: bluegr
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Monkey Island 3

Description

Using latest svn.

Starting comi shows the difficulty selection screen and immediately crashes with:

scummvm: engines/scumm/imuse_digi/dimuse_sndmgr.cpp:622: int32 Scumm::ImuseDigiSndMgr::getDataFromRegion(Scumm::ImuseDigiSndMgr::SoundDesc*, int, byte**, int32, int32): Assertion `len' failed.

Backtrace:

#3 0x00002b95f92dfa16 in Scumm::ImuseDigiSndMgr::getDataFromRegion (this=0x8927b0, soundDesc=0x8927b0, region=0, buf=0x407fffa0, offset=0, size=17640)
at engines/scumm/imuse_digi/dimuse_sndmgr.cpp:622
#4 0x00002b95f92d8f3e in Scumm::IMuseDigital::callback (this=0x7b4c90) at engines/scumm/imuse_digi/dimuse.cpp:325
#5 0x00002b95f92d91d3 in Scumm::IMuseDigital::timer_handler (refCon=0x7b4c90) at engines/scumm/imuse_digi/dimuse.cpp:47
#6 0x000000000052c05b in DefaultTimerManager::handler (this=0x85aaa0) at backends/timer/default/default-timer.cpp:110
#7 0x00000000004547b6 in timer_handler (interval=10, param=0x85aaa0) at backends/platform/sdl/sdl.cpp:51

Ticket imported from: #1761530. Ticket imported from: bugs/3338.

Change History (3)

comment:1 by bluegr, 12 years ago

I've submitted a fix for this issue. Can someone please review it and check if anything else is needed? I'm not sure if the assert on line 631 is needed, but then again, at that point len should be > 0 (since all possible compression types have been checked)

comment:2 by bluegr, 12 years ago

Kirben corrected the code at that spot as well with commit #28227

Bug is fixed now, closing

comment:3 by bluegr, 12 years ago

Owner: set to bluegr
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.