fixup! ffi: clarify detached ArrayBuffer error

Report that the ArrayBuffer is detached instead of describing its
backing store as invalid.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>

Author: trivikr

src/ffi/data.cc

@@ -685,7 +685,7 @@ void ExportBytes(const FunctionCallbackInfo<Value>& args) {
       args[0]->IsArrayBufferView()) {
     view.ReadValue(args[0]);
     if (view.WasDetached()) {
-      THROW_ERR_INVALID_ARG_VALUE(env, "Invalid ArrayBufferView backing store");
+      THROW_ERR_INVALID_ARG_VALUE(env, "ArrayBuffer is detached");
       return;
     }
   } else {

test/ffi/test-ffi-memory.js

@@ -153,6 +153,11 @@ test('ffi rejects detached array buffers and views as pointers', () => {
 
   structuredClone(arrayBuffer, { transfer: [arrayBuffer] });
 
+  assert.throws(() => ffi.exportArrayBuffer(arrayBuffer, 0n, 0), {
+    code: 'ERR_INVALID_ARG_VALUE',
+    message: 'ArrayBuffer is detached',
+  });
+
   for (const [value, rawPointerMessage, argumentMessage] of [
     [
       arrayBuffer,