diff options
author | Ujjwal Sharma <usharma1998@gmail.com> | 2018-04-03 23:04:20 +0530 |
---|---|---|
committer | Luigi Pinca <luigipinca@gmail.com> | 2018-04-06 10:51:35 +0200 |
commit | 38a692963f000e3bd0f8413617d3b5774039dff8 (patch) | |
tree | 58cd6a782ae60504beaafc0b614fa06767d8c44b /src/node_crypto.h | |
parent | 496d6023e0c372c76746c35fdba800fa943cbffc (diff) | |
download | android-node-v8-38a692963f000e3bd0f8413617d3b5774039dff8.tar.gz android-node-v8-38a692963f000e3bd0f8413617d3b5774039dff8.tar.bz2 android-node-v8-38a692963f000e3bd0f8413617d3b5774039dff8.zip |
fs: make ReadStream throw TypeError on NaN
Make ReadStream (and thus createReadStream) throw a TypeError signalling
towards an invalid argument type when either options.start or
options.end (or obviously, both) are set to NaN.
Also add regression tests for the same.
PR-URL: https://github.com/nodejs/node/pull/19775
Fixes: https://github.com/nodejs/node/issues/19715
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'src/node_crypto.h')
0 files changed, 0 insertions, 0 deletions