Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#2754 closed defect (worksforme)

WINCE: FT crash (due to lack of memory?)

Reported by: SF/vlodeck Owned by: SF/knakos
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Full Throttle

Description

Error occures while playing first intro:

(2:61:0x9D): Assertion failed_bufferStart != NULL
(..\..\..\sound\audiostream.cpp, 264)

the game stops...

Ticket imported from: #1527443. Ticket imported from: bugs/2754.

Change History (10)

comment:1 Changed 13 years ago by SF/vlodeck

Priority: normalblocker
Summary: WinCE Smartphone WM2003SE, ScummVM 0.9.0aWinCE WM2003SE, Full Throttle, ScummVM 0.9.0a

comment:2 Changed 13 years ago by eriktorbjorn

That would be either the SMUSH player or Digital iMUSE, I
guess. That's the only thing I know of in the SCUMM engine
that uses the appendable memory stream class. The FT intro
probably uses a mixture of both.

That part of the code looks like this:

_bufferStart = (byte *)malloc(bufferSize);
assert(_bufferStart != NULL);

So apparently the malloc() failed. In the case of the SMUSH
player, I believe bufferSize is always 900,000 bytes. In the
case of Digital iMUSE, the size is calculated so I'm not
sure how large it usually is.

comment:3 Changed 13 years ago by SF/vlodeck

so is there anything that can be done?

comment:4 Changed 13 years ago by lordhoto

Priority: blockernormal

comment:5 Changed 13 years ago by SF/knakos

Owner: set to SF/knakos

comment:6 Changed 13 years ago by fingolfin

Summary: WinCE WM2003SE, Full Throttle, ScummVM 0.9.0aWINCE: FT crash (due to lack of memory?)

comment:7 Changed 13 years ago by fingolfin

Probably not much that can be done at this time. Maybe a memory upgrade
would help, knakos?

Other than that, the only fix I can think of would be to rewrite the SMUSH /
iMUSE digital code, but doing that, and doing it "properly" is a fairl huge task,
and since the current code works "well enough", it's a task that not many people
(esp. devs) are much interested in, I am afraid...

comment:8 Changed 13 years ago by SF/knakos

I haven't got around to tracing this one down. After
eriktorbjorn's comment it seems like low mem, but i'd like
to verify it once, to be sure.

comment:9 Changed 13 years ago by SF/knakos

Works OK in 0.9.1, please free up some mem.

comment:10 Changed 13 years ago by SF/knakos

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