[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gnash-commit] [SCM] Gnash branch, master, updated. release_0_8_9_fi
From: |
Sandro Santilli |
Subject: |
Re: [Gnash-commit] [SCM] Gnash branch, master, updated. release_0_8_9_final-2042-g711f2a3 |
Date: |
Thu, 22 May 2014 11:23:17 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, May 22, 2014 at 09:59:16AM +0200, Bastiaan Jacques wrote:
>
>
> On Thu, 22 May 2014, Sandro Santilli wrote:
>
> >IIRC, the problem was with quitting gnash right after start, but
> >before the loading thread had a chance to start.
> >The result was that the main thread could not "join" the thread
> >because the _thread variable was still unassigned, but while quitting
> >the thread would start and try to access now-deleted memory, resulting
> >in a crash.
>
> At present, the function that assigns _thread holds a lock, and so does
> the function called from the thread which accesses _thread. So it is not
> possible even after this change that that _thread is accessed from the
> other thread before it is assigned.
The problem was not access to the _thread variable, but to the
movie definition whose reference was passed to the thread starter.
--strk;