diff options
author | Yu Watanabe <watanabe.yu+github@gmail.com> | 2023-01-09 16:11:52 +0900 |
---|---|---|
committer | Yu Watanabe <watanabe.yu+github@gmail.com> | 2023-01-10 14:25:20 +0900 |
commit | e9336d6ac346df38d96c91ba0447b3c76ee6697b (patch) | |
tree | cb5efd0c8a8bba89d854343e6ace40e31b9c6f30 /src/core/device.c | |
parent | 58b0a3e5112a27daa181383458f68955eb081551 (diff) | |
download | systemd-e9336d6ac346df38d96c91ba0447b3c76ee6697b.tar.gz |
core/device: ignore failed uevents
When udevd failed to process the device, SYSTEMD_ALIAS or any other
properties may contain invalid values. Let's refuse to handle the uevent.
Diffstat (limited to 'src/core/device.c')
-rw-r--r-- | src/core/device.c | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/src/core/device.c b/src/core/device.c index 699b134696..ec018a4c44 100644 --- a/src/core/device.c +++ b/src/core/device.c @@ -1137,6 +1137,25 @@ static int device_dispatch_io(sd_device_monitor *monitor, sd_device *dev, void * if (action == SD_DEVICE_MOVE) device_remove_old_on_move(m, dev); + /* When udevd failed to process the device, SYSTEMD_ALIAS or any other properties may contain invalid + * values. Let's refuse to handle the uevent. */ + if (sd_device_get_property_value(dev, "UDEV_WORKER_FAILED", NULL) >= 0) { + int v; + + if (device_get_property_int(dev, "UDEV_WORKER_ERRNO", &v) >= 0) + log_device_warning_errno(dev, v, "systemd-udevd failed to process the device, ignoring: %m"); + else if (device_get_property_int(dev, "UDEV_WORKER_EXIT_STATUS", &v) >= 0) + log_device_warning(dev, "systemd-udevd failed to process the device with exit status %i, ignoring.", v); + else if (device_get_property_int(dev, "UDEV_WORKER_SIGNAL", &v) >= 0) { + const char *s; + (void) sd_device_get_property_value(dev, "UDEV_WORKER_SIGNAL_NAME", &s); + log_device_warning(dev, "systemd-udevd failed to process the device with signal %i(%s), ignoring.", v, strna(s)); + } else + log_device_warning(dev, "systemd-udevd failed to process the device with unknown result, ignoring."); + + return 0; + } + /* A change event can signal that a device is becoming ready, in particular if the device is using * the SYSTEMD_READY logic in udev so we need to reach the else block of the following if, even for * change events */ |