diff options
| author | Kay Sievers <kay@vrfy.org> | 2014-06-03 10:46:51 +0200 |
|---|---|---|
| committer | Anthony G. Basile <blueness@gentoo.org> | 2014-06-06 17:38:01 -0400 |
| commit | b493e3e7021fea152016a31f92d948e1197667b2 (patch) | |
| tree | 610f96e9686b8fefecb29858ffab7ffc480d7753 /src/udev | |
| parent | 09aadd5a149ae4c423c18938d5fc8589846a9915 (diff) | |
udev: always close lock file descriptor
https://bugs.freedesktop.org/show_bug.cgi?id=79576
Signed-off-by: Anthony G. Basile <blueness@gentoo.org>
Diffstat (limited to 'src/udev')
| -rw-r--r-- | src/udev/udevd.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/udev/udevd.c b/src/udev/udevd.c index bcbb5d4d2c..b05907edb1 100644 --- a/src/udev/udevd.c +++ b/src/udev/udevd.c @@ -301,6 +301,7 @@ static void worker_new(struct event *event) 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; + fd_lock = safe_close(fd_lock); goto skip; } } @@ -317,8 +318,7 @@ static void worker_new(struct event *event) udev_device_update_db(dev); } - if (fd_lock >= 0) - close(fd_lock); + safe_close(fd_lock); /* send processed event back to libudev listeners */ udev_monitor_send_device(worker_monitor, NULL, dev); @@ -377,10 +377,8 @@ skip: } out: udev_device_unref(dev); - if (fd_signal >= 0) - close(fd_signal); - if (fd_ep >= 0) - close(fd_ep); + safe_close(fd_signal); + safe_close(fd_ep); close(fd_inotify); close(worker_watch[WRITE_END]); udev_rules_unref(rules); |