Merge pull request #443 from mikemorris/master

EscapableHandleScope in Nan::NewBuffer for Node 0.8 and 0.10
nodejs · Sep 8, 2015 · a71301e · a71301e
2 parents 7b2ec2b + b1654d7
commit a71301e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/nan.h b/nan.h
@@ -1026,12 +1026,12 @@ class Utf8String {
     , node::Buffer::free_callback callback
     , void *hint
   ) {
-    HandleScope scope;
+    EscapableHandleScope scope;
     // arbitrary buffer lengths requires
     // NODE_MODULE_VERSION >= IOJS_3_0_MODULE_VERSION
     assert(length <= imp::kMaxLength && "too large buffer");
     return MaybeLocal<v8::Object>(
-        New(node::Buffer::New(data, length, callback, hint)->handle_));
+        scope.Escape(New(node::Buffer::New(data, length, callback, hint)->handle_)));
   }
 
   NAN_INLINE MaybeLocal<v8::Object> CopyBuffer(