search

alderlopez / chromiumembedded

Automatically exported from code.google.com/p/chromiumembedded
0 stars 1 forks source link

Memory leak in bool CefV8ValueCToCpp::ExecuteFunctionWithContext(...) #526

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
There's a memory leak in CEF, revision 465...

bool CefV8ValueCToCpp::ExecuteFunctionWithContext(
    CefRefPtr<CefV8Context> context, CefRefPtr<CefV8Value> object,
    const CefV8ValueList& arguments, CefRefPtr<CefV8Value>& retval,
    CefRefPtr<CefV8Exception>& exception, bool rethrow_exception)

The argument list assigned to some newly allocated memory, but is never 
released:

argumentsList = new cef_v8value_t*[argumentsCount];

The above call should be balanced with a corresponding delete when the argument 
list is no longer needed.

Original issue reported on code.google.com by magreenb...@gmail.com on 14 Feb 2012 at 3:28

GoogleCodeExporter commented 9 years ago 
Fixed in revision 491.

Original comment by magreenb...@gmail.com on 14 Feb 2012 at 5:41

GoogleCodeExporter commented 9 years ago 
Another related leak fixed in revision 497.

Original comment by magreenb...@gmail.com on 15 Feb 2012 at 4:30