From 4443b1857be367b492cf74523e0842125d28ce64 Mon Sep 17 00:00:00 2001
From: Yu Watanabe <watanabe.yu+github@gmail.com>
Date: Thu, 28 Apr 2022 16:39:49 +0900
Subject: [PATCH] udev: drop unnecessary call of udev_watch_end()

As it is already called by udev_event_execute_rules().
---
 src/udev/udev-event.c | 16 ++++++----------
 src/udev/udev-event.h |  2 +-
 src/udev/udevd.c      |  4 +---
 3 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/src/udev/udev-event.c b/src/udev/udev-event.c
index 9b53105199..62c6addb9a 100644
--- a/src/udev/udev-event.c
+++ b/src/udev/udev-event.c
@@ -1151,23 +1151,19 @@ void udev_event_execute_run(UdevEvent *event, usec_t timeout_usec, int timeout_s
         }
 }
 
-int udev_event_process_inotify_watch(UdevEvent *event, int inotify_fd) {
+void udev_event_process_inotify_watch(UdevEvent *event, int inotify_fd) {
         sd_device *dev;
 
         assert(event);
         assert(inotify_fd >= 0);
 
-        dev = event->dev;
+        dev = ASSERT_PTR(event->dev);
 
-        assert(dev);
+        if (!event->inotify_watch)
+                return;
 
         if (device_for_action(dev, SD_DEVICE_REMOVE))
-                return 0;
+                return;
 
-        if (event->inotify_watch)
-                (void) udev_watch_begin(inotify_fd, dev);
-        else
-                (void) udev_watch_end(inotify_fd, dev);
-
-        return 0;
+        (void) udev_watch_begin(inotify_fd, dev);
 }
diff --git a/src/udev/udev-event.h b/src/udev/udev-event.h
index d201fb580a..74d065ce23 100644
--- a/src/udev/udev-event.h
+++ b/src/udev/udev-event.h
@@ -76,7 +76,7 @@ int udev_event_execute_rules(
                 Hashmap *properties_list,
                 UdevRules *rules);
 void udev_event_execute_run(UdevEvent *event, usec_t timeout_usec, int timeout_signal);
-int udev_event_process_inotify_watch(UdevEvent *event, int inotify_fd);
+void udev_event_process_inotify_watch(UdevEvent *event, int inotify_fd);
 
 static inline usec_t udev_warn_timeout(usec_t timeout_usec) {
         return DIV_ROUND_UP(timeout_usec, 3);
diff --git a/src/udev/udevd.c b/src/udev/udevd.c
index 25496975d5..4bb6bf3a33 100644
--- a/src/udev/udevd.c
+++ b/src/udev/udevd.c
@@ -654,9 +654,7 @@ static int worker_process_device(Manager *manager, sd_device *dev) {
                 /* in case rtnl was initialized */
                 manager->rtnl = sd_netlink_ref(udev_event->rtnl);
 
-        r = udev_event_process_inotify_watch(udev_event, manager->inotify_fd);
-        if (r < 0)
-                return r;
+        udev_event_process_inotify_watch(udev_event, manager->inotify_fd);
 
         log_device_uevent(dev, "Device processed");
         return 0;