[ scummvm-Bugs-3565505 ] SCI : crash when loading a savegame

[SourceForge.net]

Bugs item #3565505, was opened at 2012-09-07 05:29
Message generated for change (Tracker Item Submitted) made by misterhands
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418820&aid=3565505&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Unknown Crash/Other
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Remy (misterhands)
Assigned to: Nobody/Anonymous (nobody)
Summary: SCI : crash when loading a savegame

Initial Comment:
SCUMMVM 1.6.0git1351-g74d5214 Win64

This occurs in certain SCI games, like QFG1VGA, and LSL2.
This may happen in other SCI games too.

When you load a savegame at a time when you're not supposed to (and loading is disabled in the original interpreter), quite logically, SCUMMVM crashes, with output that looks like this :

For LSL2, when trying to load at the copy protection screen :
	WARNING: Attempt to free Hunk from address 001a:0004: Invalid segment type!
	WARNING: getObject(): Trying to get an invalid object!
	lookupSelector(): Attempt to send to non-object or invalid script. Address was 0017:00c0!

For QFG1VGA, when trying to load at the legal messages at the start of the game :
	WARNING: getObject(): Trying to get an invalid object!
	lookupSelector(): Attempt to send to non-object or invalid script. Address was 002c:00cc!

So the fix would be to either disable loading at these places like the original, 
or make sure that everything is properly initialized so that a savegame can be loaded without crashing.
I think the latter would be better, if that's possible.

Thanks!

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418820&aid=3565505&group_id=37116