lib: add case of bufferLength to validateOffsetLengthRead() in utils.js

In validateOffsetLengthRead(), an error message is generated if offset
or length are out of range. There should also be an error message if
buffer is empty, in which case it cannot write data in it.

Generally this validateOffsetLengthRead() is triggered with
fs.readSync(fd, buffer, offset, length, position), where if buffer is
empty, the case for zero bufferLength is triggered and the
corresponding message is thrown.

Fixes: nodejs#21193

lib/internal/errors.js

@@ -637,8 +637,6 @@ E('ERR_INVALID_ARG_VALUE', (name, value, reason = 'is invalid') => {
   let inspected = util.inspect(value);
   if (inspected.length > 128) {
     inspected = `${inspected.slice(0, 128)}...`;
-  } else if (inspected === '0') {
-    inspected = 'no value.';
   }
   return `The argument '${name}' ${reason}. Received ${inspected}`;
 }, TypeError, RangeError);

lib/internal/fs/utils.js

@@ -294,8 +294,7 @@ function validateOffsetLengthRead(offset, length, bufferLength) {
 
   if (offset < 0 || offset >= bufferLength) {
     if (bufferLength === 0) {
-      throw ERR_INVALID_ARG_VALUE('buffer', 0,
-                                  'is empty and can\'t be written');
+      err = new ERR_OUT_OF_RANGE('bufferLength', '> 0', bufferLength);
     } else {
       err = new ERR_OUT_OF_RANGE('offset',
                                  `>= 0 && <= ${bufferLength}`, offset);