Description: When making async calls, we are getting a memory leak. This change fixes this memory leak.

Maciek · Maciek · commit 9c9b0cf48835 · 2015-04-10T10:23:02.000-07:00
diff --git a/src/methodCallBaton.cpp b/src/methodCallBaton.cpp
@@ -64,6 +64,7 @@ v8::Handle<v8::Value> MethodCallBaton::runSync() {
 // called by NanAsyncWorker. This will be on a worker thread
 void MethodCallBaton::Execute() {
   JNIEnv* env = javaGetEnv(this->m_java->getJvm(), this->m_java->getClassLoader());
+  JavaScope javaScope(env);
   ExecuteInternal(env);
 }
 
@@ -73,6 +74,7 @@ void MethodCallBaton::WorkComplete() {
 
   if(callback) {
     JNIEnv* env = javaGetEnv(this->m_java->getJvm(), this->m_java->getClassLoader());
+    JavaScope javaScope(env);
     v8::Handle<v8::Value> result = resultsToV8(env);
     if (result->IsNativeError()) {
       v8::Handle<v8::Value> argv[] = {
