diff options
| author | Rami Moshe <rami.moshe@jivesoftware.com> | 2017-09-23 01:40:37 -0400 |
|---|---|---|
| committer | Refael Ackermann <refack@gmail.com> | 2017-10-28 14:04:53 -0400 |
| commit | 9788e96836243e2439867b81c6d6bf522115a0f5 (patch) | |
| tree | c39d8173d2d8b27756533fd8f8cd056588b87e69 /lib/querystring.js | |
| parent | 1c0772444c2649f1fc0e6d1a1571b981995a08c7 (diff) | |
| download | android-node-v8-9788e96836243e2439867b81c6d6bf522115a0f5.tar.gz android-node-v8-9788e96836243e2439867b81c6d6bf522115a0f5.tar.bz2 android-node-v8-9788e96836243e2439867b81c6d6bf522115a0f5.zip | |
querystring: convert to using internal/errors
PR-URL: https://github.com/nodejs/node/pull/15565
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Diffstat (limited to 'lib/querystring.js')
| -rw-r--r-- | lib/querystring.js | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/lib/querystring.js b/lib/querystring.js index 936b7463c9..ec6ad51a89 100644 --- a/lib/querystring.js +++ b/lib/querystring.js @@ -24,6 +24,7 @@ 'use strict'; const { Buffer } = require('buffer'); +const errors = require('internal/errors'); const { hexTable, isHexTable @@ -174,11 +175,12 @@ function qsEscape(str) { } // Surrogate pair ++i; - var c2; - if (i < str.length) - c2 = str.charCodeAt(i) & 0x3FF; - else - throw new URIError('URI malformed'); + + if (i >= str.length) + throw new errors.URIError('ERR_INVALID_URI'); + + var c2 = str.charCodeAt(i) & 0x3FF; + lastPos = i + 1; c = 0x10000 + (((c & 0x3FF) << 10) | c2); out += hexTable[0xF0 | (c >> 18)] + |