Opened 12 years ago

Closed 12 years ago

Last modified 12 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
Keywords: Cc:
Game: Lure of the Temptress

Description

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 12 years ago.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by jvprat

Attachment: lure.001 added

comment:1 Changed 12 years ago by fingolfin

Owner: set to dreammaster

comment:2 Changed 12 years ago by jvprat

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 Changed 12 years ago by sev-

Priority: normalblocker

comment:4 Changed 12 years ago by sev-

Raising priority. This is a release-critical bug.

comment:5 Changed 12 years ago by dreammaster

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 Changed 12 years ago by dreammaster

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