Re: [Gnash-commit] [SCM] Gnash branch, master, updated. release_0_8_9_final-2042-g711f2a3

[Sandro Santilli]

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;