#7179 SQ4 CD DOS crash when selecting 'Catalog' in Hz So Good

[CaptainJei]

ScummVM version: 1.9.0git4196-ga1affaa (Aug 8 2016 20:22:2)
Language: English
Version: CD, DOS
Platform: Win32

When the player goes into Hz So Good and selects the 'Catalog' option, the debugger console opens with the error message, "ERROR: Unknown kDisplay argument (0068:ffff) from method doCatalog::changeState (script 391, localCall 84)!"

This does not happen on the last stable release.

[Filippos Karapetis]

Thanks for your report. Could you please attach a saved game?

[CaptainJei]

Sure. If this is the right file, you should be standing right outside Hz Be Good.

[Omer Mor]

It's reproducable by teleporting to room 390 via the debugger at the start of the game.
Perhaps it's related to the workaround at https://github.com/scummvm/scummvm/blob/master/engines/sci/engine/workarounds.cpp#L476 ?
It was introduced in https://github.com/scummvm/scummvm/commit/8f5d65d0f11289bd4b2595ea4ad08b5d2344b133 and seem to touch the function (localproc_84) that makes the faulting kDisplay call.

[M. Kiewitz]

This is really weird. I added a workaround for this issue already back in 2010 - it is in fact a script bug, but somehow I wrote doCatalog::mode instead of the correct doCatalog::changeState. It's weird that no one reported it before, although we had tons of play throughs since then. I guess maybe selecting "catalog" is not required for game completion? Because I remember playing through the game back then by myself.

Anyway fixed via commit 0f1fd5c9553fa0db01ceb12f7d315cead6ab2a64
Thanks for reporting

EDIT: I assume that we silently ignore such issues in official releases, but the workaround was never applied until now.

[CaptainJei]

I've definitely played it through a few times since 2010 without any problems. You can beat the game without going into Hz So Good and buying the PocketPal adapter though, but then you can't monitor the droids inside Vohaul's fortress or get a full score.

Thanks for the fix!