Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#11196 closed defect (fixed)

QFG3 putting zebra skins in chest results in crash

Reported by: tomasz89 Owned by: sluicebox
Priority: normal Component: Engine: SCI
Version: Keywords: original
Cc: Game: Quest for Glory 3

Description

I'm overloaded and putting the zebra skins away in the chest in Tarna. I select the zebra skins, hover the cursor over the chest, then hold enter. This causes the debugger to appear.

Doesn't happen when I do the same with the meat rations in Tarna's room chest.

Attachments (1)

qfg3-GOG.012 (33.4 KB ) - added by tomasz89 5 years ago.
Save at the room in question

Download all attachments as: .zip

Change History (6)

by tomasz89, 5 years ago

Attachment: qfg3-GOG.012 added

Save at the room in question

comment:1 by tomasz89, 5 years ago

This is the GOG version of QFG3.

comment:2 by sluicebox, 5 years ago

Thanks for reporting this and attaching the helpful save, I can reproduce this with QFG3 1.1 that comes with the QFG Collection CD. Figuring out that the meat doesn't crash is a big help. I'll take a look soon.

To restate problem for anyone else that wants to take a look at this: holding down the Enter key to put zebra skins in the chest as fast as possible crashes after the third or fourth in IconBar:handleEvent. Meanwhile, doing the same with a different inventory item like the meat works and just keeps putting them in the chest.

comment:3 by sluicebox, 5 years ago

Update: the error occurs when holding down enter when putting your last item of any inventory type. In this save there's only 5 zebra skins so it happens quickly. There's 95 meats so it doesn't happen any time soon, but it would still happen on the last one.

comment:4 by sluicebox, 5 years ago

Keywords: original added
Owner: set to sluicebox
Resolution: fixed
Status: newclosed

Fixed in https://github.com/scummvm/scummvm/commit/039d2bc6fcdd40fded68b5b6b14f9a14977b0c5f

This bug also occurred with the chest in room 430

comment:5 by tomasz89, 5 years ago

Confirmed fixed. Thanks for the speedy response and resolution.

Note: See TracTickets for help on using tickets.