nodejs
diff --git a/Collapse file
‎src/node_buffer.h‎
Copy file name to clipboardExpand all lines: src/node_buffer.h
+5Lines changed: 5 additions & 0 deletions b/Collapse file
‎src/node_buffer.h‎
Copy file name to clipboardExpand all lines: src/node_buffer.h
+5Lines changed: 5 additions & 0 deletions
diff --git a/Collapse file
‎src/node_crypto.cc‎
Copy file name to clipboardExpand all lines: src/node_crypto.cc
+3-3Lines changed: 3 additions & 3 deletions b/Collapse file
‎src/node_crypto.cc‎
Copy file name to clipboardExpand all lines: src/node_crypto.cc
+3-3Lines changed: 3 additions & 3 deletions
diff --git a/Collapse file
‎src/stream_wrap.cc‎
Copy file name to clipboardExpand all lines: src/stream_wrap.cc
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎src/stream_wrap.cc‎
Copy file name to clipboardExpand all lines: src/stream_wrap.cc
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎src/udp_wrap.cc‎
Copy file name to clipboardExpand all lines: src/udp_wrap.cc
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎src/udp_wrap.cc‎
Copy file name to clipboardExpand all lines: src/udp_wrap.cc
+1-1Lines changed: 1 addition & 1 deletion
@@ -67,12 +67,17 @@ static inline bool IsWithinBounds(size_t off, size_t len, size_t max) {
 // in src/node_internals.h because of a circular dependency.
 #if defined(NODE_WANT_INTERNALS)
 v8::MaybeLocal<v8::Object> New(Environment* env, size_t size);
+// Makes a copy of |data|.
 v8::MaybeLocal<v8::Object> New(Environment* env, const char* data, size_t len);
+// Takes ownership of |data|.
 v8::MaybeLocal<v8::Object> New(Environment* env,
                                char* data,
                                size_t length,
                                FreeCallback callback,
                                void* hint);
+// Takes ownership of |data|.  Must allocate |data| with malloc() or realloc()
+// because ArrayBufferAllocator::Free() deallocates it again with free().
+// Mixing operator new and free() is undefined behavior so don't do that.
 v8::MaybeLocal<v8::Object> Use(Environment* env, char* data, size_t length);
 #endif  // defined(NODE_WANT_INTERNALS)
 
 
@@ -4477,7 +4477,7 @@ void ECDH::ComputeSecret(const FunctionCallbackInfo<Value>& args) {
     return env->ThrowError("Failed to compute ECDH key");
   }
 
-  Local<Object> buf = Buffer::New(env, out, out_len).ToLocalChecked();
+  Local<Object> buf = Buffer::Use(env, out, out_len).ToLocalChecked();
   args.GetReturnValue().Set(buf);
 }
 
@@ -4542,7 +4542,7 @@ void ECDH::GetPrivateKey(const FunctionCallbackInfo<Value>& args) {
   }
 
   Local<Object> buf =
-      Buffer::New(env, reinterpret_cast<char*>(out), size).ToLocalChecked();
+      Buffer::Use(env, reinterpret_cast<char*>(out), size).ToLocalChecked();
   args.GetReturnValue().Set(buf);
 }
 
@@ -4945,7 +4945,7 @@ void RandomBytesCheck(RandomBytesRequest* req, Local<Value> argv[2]) {
     size_t size;
     req->return_memory(&data, &size);
     argv[0] = Null(req->env()->isolate());
-    argv[1] = Buffer::New(req->env(), data, size).ToLocalChecked();
+    argv[1] = Buffer::Use(req->env(), data, size).ToLocalChecked();
   }
 }
 
 
@@ -223,7 +223,7 @@ void StreamWrap::OnReadImpl(ssize_t nread,
     CHECK_EQ(pending, UV_UNKNOWN_HANDLE);
   }
 
-  Local<Object> obj = Buffer::New(env, base, nread).ToLocalChecked();
+  Local<Object> obj = Buffer::Use(env, base, nread).ToLocalChecked();
   wrap->EmitData(nread, obj, pending_obj);
 }
 
 
@@ -408,7 +408,7 @@ void UDPWrap::OnRecv(uv_udp_t* handle,
   }
 
   char* base = static_cast<char*>(realloc(buf->base, nread));
-  argv[2] = Buffer::New(env, base, nread).ToLocalChecked();
+  argv[2] = Buffer::Use(env, base, nread).ToLocalChecked();
   argv[3] = AddressToJS(env, addr);
   wrap->MakeCallback(env->onmessage_string(), ARRAY_SIZE(argv), argv);
 }
Original file line number	Diff line number	Diff line change
`@@ -4477,7 +4477,7 @@ void ECDH::ComputeSecret(const FunctionCallbackInfo<Value>& args) {`
`4477`	`4477`	`return env->ThrowError("Failed to compute ECDH key");`
`4478`	`4478`	`}`
`4479`	`4479`
`4480`		`- Local<Object> buf = Buffer::New(env, out, out_len).ToLocalChecked();`
	`4480`	`+ Local<Object> buf = Buffer::Use(env, out, out_len).ToLocalChecked();`
`4481`	`4481`	`args.GetReturnValue().Set(buf);`
`4482`	`4482`	`}`
`4483`	`4483`
`@@ -4542,7 +4542,7 @@ void ECDH::GetPrivateKey(const FunctionCallbackInfo<Value>& args) {`
`4542`	`4542`	`}`
`4543`	`4543`
`4544`	`4544`	`Local<Object> buf =`
`4545`		`- Buffer::New(env, reinterpret_cast<char*>(out), size).ToLocalChecked();`
	`4545`	`+ Buffer::Use(env, reinterpret_cast<char*>(out), size).ToLocalChecked();`
`4546`	`4546`	`args.GetReturnValue().Set(buf);`
`4547`	`4547`	`}`
`4548`	`4548`
`@@ -4945,7 +4945,7 @@ void RandomBytesCheck(RandomBytesRequest* req, Local<Value> argv[2]) {`
`4945`	`4945`	`size_t size;`
`4946`	`4946`	`req->return_memory(&data, &size);`
`4947`	`4947`	`argv[0] = Null(req->env()->isolate());`
`4948`		`- argv[1] = Buffer::New(req->env(), data, size).ToLocalChecked();`
	`4948`	`+ argv[1] = Buffer::Use(req->env(), data, size).ToLocalChecked();`
`4949`	`4949`	`}`
`4950`	`4950`	`}`
`4951`	`4951`
Original file line number	Diff line number	Diff line change
`@@ -223,7 +223,7 @@ void StreamWrap::OnReadImpl(ssize_t nread,`
`223`	`223`	`CHECK_EQ(pending, UV_UNKNOWN_HANDLE);`
`224`	`224`	`}`
`225`	`225`
`226`		`- Local<Object> obj = Buffer::New(env, base, nread).ToLocalChecked();`
	`226`	`+ Local<Object> obj = Buffer::Use(env, base, nread).ToLocalChecked();`
`227`	`227`	`wrap->EmitData(nread, obj, pending_obj);`
`228`	`228`	`}`
`229`	`229`
Original file line number	Diff line number	Diff line change
`@@ -408,7 +408,7 @@ void UDPWrap::OnRecv(uv_udp_t* handle,`
`408`	`408`	`}`
`409`	`409`
`410`	`410`	`char* base = static_cast<char*>(realloc(buf->base, nread));`
`411`		`- argv[2] = Buffer::New(env, base, nread).ToLocalChecked();`
	`411`	`+ argv[2] = Buffer::Use(env, base, nread).ToLocalChecked();`
`412`	`412`	`argv[3] = AddressToJS(env, addr);`
`413`	`413`	`wrap->MakeCallback(env->onmessage_string(), ARRAY_SIZE(argv), argv);`
`414`	`414`	`}`