code/wrench-js
Archived
1
Fork 0
Code Issues 22 Pull requests 7 Projects Releases Packages Wiki Activity Actions

readdirSyncRecursive: follow symlinks only if we haven't read the destination yet #87

Open
theosp wants to merge 1 commit from theosp/readdirSyncRecursive-symlink-loop-bugfix into master
pull from: theosp/readdirSyncRecursive-symlink-loop-bugfix
merge into: code:master
Conversation 0 Commits 1 Files changed 1 +7
1 changed files with 7 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

7
lib/wrench.js
View file

@ -23,7 +23,14 @@ var fs = require("fs"),
exports.readdirSyncRecursive = function(baseDir) { exports.readdirSyncRecursive = function(baseDir) {
baseDir = baseDir.replace(/\/$/, ''); baseDir = baseDir.replace(/\/$/, '');
var checkedInodes = {};
var readdirSyncRecursive = function(baseDir) { var readdirSyncRecursive = function(baseDir) {
var inode = fs.statSync(baseDir).ino;
if (inode in checkedInodes) {
return [];
}
checkedInodes[inode] = true;
var files = [], var files = [],
curFiles, curFiles,
nextDirs, nextDirs,