Opened 20 years ago

Closed 20 years ago

Last modified 20 years ago

#1727 closed defect (fixed)

FOTAQ: NULL pointer dereference in Talk::defaultAnimation

Reported by: SF/glynnc Owned by: cyxx
Priority: normal Component: Engine: Queen
Version: Keywords:
Cc: Game: Flight of the Amazon Queen

Description

The debug() call at the top of Talk::defaultAnimation (queen/talk.cpp, line 736) dereferences the "parameters" argument without checking whether it is NULL. This causes scummvm to die on SIGSEGV e.g. when talking to Faye immediately after the plane crashes.

SCUMMVM version: CVS head as of 2004-08-16

$ scummvm -v ScummVM 0.7.0CVS (Aug 16 2004 21:20:20)

Game version: FOTAQ_Floppy, downloaded via: http://prdownloads.sourceforge.net/scummvm/FOTAQ_Floppy.zip?download

$ md5sum FOTAQ_Floppy.zip eafb6bc4edc6255b871477cfc54312de FOTAQ_Floppy.zip

Ticket imported from: #1010929. Ticket imported from: bugs/1727.

Attachments (1)

queen.s05 (12.3 KB ) - added by SF/glynnc 20 years ago.

Download all attachments as: .zip

Change History (5)

by SF/glynnc, 20 years ago

Attachment: queen.s05 added

comment:1 by cyxx, 20 years ago

Oops... that debug() call should have been gone with my recent "cleanup commits"...

comment:2 by cyxx, 20 years ago

Owner: set to cyxx
Status: newclosed

comment:3 by cyxx, 20 years ago

Resolution: fixed

comment:4 by cyxx, 20 years ago

Summary: NULL pointer dereference in Talk::defaultAnimationFOTAQ: NULL pointer dereference in Talk::defaultAnimation
Note: See TracTickets for help on using tickets.