Opened 11 years ago

Closed 11 years ago

#3872 closed defect (fixed)

IHNM: -x option crashes ScummVM, if savegame doesn't exist

Reported by: eriktorbjorn Owned by: bluegr
Priority: normal Component: Engine: SAGA
Keywords: Cc:
Game: I Have No Mouth

Description

Latest 0.12 SVN English version of the game

The -x command-line option crashes ScummVM, if you use it to load a savegame that doesn't exist.

(Strangely enough, Inherit the Earth does not crash. It simply seems to ignore the savegame, and starts the game from the beginning as expected.)

Ticket imported from: #2057194. Ticket imported from: bugs/3872.

Change History (3)

comment:1 by eriktorbjorn, 11 years ago

Even loading an existing savegame with the -x option gives me Valgrind warnings like these. Could it be something about the event list that isn't properly initialised?

==5524== Thread 1: ==5524== Conditional jump or move depends on uninitialised value(s) ==5524== at 0x84B3CAD: Saga::Events::handleOneShot(Saga::Event*) (events.cpp:348) ==5524== by 0x84B4C9C: Saga::Events::handleEvents(long) (events.cpp:76) ==5524== by 0x848C20E: Saga::SagaEngine::go() (saga.cpp:260) ==5524== by 0x8053D99: _ZL7runGamePK14PluginSubclassI10MetaEngineER7OSystemRKN6Common6StringE (main.cpp:214) ==5524== by 0x80546CD: scummvm_main (main.cpp:313) ==5524== by 0x8051741: main (main.cpp:108) ==5524== ==5524== Conditional jump or move depends on uninitialised value(s) ==5524== at 0x84B6FC2: Saga::Gfx::setPaletteColor(int, int, int, int) (gfx.cpp:234) ==5524== by 0x84B3DA3: Saga::Events::handleOneShot(Saga::Event*) (events.cpp:354) ==5524== by 0x84B4C9C: Saga::Events::handleEvents(long) (events.cpp:76) ==5524== by 0x848C20E: Saga::SagaEngine::go() (saga.cpp:260) ==5524== by 0x8053D99: _ZL7runGamePK14PluginSubclassI10MetaEngineER7OSystemRKN6Common6StringE (main.cpp:214) ==5524== by 0x80546CD: scummvm_main (main.cpp:313) ==5524== by 0x8051741: main (main.cpp:108) ==5524== ==5524== Conditional jump or move depends on uninitialised value(s) ==5524== at 0x84B7021: Saga::Gfx::setPaletteColor(int, int, int, int) (gfx.cpp:238) ==5524== by 0x84B3DA3: Saga::Events::handleOneShot(Saga::Event*) (events.cpp:354) ==5524== by 0x84B4C9C: Saga::Events::handleEvents(long) (events.cpp:76) ==5524== by 0x848C20E: Saga::SagaEngine::go() (saga.cpp:260) ==5524== by 0x8053D99: _ZL7runGamePK14PluginSubclassI10MetaEngineER7OSystemRKN6Common6StringE (main.cpp:214) ==5524== by 0x80546CD: scummvm_main (main.cpp:313) ==5524== by 0x8051741: main (main.cpp:108) ==5524== ==5524== Conditional jump or move depends on uninitialised value(s) ==5524== at 0x84B7081: Saga::Gfx::setPaletteColor(int, int, int, int) (gfx.cpp:242) ==5524== by 0x84B3DA3: Saga::Events::handleOneShot(Saga::Event*) (events.cpp:354) ==5524== by 0x84B4C9C: Saga::Events::handleEvents(long) (events.cpp:76) ==5524== by 0x848C20E: Saga::SagaEngine::go() (saga.cpp:260) ==5524== by 0x8053D99: _ZL7runGamePK14PluginSubclassI10MetaEngineER7OSystemRKN6Common6StringE (main.cpp:214) ==5524== by 0x80546CD: scummvm_main (main.cpp:313) ==5524== by 0x8051741: main (main.cpp:108)

comment:2 by bluegr, 11 years ago

Owner: set to bluegr
Resolution: fixed
Status: newclosed

comment:3 by bluegr, 11 years ago

Fixed in the latest SVN version and version 0.12.0

Note: See TracTickets for help on using tickets.