Re: [Gnash-commit] /srv/bzr/gnash/trunk r10646: Prevent the plugin from spawning two processes for the same movie.

[Markus Gothe]

nice catch!

//Markus

On 2 Mar 2009, at 20:15, Bastiaan Jacques wrote:

> ------------------------------------------------------------
> revno: 10646
> committer: Bastiaan Jacques <address@hidden>
> branch nick: trunk
> timestamp: Mon 2009-03-02 20:15:25 +0100
> message:
>  Prevent the plugin from spawning two processes for the same movie.
> modified:
>  plugin/plugin.cpp
> === modified file 'plugin/plugin.cpp'
> --- a/plugin/plugin.cpp 2009-02-24 18:20:50 +0000
> +++ b/plugin/plugin.cpp 2009-03-02 19:15:25 +0000
> @@ -618,6 +618,13 @@
> nsPluginInstance::NewStream(NPMIMEType /*type*/, NPStream* stream,
>                             NPBool /*seekable*/, uint16_t* /*stype*/)
> {
> +    if (_childpid) {
> +        // Apparently the child process has already been started  
> for this
> +        // plugin instance. It is puzzling that this method gets  
> called
> +        // again. Starting a new process for the same movie will  
> cause
> +        // problems later, so we'll stop here.
> +        return NPERR_GENERIC_ERROR;
> +    }
>     _swf_url = stream->url;
>
> #if GNASH_PLUGIN_DEBUG > 1
>
> _______________________________________________
> Gnash-commit mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/gnash-commit

PGP.sig
Description: This is a digitally signed message part