lib: unmask mode_t values with 0o777 · nodejs/node@b4b7d36

@@ -77,7 +77,7 @@ const {

7777} = require('internal/constants');7878const {7979 isUint32,80-validateAndMaskMode,80+validateMode,8181 validateInteger,8282 validateInt32,8383 validateUint32

@@ -416,7 +416,7 @@ function open(path, flags, mode, callback) {

416416callback = makeCallback(mode);417417mode = 0o666;418418} else {419-mode = validateAndMaskMode(mode, 'mode', 0o666);419+mode = validateMode(mode, 'mode', 0o666);420420callback = makeCallback(callback);421421}422422

@@ -434,7 +434,7 @@ function openSync(path, flags, mode) {

434434path = getPathFromURL(path);435435validatePath(path);436436const flagsNumber = stringToFlags(flags);437-mode = validateAndMaskMode(mode, 'mode', 0o666);437+mode = validateMode(mode, 'mode', 0o666);438438439439const ctx = { path };440440const result = binding.open(pathModule.toNamespacedPath(path),

@@ -721,7 +721,7 @@ function mkdir(path, mode, callback) {

721721mode = 0o777;722722} else {723723callback = makeCallback(callback);724-mode = validateAndMaskMode(mode, 'mode', 0o777);724+mode = validateMode(mode, 'mode', 0o777);725725}726726727727const req = new FSReqWrap();

@@ -732,7 +732,7 @@ function mkdir(path, mode, callback) {

732732function mkdirSync(path, mode) {733733path = getPathFromURL(path);734734validatePath(path);735-mode = validateAndMaskMode(mode, 'mode', 0o777);735+mode = validateMode(mode, 'mode', 0o777);736736const ctx = { path };737737binding.mkdir(pathModule.toNamespacedPath(path), mode, undefined, ctx);738738handleErrorFromBinding(ctx);

@@ -915,7 +915,7 @@ function unlinkSync(path) {

915915916916function fchmod(fd, mode, callback) {917917validateInt32(fd, 'fd', 0);918-mode = validateAndMaskMode(mode, 'mode');918+mode = validateMode(mode, 'mode');919919callback = makeCallback(callback);920920921921const req = new FSReqWrap();

@@ -925,7 +925,7 @@ function fchmod(fd, mode, callback) {

925925926926function fchmodSync(fd, mode) {927927validateInt32(fd, 'fd', 0);928-mode = validateAndMaskMode(mode, 'mode');928+mode = validateMode(mode, 'mode');929929const ctx = {};930930binding.fchmod(fd, mode, undefined, ctx);931931handleErrorFromBinding(ctx);

@@ -966,7 +966,7 @@ function lchmodSync(path, mode) {

966966function chmod(path, mode, callback) {967967path = getPathFromURL(path);968968validatePath(path);969-mode = validateAndMaskMode(mode, 'mode');969+mode = validateMode(mode, 'mode');970970callback = makeCallback(callback);971971972972const req = new FSReqWrap();

@@ -977,7 +977,7 @@ function chmod(path, mode, callback) {

977977function chmodSync(path, mode) {978978path = getPathFromURL(path);979979validatePath(path);980-mode = validateAndMaskMode(mode, 'mode');980+mode = validateMode(mode, 'mode');981981982982const ctx = { path };983983binding.chmod(pathModule.toNamespacedPath(path), mode, undefined, ctx);