nodejs
diff --git a/Collapse file
‎lib/buffer.js‎
Copy file name to clipboardExpand all lines: lib/buffer.js
+2-8Lines changed: 2 additions & 8 deletions b/Collapse file
‎lib/buffer.js‎
Copy file name to clipboardExpand all lines: lib/buffer.js
+2-8Lines changed: 2 additions & 8 deletions
diff --git a/Collapse file
‎test/parallel/test-buffer-alloc.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-buffer-alloc.js
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎test/parallel/test-buffer-alloc.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-buffer-alloc.js
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-buffer-copy.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-buffer-copy.js
+9-2Lines changed: 9 additions & 2 deletions b/Collapse file
‎test/parallel/test-buffer-copy.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-buffer-copy.js
+9-2Lines changed: 9 additions & 2 deletions
@@ -231,8 +231,8 @@ function _copy(source, target, targetStart, sourceStart, sourceEnd) {
     sourceStart = 0;
   } else {
     sourceStart = toInteger(sourceStart, 0);
-    if (sourceStart < 0)
-      throw new ERR_OUT_OF_RANGE('sourceStart', '>= 0', sourceStart);
+    if (sourceStart < 0 || sourceStart > source.length)
+      throw new ERR_OUT_OF_RANGE('sourceStart', `>= 0 && <= ${source.length}`, sourceStart);
   }
 
   if (sourceEnd === undefined) {
@@ -246,12 +246,6 @@ function _copy(source, target, targetStart, sourceStart, sourceEnd) {
   if (targetStart >= target.length || sourceStart >= sourceEnd)
     return 0;
 
-  if (sourceStart > source.length) {
-    throw new ERR_OUT_OF_RANGE('sourceStart',
-                               `<= ${source.length}`,
-                               sourceStart);
-  }
-
   return _copyActual(source, target, targetStart, sourceStart, sourceEnd);
 }
 
 
@@ -124,7 +124,7 @@ b.copy(Buffer.alloc(0), 1, 1, 1);
 b.copy(Buffer.alloc(1), 1, 1, 1);
 
 // Try to copy 0 bytes from past the end of the source buffer
-b.copy(Buffer.alloc(1), 0, 2048, 2048);
+b.copy(Buffer.alloc(1), 0, 1024, 1024);
 
 // Testing for smart defaults and ability to pass string values as offset
 {
 
@@ -155,8 +155,15 @@ assert.throws(
   {
     code: 'ERR_OUT_OF_RANGE',
     name: 'RangeError',
-    message: 'The value of "sourceStart" is out of range. ' +
-             'It must be >= 0. Received -1'
+  }
+);
+
+// Copy throws if sourceStart is greater than length of source
+assert.throws(
+  () => Buffer.allocUnsafe(5).copy(Buffer.allocUnsafe(5), 0, 100),
+  {
+    code: 'ERR_OUT_OF_RANGE',
+    name: 'RangeError',
   }
 );
Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ b.copy(Buffer.alloc(0), 1, 1, 1);`
`124`	`124`	`b.copy(Buffer.alloc(1), 1, 1, 1);`
`125`	`125`
`126`	`126`	`// Try to copy 0 bytes from past the end of the source buffer`
`127`		`-b.copy(Buffer.alloc(1), 0, 2048, 2048);`
	`127`	`+b.copy(Buffer.alloc(1), 0, 1024, 1024);`
`128`	`128`
`129`	`129`	`// Testing for smart defaults and ability to pass string values as offset`
`130`	`130`	`{`