[Scummvm-tracker] [ScummVM :: Bugs] #14546: SCI: Crash in Island of Dr. Brain microscope puzzle

[ScummVM :: Bugs]

#14546: SCI: Crash in Island of Dr. Brain microscope puzzle
-------------------------+----------------------------------
Reporter:  eriktorbjorn  |       Owner:  deckarep
    Type:  defect        |      Status:  closed
Priority:  normal        |   Component:  Engine: SCI
 Version:                |  Resolution:  fixed
Keywords:                |        Game:  Island of Dr. Brain
-------------------------+----------------------------------
Comment (by sluicebox):

 I'm late to the party on this one, but I'd like to clarify some things:

 1. deckarep's workaround is correct. I verified the initial value of the
 uninitialized variable in the original interpreter, and it is indeed 1.
 The workaround gets us the original behavior that avoids the divide by
 zero.

 2. eriktorbjorn got two different crashes but they were really the same
 thing. The difference is that on dev builds we call `error` on
 uninitialized reads and on release builds we call `warning` and use zero
 and hope for the best; zero is usually the right answer and that lets
 players proceed with the game. In this case that didn't work, and led to
 the divide by zero. The phone was running a release build so it got the
 divide by zero error instead of the uninitialized read error.
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/14546#comment:4>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM