Opened 20 months ago

Last modified 13 months ago

#10453 new defect

SCI: Patch file conventions should not be version agnostic

Reported by: ikantspelwurdz Owned by:
Priority: normal Component: Engine: SCI
Keywords: Cc:
Game: Quest for Glory 1


Seen in stable Version 2.0.0 and also in a daily build on 3/22/2018.

Start a new game. Pick the Fighter, walk left one screen, and enter the magic shop. The game crashes. I have tested this with the data files provided by GOG, and also with data files on my Quest for Glory Anthology CD-ROM.

Savegame provided, using the GOG files, but shouldn't really be needed since the crash can be reproduced almost immediately.

Error message:
Assertion failed!

Program: <dir>\scummvm.exe
File: /data/sharedrepo/engines/sci/engine/script.cpp, line 916

Expression: blockSize > 0

Attachments (1)

qfg1.001 (12.8 KB ) - added by ikantspelwurdz 20 months ago.

Download all attachments as: .zip

Change History (8)

by ikantspelwurdz, 20 months ago

Attachment: qfg1.001 added

comment:1 by ikantspelwurdz, 20 months ago

Edit - I just discovered that the crash goes away if I delete this file from the game directory:

The file and a bunch of others with the extensions .HEP and .SCR are dated 8/3/1992. I'm guessing this is part of a later version of the game or patch that helps with the DOS interpreter, but breaks ScummVM. It's included with the GOG release of the game.

comment:2 by ikantspelwurdz, 20 months ago

Owner: set to ikantspelwurdz
Resolution: invalid
Status: newclosed

Ok, I found the problem. This copy of the game had files from a patch intended for QFG1 VGA. Not a ScummVM issue (probably).

comment:3 by bonki, 20 months ago

In your original report you state that this also happens with the files from the QFG Anthology CDROM - is this not the case, then?

Also, if ScummVM properly detects the game as EGA and said files don't belong to the EGA version but the original works anyway with them in place I'd say this is a bug which we probably should fix so I'd rather this stays open until someone from the SCI team adds their 2c.

comment:4 by bonki, 20 months ago

Priority: highnormal
Resolution: invalid
Status: closednew

comment:5 by csnover, 20 months ago

SCI had two naming conventions for patch files, type.123 and 123.TYP. The former was used by early SCI games and the latter used starting around SCI1.1. QFG1EGA uses the former, QFG1VGA uses the latter.

The original QFG1EGA interpreter would just not notice these new style patches at all, but ScummVM loads them unconditionally, which it obviously should not be doing since they are for a totally different of the interpreter. So while the mastering of this CD was poor, this is a bug in ScummVM which needs to be fixed (in DirectoryResourceSource).

comment:6 by digitall, 13 months ago

Owner: ikantspelwurdz removed

comment:7 by bluegr, 13 months ago

Summary: QFG1 EGA: Crash when entering magic shopSCI: Patch file conventions should not be version agnostic
Note: See TracTickets for help on using tickets.