diff --git a/lib/fs.js b/lib/fs.js index 7a3480a0ec..2a96a91d9b 100644 --- a/lib/fs.js +++ b/lib/fs.js @@ -1180,8 +1180,8 @@ function chmodSync(path, mode) { function lchown(path, uid, gid, callback) { callback = makeCallback(callback); path = getValidatedPath(path); - validateInteger(uid, 'uid', -1, kMaxUserId); - validateInteger(gid, 'gid', -1, kMaxUserId); + validateInteger(uid, 'uid', { min: -1, max: kMaxUserId }); + validateInteger(gid, 'gid', { min: -1, max: kMaxUserId }); const req = new FSReqCallback(); req.oncomplete = callback; binding.lchown(pathModule.toNamespacedPath(path), uid, gid, req); @@ -1189,8 +1189,8 @@ function lchown(path, uid, gid, callback) { function lchownSync(path, uid, gid) { path = getValidatedPath(path); - validateInteger(uid, 'uid', -1, kMaxUserId); - validateInteger(gid, 'gid', -1, kMaxUserId); + validateInteger(uid, 'uid', { min: -1, max: kMaxUserId }); + validateInteger(gid, 'gid', { min: -1, max: kMaxUserId }); const ctx = { path }; binding.lchown(pathModule.toNamespacedPath(path), uid, gid, undefined, ctx); handleErrorFromBinding(ctx); @@ -1198,8 +1198,8 @@ function lchownSync(path, uid, gid) { function fchown(fd, uid, gid, callback) { validateInt32(fd, 'fd', 0); - validateInteger(uid, 'uid', -1, kMaxUserId); - validateInteger(gid, 'gid', -1, kMaxUserId); + validateInteger(uid, 'uid', { min: -1, max: kMaxUserId }); + validateInteger(gid, 'gid', { min: -1, max: kMaxUserId }); const req = new FSReqCallback(); req.oncomplete = makeCallback(callback); @@ -1208,8 +1208,8 @@ function fchown(fd, uid, gid, callback) { function fchownSync(fd, uid, gid) { validateInt32(fd, 'fd', 0); - validateInteger(uid, 'uid', -1, kMaxUserId); - validateInteger(gid, 'gid', -1, kMaxUserId); + validateInteger(uid, 'uid', { min: -1, max: kMaxUserId }); + validateInteger(gid, 'gid', { min: -1, max: kMaxUserId }); const ctx = {}; binding.fchown(fd, uid, gid, undefined, ctx); @@ -1219,8 +1219,8 @@ function fchownSync(fd, uid, gid) { function chown(path, uid, gid, callback) { callback = makeCallback(callback); path = getValidatedPath(path); - validateInteger(uid, 'uid', -1, kMaxUserId); - validateInteger(gid, 'gid', -1, kMaxUserId); + validateInteger(uid, 'uid', { min: -1, max: kMaxUserId }); + validateInteger(gid, 'gid', { min: -1, max: kMaxUserId }); const req = new FSReqCallback(); req.oncomplete = callback; @@ -1229,8 +1229,8 @@ function chown(path, uid, gid, callback) { function chownSync(path, uid, gid) { path = getValidatedPath(path); - validateInteger(uid, 'uid', -1, kMaxUserId); - validateInteger(gid, 'gid', -1, kMaxUserId); + validateInteger(uid, 'uid', { min: -1, max: kMaxUserId }); + validateInteger(gid, 'gid', { min: -1, max: kMaxUserId }); const ctx = { path }; binding.chown(pathModule.toNamespacedPath(path), uid, gid, undefined, ctx); handleErrorFromBinding(ctx);