[Scummvm-tracker] [ScummVM] #10116: MM C64 DEMO: Attempting to display the original save/load screen causes a crash.

Eugene Sandulenko trac at scummvm.org
Sun Nov 26 23:06:15 CET 2017


#10116: MM C64 DEMO: Attempting to display the original save/load screen causes a
crash.
-----------------------------+----------------------------------
  Reporter:  robertmegone    |      Owner:  sev-
      Type:  defect          |     Status:  new
  Priority:  blocker         |  Component:  Engine: SCUMM
Resolution:                  |   Keywords:  semi-reproducible-ub
      Game:  Maniac Mansion  |
-----------------------------+----------------------------------

Comment (by sev-):

 What made you link that UB to the crash in the first place? Do you have
 any non-theoretic evidence that the UB could, in fact, lead to the
 unexpected or unintended results? This code has been tested on dozens of
 platforms and millions of devices. It still works well, and if it was
 broken, no saves would work in SCUMM engine reliably.

 The demo does not have the relevant code anyway, and it is pointless to
 attempt to run it. Thus, the patch makes sense in any case.

 I cannot explain it to you as you are not a small child, and even if you
 were, it is too long to describe the programming from the scratch.

--
Ticket URL: <https://bugs.scummvm.org/ticket/10116#comment:17>
ScummVM <https://bugs.scummvm.org>
ScummVM


More information about the Scummvm-tracker mailing list