Re: [Gnash-dev] Gnash ExternalInterface check leaking memory?

[Rob Savoye]

On 08/05/10 20:58, John Gilmore wrote:
> When I went to patch the select() in
> ExternalInterface::ExternalEventCheck, I noticed that it allocates a
> buffer with "char *buf = new char[bytes+1];", then fills buf from a
> read(), then passes it to parseInvoke, But nobody ever deletes buf.

  Ok, with revno #12365, I disabled the select(), and used a
boost::scoped_array instead, which will go away when the
ExternalEventCheck() returns.

> This is the only call to parseInvoke, so perhaps its argument type
> should be changed.

  It gets converted to a string anyway, as parseInvoke() uses std::
string methods like string::find().

        - rob -