buffer: Fix Buffer.copy() third arg default

Third argument should default to source.length, not 0. Broken in 00b4b7b. "buffer: remove minor Buffer::Copy deoptimizations"
nodejs · Jan 27, 2013 · f0bad5a · f0bad5a
1 parent 0972acb
commit f0bad5a
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/node_buffer.cc b/src/node_buffer.cc
@@ -403,7 +403,8 @@ Handle<Value> Buffer::Copy(const Arguments &args) {
   size_t target_length = Buffer::Length(target);
   size_t target_start = args[1]->Uint32Value();
   size_t source_start = args[2]->Uint32Value();
-  size_t source_end = args[3]->Uint32Value();
+  size_t source_end = args[3]->IsUndefined() ? source->length_
+                                             : args[3]->Uint32Value();
 
   if (source_end < source_start) {
     return ThrowRangeError("sourceEnd < sourceStart");