Opened 7 years ago

Closed 7 years ago

#6212 closed defect (fixed)

EOB1: Crash when using Mage Spells such as Magic Missile

Reported by: SF/truus999 Owned by: athrxx
Priority: normal Component: Engine: Kyra
Keywords: Cc:
Game: Eye of the Beholder 1

Description

When my mage fires a magic missile or lightning bolt the geme crashes. Both spells worrked fine before it happemd for the first time on level 11 I think.

Just load the saved game and let the mage (Fon) fire a magic missile.

Ticket imported from: #3596548. Ticket imported from: bugs/6212.

Attachments (1)

eob.107 (24.8 KB ) - added by SF/truus999 7 years ago.
Saved game

Download all attachments as: .zip

Change History (12)

by SF/truus999, 7 years ago

Attachment: eob.107 added

Saved game

comment:1 by SF/truus999, 7 years ago

Summary: Game crashesEOB: Game crashes

comment:2 by SF/truus999, 7 years ago

Summary: EOB: Game crashesEOB1: Game crashes

comment:3 by SF/truus999, 7 years ago

Almost forgot:

ScummVM 1.6.0git2170g89a330c (1 december 2012)
DOS/English

comment:4 by athrxx, 7 years ago

Owner: set to athrxx

comment:5 by digitall, 7 years ago

Summary: EOB1: Game crashesEOB1: Crash when using Mage Spells such as Magic Missile

comment:6 by digitall, 7 years ago

Replicated from the savegame with the latest Git master build. Running with valgrind shows the following backtrace:
==22793== Invalid read of size 1
==22793== at 0x821A446: Kyra::EoBCoreEngine::getMageLevel(int) (eobcommon.cpp
:1094)
==22793== by 0x818CE1E: Kyra::EoBCoreEngine::spellCallback_end_melfsAcidArrow
(void*) (magic_eob.cpp:883)
==22793== by 0x818A168: Kyra::EoBCoreEngine::removeCharacterEffect(int, int,
int) (magic_eob.cpp:249)
==22793== by 0x8189BD4: Kyra::EoBCoreEngine::castSpell(int, int) (magic_eob.c
pp:182)
==22793== by 0x81785BB: Kyra::EoBCoreEngine::clickedSpellbookList(Kyra::Butto
n*) (gui_eob.cpp:1019)
==22793== by 0x81C44B9: Common::Functor1Mem<Kyra::Button*, int, Kyra::EoBCore
Engine>::operator()(Kyra::Button*) const (func.h:450)
==22793== by 0x817AC7C: Kyra::GUI_EoB::processButtonList(Kyra::Button*, unsig
ned short, signed char) (gui_eob.cpp:1879)
==22793== by 0x81F0612: Kyra::KyraEngine_v1::checkInput(Kyra::Button*, bool,
int) (kyra_v1.cpp:343)

comment:7 by digitall, 7 years ago

truus999: A fix for this bug was committed by athrxx as 79eb6c898a704f2e41855893643ff3b3aa0bb287.
Please can you test with the next nightly build and confirm if this is now fixed:
http://buildbot.scummvm.org/builds.html

Thanks.

comment:8 by digitall, 7 years ago

Resolution: fixed
Status: newpending

comment:9 by SF/truus999, 7 years ago

Status: pendingnew

comment:10 by SF/truus999, 7 years ago

Confirmed. No more problems with the latest daily build.

Thanks for fixing.

René

comment:11 by lordhoto, 7 years ago

Status: newclosed
Note: See TracTickets for help on using tickets.