Skip to content

Commit 1d89180

Browse files
committed
fixup! ffi: throw on allocation failure in toArrayBuffer()
1 parent 54907ac commit 1d89180

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

‎test/ffi/test-ffi-memory.js‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,8 @@ test('ffi rejects unsafe integers as an offset or length', () => {
373373

374374
test('ffi toBuffer and toArrayBuffer throw when the copy cannot be allocated', {
375375
skip: (bufferConstants.MAX_LENGTH < 2 ** 50 && 'requires a 64-bit buffer length limit') ||
376-
(common.isASan && 'ASan aborts on huge allocations'),
376+
(common.isASan && 'ASan aborts on huge allocations') ||
377+
(common.isAIX && 'huge allocations may succeed on AIX and get the process killed'),
377378
}, () => {
378379
// The allocation fails before the source pointer is read.
379380
const error = { code: 'ERR_MEMORY_ALLOCATION_FAILED' };

0 commit comments

Comments
 (0)