Opened 16 years ago

Closed 16 years ago

Last modified 8 months ago

#8199 closed patch (wontfix)

SKY: Possible fix for crash when changing scaler

Reported by: eriktorbjorn Owned by: SF/khalek
Priority: normal Component: Engine: Sky
Keywords: Cc:
Game: Beneath a Steel Sky

Description

ScummVM sometimes crashes if you change scaler during
the CD intro. I think that is because update_screen()
is being called from two different threads, without any
safety net against races. This patch attempts to fix
that, but I'm a bit uncertain about exactly which
functions need to be guarded so it may be slightly
wrong and/or incomplete.

Ticket imported from: #726713. Ticket imported from: patches/304.

Attachments (1)

sky-mutex.diff (2.6 KB) - added by eriktorbjorn 16 years ago.
Patch against an April 23 CVS snapshot

Download all attachments as: .zip

Change History (6)

Changed 16 years ago by eriktorbjorn

Attachment: sky-mutex.diff added

Patch against an April 23 CVS snapshot

comment:1 Changed 16 years ago by fingolfin

Hm, I'll have a look at it later today. But it might be a better appproach to
put the locking code into the SDL backend itself. Or not. Really need to try.

comment:2 Changed 16 years ago by fingolfin

I put mutex code into the SDL backend, which should make this patch
obsolete. Note that other backends (morphos, palmos, wince,...) may
have to be enhanced to support multiple threads properly, too.

comment:3 Changed 16 years ago by SF/khalek

scaler changing works fine here with cvs now, closing as I
don't see a reason for this to stay open

comment:4 Changed 16 years ago by SF/khalek

Owner: set to SF/khalek
Resolution: wontfix
Status: newclosed

comment:5 Changed 8 months ago by digitall

Component: Engine: Sky
Game: Beneath a Steel Sky
Note: See TracTickets for help on using tickets.