#14498 closed defect (fixed)

SCUMM: INDY3 (MAC): copyRectToSurface assert fails when loading a save

Reported by: dwatteau Owned by: AndywinXp
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game: Indiana Jones 3

Description

Trying to load my Indy3 EGA Macintosh saves from last year, the following assert appears to always be triggered:

Assertion failed: (destX >= 0 && destX < w), function copyRectToSurface, file surface.cpp, line 169.

I'm loading the saves from the GMM, I'm using original_gui=true and it's a debug build. The saves do work with the same setup in ScummVM 2.7.0.

Git bisect tells me that this is related to commit dd1232325d9b310a85244f78ca5d12c063cf047f (GRAPHICS: Add support for cursor as Surface).

Attaching the save and the backtrace below.

Attachments (2)

scummvm-assert-indy3-ega-macintosh-14498.txt (2.5 KB ) - added by dwatteau 18 months ago.
LLDB backtrace when the assert is triggered
indy3-ega-mac.s26 (12.0 KB ) - added by dwatteau 18 months ago.
Example of an Indy3 EGA Macintosh save triggering the assert in current Git master

Download all attachments as: .zip

Change History (5)

by dwatteau, 18 months ago

LLDB backtrace when the assert is triggered

by dwatteau, 18 months ago

Attachment: indy3-ega-mac.s26 added

Example of an Indy3 EGA Macintosh save triggering the assert in current Git master

comment:1 by dwatteau, 18 months ago

Possibly related to issue #14468, in some way?

comment:2 by AndywinXp, 18 months ago

Owner: set to AndywinXp
Resolution: fixed
Status: newpending

Thanks! Should be fixed now :-)

comment:3 by dwatteau, 18 months ago

Status: pendingclosed

It's all good! My 40 saves for this game now all load without any issue.

Thank you very much for the very quick fix!

Note: See TracTickets for help on using tickets.