vol_id: fix language in manpage

[elogind.git] / udev / udev-watch.c

diff --git a/udev/udev-watch.c b/udev/udev-watch.c
index 2d672e211cf4eaf3e1f7f476d74352d24857981b..e2c096af0f4d0f3459f57805815894c9c7d4c564 100644 (file)
--- a/udev/udev-watch.c
+++ b/udev/udev-watch.c
@@ -85,6 +85,7 @@ void udev_watch_restore(struct udev *udev)
                while ((ent = readdir(dir)) != NULL) {
                        char path[UTIL_PATH_SIZE];
                        char buf[UTIL_PATH_SIZE];
+                       ssize_t syslen;
                        ssize_t len;
                        struct udev_device *dev;
 
@@ -95,13 +96,13 @@ void udev_watch_restore(struct udev *udev)
                        util_strlcat(path, "/", sizeof(path));
                        util_strlcat(path, ent->d_name, sizeof(path));
 
-                       len = readlink(path, buf, sizeof(buf));
-                       if (len <= 0) {
+                       syslen = util_strlcpy(buf, udev_get_sys_path(udev), sizeof(buf));
+                       len = readlink(path, &buf[syslen], sizeof(buf)-syslen);
+                       if (len <= 0 || len >= (ssize_t)(sizeof(buf)-syslen)) {
                                unlink(path);
                                continue;
                        }
-
-                       buf[len] = '\0';
+                       buf[syslen + len] = '\0';
                        dbg(udev, "old watch to '%s' found\n", buf);
                        dev = udev_device_new_from_syspath(udev, buf);
                        if (dev == NULL) {
@@ -124,48 +125,35 @@ void udev_watch_restore(struct udev *udev)
        }
 }
 
-static const char *udev_watch_filename(struct udev *udev, int wd)
-{
-       static char filename[UTIL_PATH_SIZE];
-       char str[32];
-
-       sprintf(str, "%d", wd);
-       util_strlcpy(filename, udev_get_dev_path(udev), sizeof(filename));
-       util_strlcat(filename, "/.udev/watch/", sizeof(filename));
-       util_strlcat(filename, str, sizeof(filename));
-
-       return filename;
-}
-
 void udev_watch_begin(struct udev *udev, struct udev_device *dev)
 {
-       const char *filename;
+       char filename[UTIL_PATH_SIZE];
        int wd;
 
-       if (inotify_fd < 0 || major(udev_device_get_devnum(dev)) == 0)
+       if (inotify_fd < 0)
                return;
 
+       info(udev, "adding watch on '%s'\n", udev_device_get_devnode(dev));
        wd = inotify_add_watch(inotify_fd, udev_device_get_devnode(dev), IN_CLOSE_WRITE);
        if (wd < 0) {
                err(udev, "inotify_add_watch(%d, %s, %o) failed: %m\n",
                    inotify_fd, udev_device_get_devnode(dev), IN_CLOSE_WRITE);
        }
 
-       filename = udev_watch_filename(udev, wd);
+       snprintf(filename, sizeof(filename), "%s/.udev/watch/%d", udev_get_dev_path(udev), wd);
        util_create_path(udev, filename);
        unlink(filename);
-       symlink(udev_device_get_syspath(dev), filename);
+       symlink(udev_device_get_devpath(dev), filename);
 
        udev_device_set_watch_handle(dev, wd);
-       udev_device_update_db(dev);
 }
 
 void udev_watch_end(struct udev *udev, struct udev_device *dev)
 {
        int wd;
-       const char *filename;
+       char filename[UTIL_PATH_SIZE];
 
-       if (inotify_fd < 0 || major(udev_device_get_devnum(dev)) == 0)
+       if (inotify_fd < 0)
                return;
 
        wd = udev_device_get_watch_handle(dev);
@@ -175,27 +163,27 @@ void udev_watch_end(struct udev *udev, struct udev_device *dev)
        info(udev, "removing watch on '%s'\n", udev_device_get_devnode(dev));
        inotify_rm_watch(inotify_fd, wd);
 
-       filename = udev_watch_filename(udev, wd);
+       snprintf(filename, sizeof(filename), "%s/.udev/watch/%d", udev_get_dev_path(udev), wd);
        unlink(filename);
 
        udev_device_set_watch_handle(dev, -1);
-       udev_device_update_db(dev);
 }
 
 struct udev_device *udev_watch_lookup(struct udev *udev, int wd)
 {
-       const char *filename;
+       char filename[UTIL_PATH_SIZE];
        char buf[UTIL_PATH_SIZE];
+       ssize_t syslen;
        ssize_t len;
 
        if (inotify_fd < 0 || wd < 0)
                return NULL;
 
-       filename = udev_watch_filename(udev, wd);
-       len = readlink(filename, buf, sizeof(buf));
-       if (len > 0) {
-               buf[len] = '\0';
-
+       snprintf(filename, sizeof(filename), "%s/.udev/watch/%d", udev_get_dev_path(udev), wd);
+       syslen = util_strlcpy(buf, udev_get_sys_path(udev), sizeof(buf));
+       len = readlink(filename, &buf[syslen], sizeof(buf)-syslen);
+       if (len > 0 || len < (ssize_t)(sizeof(buf)-syslen)) {
+               buf[syslen + len] = '\0';
                return udev_device_new_from_syspath(udev, buf);
        }