libudev: queue - use lstat() to check existence of symlink

[elogind.git] / udev / lib / libudev-queue.c

diff --git a/udev/lib/libudev-queue.c b/udev/lib/libudev-queue.c
index 3ca533b1e0ece48a2b210512f74eab6baa9a1dfa..7ec22370104830db6fe88b5589b3bad7ad5f5029 100644 (file)
--- a/udev/lib/libudev-queue.c
+++ b/udev/lib/libudev-queue.c
@@ -34,8 +34,8 @@ struct udev_queue {
        struct udev *udev;
        int refcount;
        unsigned long long int last_seen_udev_seqnum;
-       struct list_node queue_list;
-       struct list_node failed_list;
+       struct udev_list_node queue_list;
+       struct udev_list_node failed_list;
 };
 
 struct udev_queue *udev_queue_new(struct udev *udev)
@@ -45,14 +45,13 @@ struct udev_queue *udev_queue_new(struct udev *udev)
        if (udev == NULL)
                return NULL;
 
-       udev_queue = malloc(sizeof(struct udev_queue));
+       udev_queue = calloc(1, sizeof(struct udev_queue));
        if (udev_queue == NULL)
                return NULL;
-       memset(udev_queue, 0x00, sizeof(struct udev_queue));
        udev_queue->refcount = 1;
        udev_queue->udev = udev;
-       list_init(&udev_queue->queue_list);
-       list_init(&udev_queue->failed_list);
+       udev_list_init(&udev_queue->queue_list);
+       udev_list_init(&udev_queue->failed_list);
        return udev_queue;
 }
 
@@ -71,8 +70,8 @@ void udev_queue_unref(struct udev_queue *udev_queue)
        udev_queue->refcount--;
        if (udev_queue->refcount > 0)
                return;
-       list_cleanup(udev_queue->udev, &udev_queue->queue_list);
-       list_cleanup(udev_queue->udev, &udev_queue->failed_list);
+       udev_list_cleanup_entries(udev_queue->udev, &udev_queue->queue_list);
+       udev_list_cleanup_entries(udev_queue->udev, &udev_queue->failed_list);
        free(udev_queue);
 }
 
@@ -104,7 +103,7 @@ unsigned long long int udev_queue_get_kernel_seqnum(struct udev_queue *udev_queu
                return 0;
        buf[len-1] = '\0';
        seqnum = strtoull(buf, NULL, 10);
-       info(udev_queue->udev, "seqnum=%llu\n", seqnum);
+       dbg(udev_queue->udev, "seqnum=%llu\n", seqnum);
        return seqnum;
 }
 
@@ -129,11 +128,25 @@ unsigned long long int udev_queue_get_udev_seqnum(struct udev_queue *udev_queue)
                return 0;
        buf[len-1] = '\0';
        seqnum = strtoull(buf, NULL, 10);
-       info(udev_queue->udev, "seqnum=%llu\n", seqnum);
+       dbg(udev_queue->udev, "seqnum=%llu\n", seqnum);
        udev_queue->last_seen_udev_seqnum = seqnum;
        return seqnum;
 }
 
+int udev_queue_get_udev_is_active(struct udev_queue *udev_queue)
+{
+       char filename[UTIL_PATH_SIZE];
+       struct stat statbuf;
+
+       if (udev_queue == NULL)
+               return 0;
+       util_strlcpy(filename, udev_get_dev_path(udev_queue->udev), sizeof(filename));
+       util_strlcat(filename, "/.udev/uevent_seqnum", sizeof(filename));
+       if (stat(filename, &statbuf) == 0)
+               return 1;
+       return 0;
+}
+
 int udev_queue_get_queue_is_empty(struct udev_queue *udev_queue)
 {
        char queuename[UTIL_PATH_SIZE];
@@ -145,20 +158,23 @@ int udev_queue_get_queue_is_empty(struct udev_queue *udev_queue)
        util_strlcpy(queuename, udev_get_dev_path(udev_queue->udev), sizeof(queuename));
        util_strlcat(queuename, "/.udev/queue", sizeof(queuename));
        if (stat(queuename, &statbuf) == 0) {
-               info(udev_queue->udev, "queue is not empty\n");
+               dbg(udev_queue->udev, "queue is not empty\n");
                return 0;
        }
        seqnum_kernel = udev_queue_get_kernel_seqnum(udev_queue);
        if (seqnum_kernel <= udev_queue->last_seen_udev_seqnum) {
-               info(udev_queue->udev, "queue is empty\n");
+               dbg(udev_queue->udev, "queue is empty\n");
                return 1;
        }
-       udev_queue_get_udev_seqnum(udev_queue);
+       /* update udev seqnum, and check if udev is still running */
+       if (udev_queue_get_udev_seqnum(udev_queue) == 0)
+               if (!udev_queue_get_udev_is_active(udev_queue))
+                       return 1;
        if (seqnum_kernel <= udev_queue->last_seen_udev_seqnum) {
-               info(udev_queue->udev, "queue is empty\n");
+               dbg(udev_queue->udev, "queue is empty\n");
                return 1;
        }
-       info(udev_queue->udev, "queue is empty, but kernel events still pending [%llu]<->[%llu]\n",
+       dbg(udev_queue->udev, "queue is empty, but kernel events still pending [%llu]<->[%llu]\n",
             seqnum_kernel, udev_queue->last_seen_udev_seqnum);
        return 0;
 }
@@ -178,9 +194,9 @@ int udev_queue_get_seqnum_is_finished(struct udev_queue *udev_queue, unsigned lo
        }
        snprintf(filename, sizeof(filename), "%s/.udev/queue/%llu",
                 udev_get_dev_path(udev_queue->udev), seqnum);
-       if (stat(filename, &statbuf) == 0)
+       if (lstat(filename, &statbuf) == 0)
                return 0;
-       info(udev_queue->udev, "seqnum: %llu finished\n", seqnum);
+       dbg(udev_queue->udev, "seqnum: %llu finished\n", seqnum);
        return 1;
 }
 
@@ -192,7 +208,7 @@ struct udev_list_entry *udev_queue_get_queued_list_entry(struct udev_queue *udev
 
        if (udev_queue == NULL)
                return NULL;
-       list_cleanup(udev_queue->udev, &udev_queue->queue_list);
+       udev_list_cleanup_entries(udev_queue->udev, &udev_queue->queue_list);
        util_strlcpy(path, udev_get_dev_path(udev_queue->udev), sizeof(path));
        util_strlcat(path, "/.udev/queue", sizeof(path));
        dir = opendir(path);
@@ -215,11 +231,11 @@ struct udev_list_entry *udev_queue_get_queued_list_entry(struct udev_queue *udev
                if (len < 0 || len >= (ssize_t)(sizeof(syspath)-syslen))
                        continue;
                syspath[syslen + len] = '\0';
-               info(udev_queue->udev, "found '%s' [%s]\n", syspath, dent->d_name);
-               list_entry_add(udev_queue->udev, &udev_queue->queue_list, syspath, dent->d_name, 0, 0);
+               dbg(udev_queue->udev, "found '%s' [%s]\n", syspath, dent->d_name);
+               udev_list_entry_add(udev_queue->udev, &udev_queue->queue_list, syspath, dent->d_name, 0, 0);
        }
        closedir(dir);
-       return list_get_entry(&udev_queue->queue_list);
+       return udev_list_get_entry(&udev_queue->queue_list);
 }
 
 struct udev_list_entry *udev_queue_get_failed_list_entry(struct udev_queue *udev_queue)
@@ -230,7 +246,7 @@ struct udev_list_entry *udev_queue_get_failed_list_entry(struct udev_queue *udev
 
        if (udev_queue == NULL)
                return NULL;
-       list_cleanup(udev_queue->udev, &udev_queue->failed_list);
+       udev_list_cleanup_entries(udev_queue->udev, &udev_queue->failed_list);
        util_strlcpy(path, udev_get_dev_path(udev_queue->udev), sizeof(path));
        util_strlcat(path, "/.udev/failed", sizeof(path));
        dir = opendir(path);
@@ -254,33 +270,33 @@ struct udev_list_entry *udev_queue_get_failed_list_entry(struct udev_queue *udev
                if (len < 0 || len >= (ssize_t)(sizeof(syspath)-syslen))
                        continue;
                syspath[syslen + len] = '\0';
-               info(udev_queue->udev, "found '%s' [%s]\n", syspath, dent->d_name);
+               dbg(udev_queue->udev, "found '%s' [%s]\n", syspath, dent->d_name);
                util_strlcpy(filename, syspath, sizeof(filename));
                util_strlcat(filename, "/uevent", sizeof(filename));
                if (stat(filename, &statbuf) != 0)
                        continue;
-               list_entry_add(udev_queue->udev, &udev_queue->failed_list, syspath, NULL, 0, 0);
+               udev_list_entry_add(udev_queue->udev, &udev_queue->failed_list, syspath, NULL, 0, 0);
        }
        closedir(dir);
-       return list_get_entry(&udev_queue->failed_list);
+       return udev_list_get_entry(&udev_queue->failed_list);
 }
 
-int queue_export_udev_seqnum(struct udev_queue *udev_queue, unsigned long long int seqnum)
+int udev_queue_export_udev_seqnum(struct udev_queue *udev_queue, unsigned long long int seqnum)
 {
-       return 0;
+       return -1;
 }
 
-extern int queue_export_device_queued(struct udev_queue *udev_queue, struct udev_device *udev_device)
+extern int udev_queue_export_device_queued(struct udev_queue *udev_queue, struct udev_device *udev_device)
 {
-       return 0;
+       return -1;
 }
 
-extern int queue_export_device_finished(struct udev_queue *udev_queue, struct udev_device *udev_device)
+extern int udev_queue_export_device_finished(struct udev_queue *udev_queue, struct udev_device *udev_device)
 {
-       return 0;
+       return -1;
 }
 
-extern int queue_export_device_failed(struct udev_queue *udev_queue, struct udev_device *udev_device)
+extern int udev_queue_export_device_failed(struct udev_queue *udev_queue, struct udev_device *udev_device)
 {
-       return 0;
+       return -1;
 }