diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2019-04-02 00:01:29 +0200 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2019-04-15 17:21:34 +0200 |
commit | 693401d0ddd752e5fa47b882e56e252c42c94c0e (patch) | |
tree | fadabceddc0df7bd8d982bcd6c00cc11f3565e5d /lib/internal/errors.js | |
parent | 2fed83dee884c3bddafa67bb53abf507db1a8ba3 (diff) | |
download | android-node-v8-693401d0ddd752e5fa47b882e56e252c42c94c0e.tar.gz android-node-v8-693401d0ddd752e5fa47b882e56e252c42c94c0e.tar.bz2 android-node-v8-693401d0ddd752e5fa47b882e56e252c42c94c0e.zip |
buffer: use stricter range checks
This validates the input to make sure the arguments do not overflow.
Before, if the input would overflow, it would cause the write to be
performt in the wrong spot / result in unexpected behavior.
Instead, just use a strict number validation.
PR-URL: https://github.com/nodejs/node/pull/27045
Fixes: https://github.com/nodejs/node/issues/27043
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Diffstat (limited to 'lib/internal/errors.js')
-rw-r--r-- | lib/internal/errors.js | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/internal/errors.js b/lib/internal/errors.js index 6334626d3c..bc3f43653a 100644 --- a/lib/internal/errors.js +++ b/lib/internal/errors.js @@ -994,7 +994,6 @@ E('ERR_NO_CRYPTO', 'Node.js is not compiled with OpenSSL crypto support', Error); E('ERR_NO_ICU', '%s is not supported on Node.js compiled without ICU', TypeError); -E('ERR_NO_LONGER_SUPPORTED', '%s is no longer supported', Error); E('ERR_OUT_OF_RANGE', (str, range, input, replaceDefaultBoolean = false) => { assert(range, 'Missing "range" argument'); |