fs: fix error when writing buffers > INT32_MAX

[zbjornson]

This reverts c380ee6. uv_fs_write returns an int, so it is not possible to ask it to write more than INT32_MAX. Instead, validate 'length' is an int32 in JS to avoid the assertion failure.

I think #36643 should be reopened. It's not possible that the reverted commit fixed it.

Repro:

const fs = require("fs");
const INT32_MAX =  0x7FFFFFFF;
const x = Buffer.allocUnsafe(INT32_MAX + 1);
const fd = fs.openSync("test.dat", "w");
console.log(fs.writeSync(fd, x, 0));

Error: UNKNOWN: unknown error, write
    at Object.writeSync (node:fs:882:3)
    at Object.<anonymous> (/home/zbjornson/test.js:9:16)
    at Module._compile (node:internal/modules/cjs/loader:1109:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1138:10)
    at Module.load (node:internal/modules/cjs/loader:989:32)
    at Function.Module._load (node:internal/modules/cjs/loader:829:14)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:12)
    at node:internal/main/run_main_module:17:47 {
  errno: -2147483648,
  syscall: 'write',
  code: 'UNKNOWN'
}

That errno -2147483648 is 2147483648 wrapped around.

I didn't add a test for this because the test would require allocating a 2 GB buffer.

cc @RaisinTen

[Ayase-252]

Confirmed on master branch, it throws a confusing Error,

node:internal/fs/utils:343
    throw err;
    ^

Error: EINVAL: invalid argument, write
    at Object.writeSync (node:fs:882:3)
    at Object.<anonymous> (/Users/qingyudeng/projects/node/index.js:5:16)
    at Module._compile (node:internal/modules/cjs/loader:1109:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1138:10)
    at Module.load (node:internal/modules/cjs/loader:989:32)
    at Function.Module._load (node:internal/modules/cjs/loader:829:14)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:12)
    at node:internal/main/run_main_module:17:47 {
  errno: -22,
  syscall: 'write',
  code: 'EINVAL'
}

[aduh95]

Please add a test, you can ask it to skip if there's not enough memory on the host:

node/test/parallel/test-tick-processor-arguments.js

Lines 8 to 9 in eb8a51a

	if (!common.enoughTestMem)
	common.skip('skipped due to memory requirements');

[zbjornson]

@aduh95 test added.

Thanks for the prompt reviews.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/37986/ 💚

[jasnell]

Landed in 8886b63