nodejs
diff --git a/‎src/node_buffer.cc‎
Lines changed: 8 additions & 5 deletions b/‎src/node_buffer.cc‎
Lines changed: 8 additions & 5 deletions
diff --git a/‎test/addons/buffer-free-callback/binding.cc‎
Lines changed: 1 addition & 1 deletion b/‎test/addons/buffer-free-callback/binding.cc‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎test/addons/buffer-free-callback/test.js‎
Lines changed: 17 additions & 4 deletions b/‎test/addons/buffer-free-callback/test.js‎
Lines changed: 17 additions & 4 deletions
@@ -148,11 +148,14 @@ void CallbackInfo::WeakCallback(
 
 
 void CallbackInfo::WeakCallback(Isolate* isolate, Local<Object> object) {
-  SPREAD_ARG(object, obj);
-  CHECK_EQ(obj_offset, 0);
-  CHECK_EQ(obj_c.ByteLength(), obj_length);
-
-  obj->Buffer()->Neuter();
+  CHECK(object->IsArrayBuffer());
+  Local<ArrayBuffer> buf = object.As<ArrayBuffer>();
+  ArrayBuffer::Contents obj_c = buf->GetContents();
+  char* const obj_data = static_cast<char*>(obj_c.Data());
+  if (buf->ByteLength() != 0)
+    CHECK_NE(obj_data, nullptr);
+
+  buf->Neuter();
   callback_(obj_data, hint_);
   int64_t change_in_bytes = -static_cast<int64_t>(sizeof(*this));
   isolate->AdjustAmountOfExternalAllocatedMemory(change_in_bytes);
 
@@ -16,7 +16,7 @@ void Alloc(const v8::FunctionCallbackInfo<v8::Value>& args) {
   args.GetReturnValue().Set(node::Buffer::New(
         isolate,
         buf,
-        sizeof(buf),
+        args[0]->IntegerValue(),
         FreeCallback,
         nullptr).ToLocalChecked());
 }
 
@@ -4,7 +4,20 @@
 require('../../common');
 var assert = require('assert');
 var binding = require('./build/Release/binding');
-var buf = binding.alloc();
-var slice = buf.slice(32);
-buf = null;
-binding.check(slice);
+
+function check(size) {
+  var buf = binding.alloc(size);
+  var slice = buf.slice(size >>> 1);
+
+  buf = null;
+  binding.check(slice);
+  slice = null;
+  gc();
+  gc();
+  gc();
+}
+
+check(64);
+
+// Empty ArrayBuffer does not allocate data, worth checking
+check(0);
Original file line number	Diff line number	Diff line change
@@ -16,7 +16,7 @@ void Alloc(const v8::FunctionCallbackInfo<v8::Value>& args) {
16	16	args.GetReturnValue().Set(node::Buffer::New(
17	17	isolate,
18	18	buf,
19		- sizeof(buf),
	19	+ args[0]->IntegerValue(),
20	20	FreeCallback,
21	21	nullptr).ToLocalChecked());
22	22	}