2014-06-03 17:17:17 +02:00
|
|
|
Based on 3d06f4183470d42361303086ed9dedd29c0ffc1b Mon Sep 17 00:00:00 2001
|
2014-06-03 16:25:18 +02:00
|
|
|
From: Kay Sievers <kay@vrfy.org>
|
|
|
|
Date: Tue, 3 Jun 2014 10:46:51 +0200
|
|
|
|
Subject: [PATCH] udev: always close lock file descriptor
|
|
|
|
|
|
|
|
https://bugs.freedesktop.org/show_bug.cgi?id=79576
|
|
|
|
---
|
2014-06-03 17:41:13 +02:00
|
|
|
src/udev/udevd.c | 7 ++++---
|
|
|
|
1 file changed, 4 insertions(+), 3 deletions(-)
|
2014-06-03 16:25:18 +02:00
|
|
|
|
|
|
|
--- src/udev/udevd.c
|
2014-06-03 17:41:13 +02:00
|
|
|
+++ src/udev/udevd.c 2014-06-03 15:34:42.778346368 +0000
|
|
|
|
@@ -301,6 +301,7 @@ static void worker_new(struct event *eve
|
2014-06-03 16:25:18 +02:00
|
|
|
if (fd_lock >= 0 && flock(fd_lock, LOCK_SH|LOCK_NB) < 0) {
|
|
|
|
log_debug("Unable to flock(%s), skipping event handling: %m", udev_device_get_devnode(d));
|
|
|
|
err = -EWOULDBLOCK;
|
2014-06-03 17:17:17 +02:00
|
|
|
+ close_nointr_nofail(fd_lock); fd_lock = -1;
|
2014-06-03 16:25:18 +02:00
|
|
|
goto skip;
|
|
|
|
}
|
|
|
|
}
|
2014-06-03 17:41:13 +02:00
|
|
|
@@ -318,7 +319,7 @@ static void worker_new(struct event *eve
|
2014-06-03 16:25:18 +02:00
|
|
|
}
|
|
|
|
|
2014-06-03 17:41:13 +02:00
|
|
|
if (fd_lock >= 0)
|
2014-06-03 16:25:18 +02:00
|
|
|
- close(fd_lock);
|
2014-06-03 17:41:13 +02:00
|
|
|
+ close_nointr_nofail(fd_lock);
|
2014-06-03 16:25:18 +02:00
|
|
|
|
|
|
|
/* send processed event back to libudev listeners */
|
|
|
|
udev_monitor_send_device(worker_monitor, NULL, dev);
|
2014-06-03 17:41:13 +02:00
|
|
|
@@ -378,9 +379,9 @@ skip:
|
2014-06-03 16:25:18 +02:00
|
|
|
out:
|
|
|
|
udev_device_unref(dev);
|
2014-06-03 17:41:13 +02:00
|
|
|
if (fd_signal >= 0)
|
2014-06-03 16:25:18 +02:00
|
|
|
- close(fd_signal);
|
2014-06-03 17:41:13 +02:00
|
|
|
+ close_nointr_nofail(fd_signal);
|
|
|
|
if (fd_ep >= 0)
|
2014-06-03 16:25:18 +02:00
|
|
|
- close(fd_ep);
|
2014-06-03 17:41:13 +02:00
|
|
|
+ close_nointr_nofail(fd_ep);
|
2014-06-03 16:25:18 +02:00
|
|
|
close(fd_inotify);
|
|
|
|
close(worker_watch[WRITE_END]);
|
|
|
|
udev_rules_unref(rules);
|