Opened 5 months ago
Closed 2 months ago
#14546 closed defect (fixed)
SCI: Crash in Island of Dr. Brain microscope puzzle
Reported by: | eriktorbjorn | Owned by: | deckarep |
---|---|---|---|
Priority: | normal | Component: | Engine: SCI |
Version: | Keywords: | ||
Cc: | Game: | Island of Dr. Brain |
Description (last modified by )
Latest ScummVM snapshot
English "SierraClassics" release of the game
There are at least two ways the microscope puzzle in Island of Dr. Brain can crash when asking for a hint, though I've only been able to reproduce one of them.
I'm attaching two savegames. One before even starting the microscope puzzle (since the puzzle you get is random), and one where opening the puzzle and pressing the hint button will always crash with the following message:
ERROR: Uninitialized read for temp 0 from method cartesian::buyClue (room 160, script 165, localCall ffffffff)!
The other hint button crash had this message, but I haven't been able to reproduce it:
ERROR: Invalid arithmetic operation (division - params: 0000:fffd and 0000:0000) from method carthesian::buyClue (room 160, script 165, localCall ffffffff)!
For the second one, I was playing on my phone so that was using ScummVM 2.7.
Annoyingly, since I filed this bug report I have not been able to come up with any new test cases, so this bug seems to be a slippery thing.
Attachments (2)
Change History (5)
by , 5 months ago
Attachment: | islandbrain.010 added |
---|
by , 5 months ago
Attachment: | islandbrain.012 added |
---|
comment:1 by , 5 months ago
Description: | modified (diff) |
---|
comment:2 by , 2 months ago
comment:3 by , 2 months ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
The PR has been merged, so this should be fixed now. Thanks for your work on this!
I spent some time trying to understand this crash and I think I've come to a possible solution. Thanks @eriktorbjorn for providing an attached file that helped me put together a patch.
Of course it needs careful review by the ScummVM team: https://github.com/scummvm/scummvm/pull/5342