Opened 22 years ago

Closed 22 years ago

Last modified 3 months ago

#346 closed defect (fixed)

MI1: Crash during cannibal encounter

Reported by: eriktorbjorn Owned by: BenCastricum
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game: Monkey Island 1

Description

This happens with recent CVS snapshots:

When offering the cannibals a gift, if I instead start reading LeChuck's kidnapping letter, ScummVM crashes with the following error message:

Error(25:9:0xB7): Object 907 stopped with active cutscene/override!

More precisely, it appears to come from stopObjectScript(). Could this message safely be downgraded to a warning, or is the interpreter in an unpredictable state?

From what I understand, this is the "VGA CD" versin of the game. (The music is stored as normal audio CD tracks.)

Ticket imported from: #581312. Ticket imported from: bugs/346.

Attachments (1)

monkey1.s01 (40.6 KB ) - added by eriktorbjorn 22 years ago.
Game saved just before the cannibal encounter

Download all attachments as: .zip

Change History (10)

by eriktorbjorn, 22 years ago

Attachment: monkey1.s01 added

Game saved just before the cannibal encounter

comment:1 by eriktorbjorn, 22 years ago

Summary: Crash during cannibal encounterMI1: Crash during cannibal encounter

comment:2 by SF/ender, 22 years ago

the interpreter is in an unpredictable state. i dont anyone knows if this works in the NORMAL monkey1 engine?

comment:3 by eriktorbjorn, 22 years ago

Well, I'll be... I played to that point using the original DOS interpreter, and it crashed with the exact same error message. Obviously the bug is in the game itself, not in ScummVM.

comment:4 by SF/ender, 22 years ago

Owner: set to SF/ender
Resolution: invalid
Status: newclosed

comment:5 by SF/ender, 22 years ago

I thought as much ;) I totally rewrote the override code some months ago,
and I spent a lot of time getting it as 'perfect' as possible.

I think this shows my effort paid off - it even crashes in the same places :>
Closing

comment:6 by BenCastricum, 3 years ago

I opened pull request 3299 which fixes this bug.

comment:7 by BenCastricum, 3 years ago

Owner: changed from SF/ender to BenCastricum
Resolution: invalidfixed

PR#3299 got merged, this bug is fixed.

comment:8 by dwatteau, 3 months ago

In 724ab1a3:

SCUMM: Document the original interpreter behavior for Trac#346 and Trac#10571

Test done by AndywinXp, grazie mille!

comment:9 by dwatteau, 3 months ago

In f039be35:

SCUMM: Document Trac#346 workaround a bit more

One needs to wait a bit before reading the note or the map, for the
script sync issue to show up, and also the bug is also there in the
SCUMMv4 releases, but there it seems to only cause some strange
animation behavior, instead of an error (to be fixed with a new
enhancement at a later date...).

Note: See TracTickets for help on using tickets.