author | Johan Herland <johan@herland.net> | 2010-11-15 19:40:43 (UTC) |
---|---|---|
committer | Lars Hjemli <larsh@prediktor.no> | 2010-11-16 08:03:43 (UTC) |
commit | 682adbc0cad2baa1a6119013b166f52de3ee3352 (patch) (side-by-side diff) | |
tree | bef3cd4f21672a40399816dcbcab2fedf20b5f5f /ui-tree.h | |
parent | 17596459fe9a43428a261e66f65b227d15bf7ee5 (diff) | |
download | cgit-682adbc0cad2baa1a6119013b166f52de3ee3352.zip cgit-682adbc0cad2baa1a6119013b166f52de3ee3352.tar.gz cgit-682adbc0cad2baa1a6119013b166f52de3ee3352.tar.bz2 |
scan_path(): Improve handling of inaccessible directories
When scanning a tree containing inaccessible directories (e.g. '.ssh'
directories in users' homedirs, or repos with explicitly restricted access),
scan_path() currently causes three lines of "Permissions denied" errors to be
printed to the CGI error log per inaccessible directory:
Error checking path /home/foo/.ssh: Permission denied (13)
Error checking path /home/foo/.ssh/.git: Permission denied (13)
Error opening directory /home/foo/.ssh: Permission denied (13)
This is a side-effect of calling is_git_dir(path) and
is_git_dir(fmt("%s/.git", path) _before_ we try to opendir(path).
By placing the opendir(path) before the two is_git_dir() calls, we reduce the
noise to a single line per inaccessible directory:
Error opening directory /home/foo/.ssh: Permission denied (13)
Signed-off-by: Johan Herland <johan@herland.net>
Signed-off-by: Lars Hjemli <larsh@prediktor.no>
0 files changed, 0 insertions, 0 deletions