Throw error when no buffer is given for `copy()`

diff --git a/index.js b/index.js
index 2584ee76a70af1a21cb2a0b9caf6457fde109ceb..63479f43a2ae4555e756f33a22c872bfb6062410 100644 (file)
--- a/index.js
+++ b/index.js
@@ -1445,6 +1445,9 @@ Buffer.prototype.writeDoubleBE = function writeDoubleBE (value, offset, noAssert
 
 // copy(targetBuffer, targetStart=0, sourceStart=0, sourceEnd=buffer.length)
 Buffer.prototype.copy = function copy (target, targetStart, start, end) {
+  if (target === undefined) {
+    throw new TypeError('argument should be a Buffer')
+  }
   if (!start) start = 0
   if (!end && end !== 0) end = this.length
   if (targetStart >= target.length) targetStart = target.length

diff --git a/test/node/test-buffer-alloc.js b/test/node/test-buffer-alloc.js
index d34652143fa32d50e8b47217296d05a9ab5383aa..3f04b3eb8d10a453f532a8f7742ab8654d5a9c59 100644 (file)
--- a/test/node/test-buffer-alloc.js
+++ b/test/node/test-buffer-alloc.js
@@ -935,11 +935,11 @@ var ps = Buffer.poolSize;
 Buffer.poolSize = 0;
 assert(Buffer.allocUnsafe(1).parent instanceof ArrayBuffer);
 Buffer.poolSize = ps;
-
+*/
 // Test Buffer.copy() segfault
 assert.throws(() => Buffer.allocUnsafe(10).copy(),
               /TypeError: argument should be a Buffer/);
-
+/*
 var regErrorMsg =
   new RegExp('The first argument must be one of type string, Buffer, ' +
              'ArrayBuffer, Array, or Array-like Object\\.');