diff options
author | Anna Henningsen <anna@addaleax.net> | 2019-02-05 21:51:28 +0100 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2019-02-13 21:36:24 +0100 |
commit | a80b29b38a66cec456e4ec73fb8030f45d186005 (patch) | |
tree | ad44198e7620505db311cdc99f689b6d7f7a058b /test | |
parent | da0dc51e396ea4a1f12f259a8149f4177ad674e8 (diff) | |
download | android-node-v8-a80b29b38a66cec456e4ec73fb8030f45d186005.tar.gz android-node-v8-a80b29b38a66cec456e4ec73fb8030f45d186005.tar.bz2 android-node-v8-a80b29b38a66cec456e4ec73fb8030f45d186005.zip |
worker: set stack size for worker threads
This is so we can inform V8 about a known limit for the stack.
Otherwise, on some systems recursive functions may lead to
segmentation faults rather than “safe” failures.
PR-URL: https://github.com/nodejs/node/pull/26049
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'test')
-rw-r--r-- | test/parallel/test-worker-stack-overflow.js | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/test/parallel/test-worker-stack-overflow.js b/test/parallel/test-worker-stack-overflow.js new file mode 100644 index 0000000000..99a34b5369 --- /dev/null +++ b/test/parallel/test-worker-stack-overflow.js @@ -0,0 +1,11 @@ +'use strict'; +const common = require('../common'); +const assert = require('assert'); +const { Worker } = require('worker_threads'); + +const worker = new Worker('function f() { f(); } f();', { eval: true }); + +worker.on('error', common.mustCall((err) => { + assert.strictEqual(err.constructor, RangeError); + assert.strictEqual(err.message, 'Maximum call stack size exceeded'); +})); |