buffer: use IntegerValue not Uin32Value to support copying buffers gr…

…eater than 4GB in length on 64-bit systems
nodejs · Oct 22, 2024 · d15c192 · d15c192
1 parent b76bcb3
commit d15c192
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/node_buffer.cc b/src/node_buffer.cc
@@ -572,9 +572,9 @@ void SlowCopy(const FunctionCallbackInfo<Value>& args) {
   ArrayBufferViewContents<char> source(args[0]);
   SPREAD_BUFFER_ARG(args[1].As<Object>(), target);
 
-  const auto target_start = args[2]->Uint32Value(env->context()).ToChecked();
-  const auto source_start = args[3]->Uint32Value(env->context()).ToChecked();
-  const auto to_copy = args[4]->Uint32Value(env->context()).ToChecked();
+  const auto target_start = args[2]->IntegerValue(env->context()).ToChecked();
+  const auto source_start = args[3]->IntegerValue(env->context()).ToChecked();
+  const auto to_copy = args[4]->IntegerValue(env->context()).ToChecked();
 
   memmove(target_data + target_start, source.data() + source_start, to_copy);
   args.GetReturnValue().Set(to_copy);