Skip to content

Commit 3038f2f

Browse files
committed
Prevent root-owned files in user-owned locations, and vice versa.
1 parent 732c115 commit 3038f2f

19 files changed

+136
-41
lines changed

lib/config.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ var ini = require('ini')
1111
var editor = require('editor')
1212
var os = require('os')
1313
var path = require('path')
14-
var mkdirp = require('mkdirp')
14+
var mkdirp = require('gentle-fs').mkdir
1515
var umask = require('./utils/umask')
1616
var usage = require('./utils/usage')
1717
var output = require('./utils/output')

lib/config/core.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ var path = require('path')
88
var nopt = require('nopt')
99
var ini = require('ini')
1010
var Umask = configDefs.Umask
11-
var mkdirp = require('mkdirp')
11+
var mkdirp = require('gentle-fs').mkdir
1212
var umask = require('../utils/umask')
1313
var isWindows = require('../utils/is-windows.js')
1414

lib/config/set-user.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ module.exports = setUser
33
var assert = require('assert')
44
var path = require('path')
55
var fs = require('fs')
6-
var mkdirp = require('mkdirp')
6+
var mkdirp = require('gentle-fs').mkdir
77

88
function setUser (cb) {
99
var defaultConf = this.root

lib/install.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ var readPackageJson = require('read-package-json')
104104
var chain = require('slide').chain
105105
var asyncMap = require('slide').asyncMap
106106
var archy = require('archy')
107-
var mkdirp = require('mkdirp')
107+
var mkdirp = require('gentle-fs').mkdir
108108
var rimraf = require('rimraf')
109109
var iferr = require('iferr')
110110
var validate = require('aproba')

lib/install/action/extract.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const BB = require('bluebird')
55
const figgyPudding = require('figgy-pudding')
66
const stat = BB.promisify(require('graceful-fs').stat)
77
const gentlyRm = BB.promisify(require('../../utils/gently-rm.js'))
8-
const mkdirp = BB.promisify(require('mkdirp'))
8+
const mkdirp = BB.promisify(require('gentle-fs').mkdir)
99
const moduleName = require('../../utils/module-name.js')
1010
const moduleStagingPath = require('../module-staging-path.js')
1111
const move = require('../../utils/move.js')

lib/install/action/finalize.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ const path = require('path')
33
const fs = require('graceful-fs')
44
const Bluebird = require('bluebird')
55
const rimraf = Bluebird.promisify(require('rimraf'))
6-
const mkdirp = Bluebird.promisify(require('mkdirp'))
6+
const mkdirp = Bluebird.promisify(require('gentle-fs').mkdir)
77
const lstat = Bluebird.promisify(fs.lstat)
88
const readdir = Bluebird.promisify(fs.readdir)
99
const symlink = Bluebird.promisify(fs.symlink)

lib/install/action/move.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ var path = require('path')
44
var chain = require('slide').chain
55
var iferr = require('iferr')
66
var rimraf = require('rimraf')
7-
var mkdirp = require('mkdirp')
7+
var mkdirp = require('gentle-fs').mkdir
88
var rmStuff = require('../../unbuild.js').rmStuff
99
var lifecycle = require('../../utils/lifecycle.js')
1010
var move = require('../../utils/move.js')

lib/install/action/remove.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ var path = require('path')
33
var fs = require('graceful-fs')
44
var rimraf = require('rimraf')
55
var asyncMap = require('slide').asyncMap
6-
var mkdirp = require('mkdirp')
6+
var mkdirp = require('gentle-fs').mkdir
77
var npm = require('../../npm.js')
88
var andIgnoreErrors = require('../and-ignore-errors.js')
99
var move = require('../../utils/move.js')

lib/search/all-package-metadata.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const figgyPudding = require('figgy-pudding')
99
const fs = require('graceful-fs')
1010
const JSONStream = require('JSONStream')
1111
const log = require('npmlog')
12-
const mkdir = BB.promisify(require('mkdirp'))
12+
const mkdir = BB.promisify(require('gentle-fs').mkdir)
1313
const ms = require('mississippi')
1414
const npmFetch = require('libnpm/fetch')
1515
const path = require('path')

lib/utils/cache-file.js

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
// XXX use infer-owner or gentle-fs.mkdir here
12
const npm = require('../npm.js')
23
const path = require('path')
34
const chownr = require('chownr')

lib/utils/correct-mkdir.js

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
// XXX this can probably be replaced with gentle-fs.mkdir everywhere it's used
12
const chownr = require('chownr')
23
const inflight = require('inflight')
34
const log = require('npmlog')

node_modules/gentle-fs/CHANGELOG.md

+35
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

node_modules/gentle-fs/index.js

+3-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

node_modules/gentle-fs/lib/chown.js

+24
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

node_modules/gentle-fs/lib/link.js

+19-11
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

node_modules/gentle-fs/lib/mkdir.js

+22
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

node_modules/gentle-fs/package.json

+16-16
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package-lock.json

+5-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@
5959
"find-npm-prefix": "^1.0.2",
6060
"fs-vacuum": "~1.2.10",
6161
"fs-write-stream-atomic": "~1.0.10",
62-
"gentle-fs": "^2.0.1",
62+
"gentle-fs": "^2.2.1",
6363
"glob": "^7.1.4",
6464
"graceful-fs": "^4.2.0",
6565
"has-unicode": "~2.0.1",

0 commit comments

Comments
 (0)