Robert Nagy
Robert Nagy
> > I'm happy to jump into a napi working group meeting to further discuss. > > It would be awesome! > > Is [nxtedition/rocks-level#16](https://github.com/nxtedition/rocks-level/pull/16) a benchmark in real world...
Then we might as well call this node_create_buffer?
Here is a possible motivating sample: > Creating buffers from pointers can be slow as each pointer requires a finalizer to be registered. > > ```cpp > static void Finalize(napi_env...
If I read this correctly it's `SlowCopy` that has the problem?
Who exactly is calling `SlowCopy`? It seems to me that there is a invalid assumption made that the caller to `SlowCopy` validates the range. But I guess that only applies...
Can someone who can reproduce check if the following fixes the issue? ``` diff --git a/src/node_buffer.cc b/src/node_buffer.cc index ad6b794cf5d..6b2551c72fe 100644 --- a/src/node_buffer.cc +++ b/src/node_buffer.cc @@ -568,6 +568,9 @@ void StringSlice(const...
@joyeecheung any insights? I don't understand how `Buffer` even comes into the picture here... e.g. `v8.deserialize(v8.serialize({a: new Int32Array(1024)}))`
> but it still causes a crash in the same place. @ramidzkh the same crash?
How? Buffer is not part of v8.
We should try to land and include: https://github.com/nodejs/node/pull/54533 https://github.com/nodejs/node/pull/54524 https://github.com/nodejs/node/pull/54526