X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=src%2Flogin%2Flogind-acl.c;fp=src%2Flogin%2Flogind-acl.c;h=390b4b28857f77094ae63020c0d7c6f0f0a27820;hb=db0c1e3bd39c9151ba4ac8e029ec77a4ec923bd1;hp=c0ff509377f47087746a070f2c1823c92367d066;hpb=6b78df0a6ec75f25705a0f78ef895b95ab75a7ea;p=elogind.git

diff --git a/src/login/logind-acl.c b/src/login/logind-acl.c
index c0ff50937..390b4b288 100644
--- a/src/login/logind-acl.c
+++ b/src/login/logind-acl.c
@@ -183,7 +183,7 @@ int devnode_acl_all(struct udev *udev,
         Set *nodes;
         Iterator i;
         char *n;
-        DIR *dir;
+        _cleanup_closedir_ DIR *dir = NULL;
         struct dirent *dent;
         int r;
 
@@ -258,35 +258,28 @@ int devnode_acl_all(struct udev *udev,
          * these devices are not known to the kernel at this moment */
         dir = opendir("/run/udev/static_node-tags/uaccess");
         if (dir) {
-                for (dent = readdir(dir); dent != NULL; dent = readdir(dir)) {
+                FOREACH_DIRENT(dent, dir, r = -errno; goto finish) {
                         _cleanup_free_ char *unescaped_devname = NULL;
 
-                        if (dent->d_name[0] == '.')
-                                continue;
-
                         unescaped_devname = cunescape(dent->d_name);
                         if (unescaped_devname == NULL) {
                                 r = -ENOMEM;
-                                closedir(dir);
                                 goto finish;
                         }
 
                         n = strappend("/dev/", unescaped_devname);
                         if (!n) {
                                 r = -ENOMEM;
-                                closedir(dir);
                                 goto finish;
                         }
 
                         log_debug("Found static node %s for seat %s", n, seat);
                         r = set_put(nodes, n);
                         if (0 && r < 0 && r != -EEXIST) {
-                                closedir(dir);
                                 goto finish;
                         } else
                                 r = 0;
                 }
-                closedir(dir);
         }
 
         SET_FOREACH(n, nodes, i) {