Opened 2 years ago

Closed 2 years ago

#9958 closed defect (fixed)

MOHAWK: Riven: Prison Island Close Dome Crash

Reported by: dafioram Owned by: bgK
Priority: normal Component: Engine: Mohawk
Keywords: Cc:
Game: Riven

Description

ScummVM: 1.10.0git-3786-g906c6de (July 13 2017) 64bit
Tester OS: Win7-64
Game: Riven DVD, Windows, English, 10th Anniversary/v1.2

Error: Could not find sound 2 in card 31!

Similar to #9928.

  1. Load save game (prison island).
  2. start dome spinning.
  3. go to elevator and slide lever of sound puzzle left to right or play some tunes first before sliding the lever.

This crash does happen (same error) if you have saved catherine (or not), but if you have saved her you must first stop the dome from spinning and then start it again.

Attachments (1)

riven-053.rvn (24.9 KB ) - added by dafioram 2 years ago.
Prison Island, catherine not saved.

Download all attachments as: .zip

Change History (3)

by dafioram, 2 years ago

Attachment: riven-053.rvn added

Prison Island, catherine not saved.

comment:1 by bgK, 2 years ago

Thanks for your report.

This one is a script bug (it does not crash in the original, but ScummVM has extra error checks)
This is part of the script that makes the dome spin up:

playMovieBlocking(3); // Dome closing
playMovie(4);         // Dome spinning up
activatePLST(2);      // Dome closed
playMovieBlocking(4); // Dome spinning up
storeMovieOpcode(1, 0, 0, 40, 2); // Schedule ambient sound change to "dome spinning" after movie 1 finishes blocking playback
playMovie(1);         // Dome spinning

this script needs to be patched to

playMovieBlocking(3); // Dome closing
playMovie(4);         // Dome spinning up
activatePLST(2);      // Dome closed
playMovieBlocking(4); // Dome spinning up
activateSLST(2);      // Ambient sound change to "dome spinning"
playMovie(1);         // Dome spinning

Patching this will also fix the incorrect ambient sound after making the dome spin.

comment:2 by bgK, 2 years ago

Owner: set to bgK
Resolution: fixed
Status: newclosed

ScummVM now applies the aforementioned patch since commit 69dc56a.

Note: See TracTickets for help on using tickets.