Skip to content

Commit c38576e

Browse files
ConnormihaMylesBorins
authored andcommitted
buffer: simplify check size in assertSize
PR-URL: #18665 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 876e186 commit c38576e

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

lib/buffer.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -237,9 +237,7 @@ function assertSize(size) {
237237

238238
if (typeof size !== 'number') {
239239
err = new errors.TypeError('ERR_INVALID_ARG_TYPE', 'size', 'number', size);
240-
} else if (size < 0) {
241-
err = new errors.RangeError('ERR_INVALID_OPT_VALUE', 'size', size);
242-
} else if (size > kMaxLength) {
240+
} else if (size < 0 || size > kMaxLength) {
243241
err = new errors.RangeError('ERR_INVALID_OPT_VALUE', 'size', size);
244242
}
245243

0 commit comments

Comments
 (0)