Opened 12 years ago

Closed 12 years ago

#3532 closed defect (fixed)

HE: Assertion on Restart (Crash)

Reported by: raziel- Owned by: wjp
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Freddi Fish 4

Description

ScummVM 0.11.0svn (Dec 23 2007 20:03:48)
Features compiled in: Vorbis FLAC MP3 zLib MPEG2

I get an
assertion "WIO_INVENTORY == whereisobject(obj)" failed: file engines/scumm/object.cpp", line 181
followed by a crash. This happens everytime i click on
"GO" after the game has ended and folowed by a click on
the nodding green fish to perform an ingame restart.

Freddi Fish 4: The Case of the Hogfish Rustler of Briny Gulch (English/Windows)

Amiga OS4
gcc version 4.0.2 (AmigaOS build 20051012)

Save game attached
(Click on the greenish shark in the middle to go on to
the final and if you reach the outtro click on "GO" and
then on the nodding green fish)

Ticket imported from: #1857697. Ticket imported from: bugs/3532.

Attachments (1)

freddi4.s01 (28.4 KB ) - added by raziel- 12 years ago.
Right before the final part

Download all attachments as: .zip

Change History (7)

by raziel-, 12 years ago

Attachment: freddi4.s01 added

Right before the final part

comment:1 by raziel-, 12 years ago

Not isolated to Freddi 4.
It happens with all Freddi Fish games, Fatty Bears Birthday
Surprise and Spy Fox 1, so i'd say it's an overall bug on
the HE engine (or maybe a not yet implemented feature?)

comment:2 by raziel-, 12 years ago

Summary: HE FREDDI4: Assertion on Restart (Crash)HE: Assertion on Restart (Crash)

comment:3 by SF/mthreepwood, 12 years ago

I'm getting the same here in other SCUMM games (Indy3 and Loom) when restarting via the console.

comment:4 by wjp, 12 years ago

ScummEngine::restart() calls clearOwnerOf(0), which looks wrong since 0 is used as 'empty' in the inventory array. Should the loop in scumm.cpp:2196 start at 1?

comment:5 by SF/mthreepwood, 12 years ago

Yes, starting the loop at 1 fixes it for me.

comment:6 by wjp, 12 years ago

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