dir-locals-find-file tweak for "odd" .dir-locals.el (bug#10928)

* lisp/files.el (dir-locals-find-file):
Ignore non-readable or non-regular files.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 76166b50c0faea8bc056d9a433bc72d5716c2c67..b4d5a1464906fbafbab65b0e80c0674cb559c891 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,5 +1,8 @@
 2012-03-06  Glenn Morris  <rgm@gnu.org>
 
+       * files.el (dir-locals-find-file):
+       Ignore non-readable or non-regular files.  (Bug#10928)
+
        * files.el (locate-dominating-file): Doc fix.
 
 2012-03-06  Adam Spiers  <emacs@adamspiers.org>  (tiny change)

diff --git a/lisp/files.el b/lisp/files.el
index d9a3e4b0f796eabb65c0176efbc8d479acd13ddd..fae834daefef72ab3a18fac8f3ab28d9fcf2afbc 100644 (file)
--- a/lisp/files.el
+++ b/lisp/files.el
@@ -910,6 +910,10 @@ it is readable, regular, etc., you should test the result."
                     ;;   (setq user (nth 2 (file-attributes file)))
                     ;;   (and prev-user (not (equal user prev-user))))
                     (string-match locate-dominating-stop-dir-regexp file)))
+      ;; FIXME? maybe this function should (optionally?)
+      ;; use file-readable-p instead.  In many cases, an unreadable
+      ;; FILE is no better than a non-existent one.
+      ;; See eg dir-locals-find-file.
       (setq try (file-exists-p (expand-file-name name file)))
       (cond (try (setq root file))
             ((equal file (setq file (file-name-directory
@@ -3569,8 +3573,14 @@ of no valid cache entry."
         (locals-file (locate-dominating-file file dir-locals-file-name))
         (dir-elt nil))
     ;; `locate-dominating-file' may have abbreviated the name.
-    (if locals-file
-       (setq locals-file (expand-file-name dir-locals-file-name locals-file)))
+    (and locals-file
+        (setq locals-file (expand-file-name dir-locals-file-name locals-file))
+        ;; FIXME? is it right to silently ignore an unreadable file?
+        ;; Maybe we'd want to keep searching in that case.
+        ;; That is a locate-dominating-file issue.
+        (or (not (file-readable-p locals-file))
+            (not (file-regular-p locals-file)))
+        (setq locals-file nil))
     ;; Find the best cached value in `dir-locals-directory-cache'.
     (dolist (elt dir-locals-directory-cache)
       (when (and (eq t (compare-strings file nil (length (car elt))