src: handle null backing store in ArrayBufferViewContents::Read

Fixes: #62342
src: handle null backing store in ArrayBufferViewContents::Read
PR-URL: #62343Fixes: #62342
Reviewed-By: René <contact.9a5d6388@renegade334.me.uk>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

Author: mertcanaltin

src/util-inl.h

@@ -591,7 +591,9 @@ void ArrayBufferViewContents<T, S>::Read(v8::Local<v8::ArrayBufferView> abv) {
 static_assert(sizeof(T) == 1, "Only supports one-byte data at the moment");
  length_ = abv->ByteLength();
 if (length_ > sizeof(stack_storage_) || abv->HasBuffer()) {
- data_ = static_cast<T*>(abv->Buffer()->Data()) + abv->ByteOffset();
+auto buf_data = abv->Buffer()->Data();
+ data_ = buf_data != nullptr ? static_cast<T*>(buf_data) + abv->ByteOffset()
+ : stack_storage_;
  } else {
  abv->CopyContents(stack_storage_, sizeof(stack_storage_));
  data_ = stack_storage_;

test/parallel/test-crypto-authenticated.js

@@ -772,3 +772,26 @@ for (const test of TEST_CASES) {
 decipher.final();
 },/Unsupportedstateorunabletoauthenticatedata/);
 }
+
+// Refs: https://github.com/nodejs/node/issues/62342
+{
+constkey=crypto.randomBytes(16);
+constnonce=crypto.randomBytes(13);
+
+constcipher=crypto.createCipheriv('aes-128-ccm',key,nonce,{
+authTagLength: 16,
+});
+cipher.setAAD(Buffer.alloc(0),{plaintextLength: 0});
+cipher.update(newDataView(newArrayBuffer(0)));
+cipher.final();
+consttag=cipher.getAuthTag();
+assert.strictEqual(tag.length,16);
+
+constdecipher=crypto.createDecipheriv('aes-128-ccm',key,nonce,{
+authTagLength: 16,
+});
+decipher.setAuthTag(tag);
+decipher.setAAD(Buffer.alloc(0),{plaintextLength: 0});
+decipher.update(newDataView(newArrayBuffer(0)));
+decipher.final();
+}