Fix crash impacting sharp & resvg

[Jarred-Sumner]

What does this PR do?

The implementation of napi_create_external_arraybuffer and napi_create_external_buffer was wrong. It called the finalizer with incorrect arguments, causing a crash when the memory is freed. This calls the finalizer with the expected arguments.

How did you verify your code works?

Integration test via resvg. Manually tested sharp as well.

[github-actions]

❌ @Jarred-Sumner 1 files with test failures on linux-x64-baseline:

• test/js/web/fetch/fetch.test.ts

View test output

^{#9a27d89922de52d84c65189adb1d46c12a805e4d}

[github-actions]

❌ @Jarred-Sumner 6 files with test failures on bun-darwin-aarch64:

• test/js/bun/spawn/spawn.test.ts
• test/js/bun/test/test-test.test.ts
• test/js/node/dns/node-dns.test.js
• test/js/node/fs/fs.test.ts
• test/js/node/watch/fs.watch.test.ts
• test/js/web/worker.test.ts

View test output

^{#9a27d89922de52d84c65189adb1d46c12a805e4d}

[github-actions]

❌ @Jarred-Sumner 7 files with test failures on bun-darwin-x64-baseline:

• test/js/bun/spawn/spawn-streaming-stdin.test.ts
• test/js/bun/sqlite/sqlite.test.js
• test/js/node/fs/fs.test.ts
• test/js/third_party/webpack/webpack.test.ts
• test/js/web/fetch/fetch.test.ts
• test/js/web/streams/streams.test.js
• test/js/web/timers/setTimeout.test.js

View test output

^{#9a27d89922de52d84c65189adb1d46c12a805e4d}