Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#3531 closed defect (fixed)

LURE: Saving in a conversation crashes when loading

Reported by: jvprat Owned by: dreammaster
Priority: blocker Component: Engine: Lure
Version: Keywords:
Cc: Game: Lure of the Temptress


This has happened while testing the spanish version of lure with svn head.

I did the attached savegame while talking to Ratpouch. Loading it later shows the conversation dialog without the text content, and you can't do anything until you click on the dialog, which crashes with this error:

scummvm: ../../full-svn/scummvm/trunk/engines/lure/hotspots.cpp:3297: static void Lure::HotspotTickHandlers::talkAnimHandler(Lure::Hotspot&): Assertion `talkDestCharacter != 0' failed.

Ticket imported from: #1857335. Ticket imported from: bugs/3531.

Attachments (1)

lure.001 (20.2 KB ) - added by jvprat 16 years ago.

Download all attachments as: .zip

Change History (7)

by jvprat, 16 years ago

Attachment: lure.001 added

comment:1 by fingolfin, 16 years ago

Owner: set to dreammaster

comment:2 by jvprat, 16 years ago

I can reproduce it on every conversation saving with F5, so maybe disabling this key when talking could be enough, as there's no access to the menus in that situation.

On the other hand, in the middle of a conversation initiated by an NPC or while giving instructions to Ratpouch, you have access to the menus and you can save normally. When you load that savegame the dialog is also empty but it doesn't crash.

To be honest, I don't remember exactly how I did the attached savegame, but I suppose it must be the first case.

comment:3 by sev-, 16 years ago

Priority: normalblocker

comment:4 by sev-, 16 years ago

Raising priority. This is a release-critical bug.

comment:5 by dreammaster, 16 years ago

This is fixed - as with the menu, I now block F5 saving when in a conversation or viewing a remote room. I've left F7/F9 intact though, even though you can't access the menu at these points, and added extra code to the restore/restart process to ensure that the data associated with the conversation gets properly cleared.

comment:6 by dreammaster, 16 years ago

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