chiark
/
gitweb
/
~ianmdlvl
/
elogind.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
sysv-generator: properly add Makefile symlink
[elogind.git]
/
src
/
core
/
path.c
diff --git
a/src/core/path.c
b/src/core/path.c
index cd7478f1fc179d71ed1ef3dbe79c2dbfe460f524..656ed6941d9037f7aa3acae08fc7740e6ad7da1d 100644
(file)
--- a/
src/core/path.c
+++ b/
src/core/path.c
@@
-136,7
+136,7
@@
int path_spec_watch(PathSpec *s, sd_event_io_handler_t handler) {
}
if (!exists) {
}
if (!exists) {
- log_error
(
"Failed to add watch on any of the components of %s: %m",
+ log_error
_errno(errno,
"Failed to add watch on any of the components of %s: %m",
s->path);
r = -errno; /* either EACCESS or ENOENT */
goto fail;
s->path);
r = -errno; /* either EACCESS or ENOENT */
goto fail;
@@
-157,10
+157,9
@@
void path_spec_unwatch(PathSpec *s) {
}
int path_spec_fd_event(PathSpec *s, uint32_t revents) {
}
int path_spec_fd_event(PathSpec *s, uint32_t revents) {
-
_cleanup_free_ uint8_t *buf = NULL
;
+
uint8_t buffer[INOTIFY_EVENT_MAX] _alignas_(struct inotify_event)
;
struct inotify_event *e;
struct inotify_event *e;
- ssize_t k;
- int l;
+ ssize_t l;
int r = 0;
if (revents != EPOLLIN) {
int r = 0;
if (revents != EPOLLIN) {
@@
-168,37
+167,18
@@
int path_spec_fd_event(PathSpec *s, uint32_t revents) {
return -EINVAL;
}
return -EINVAL;
}
- if (ioctl(s->inotify_fd, FIONREAD, &l) < 0) {
- log_error("FIONREAD failed: %m");
- return -errno;
- }
-
- assert(l > 0);
+ l = read(s->inotify_fd, buffer, sizeof(buffer));
+ if (l < 0) {
+ if (errno == EAGAIN || errno == EINTR)
+ return 0;
- buf = malloc(l);
- if (!buf)
- return log_oom();
-
- k = read(s->inotify_fd, buf, l);
- if (k < 0) {
- log_error("Failed to read inotify event: %m");
- return -errno;
+ return log_error_errno(errno, "Failed to read inotify event: %m");
}
}
- e = (struct inotify_event*) buf;
-
- while (k > 0) {
- size_t step;
-
+ FOREACH_INOTIFY_EVENT(e, buffer, l) {
if ((s->type == PATH_CHANGED || s->type == PATH_MODIFIED) &&
s->primary_wd == e->wd)
r = 1;
if ((s->type == PATH_CHANGED || s->type == PATH_MODIFIED) &&
s->primary_wd == e->wd)
r = 1;
-
- step = sizeof(struct inotify_event) + e->len;
- assert(step <= (size_t) k);
-
- e = (struct inotify_event*) ((uint8_t*) e + step);
- k -= step;
}
return r;
}
return r;
@@
-250,7
+230,7
@@
static void path_spec_mkdir(PathSpec *s, mode_t mode) {
r = mkdir_p_label(s->path, mode);
if (r < 0)
r = mkdir_p_label(s->path, mode);
if (r < 0)
- log_warning_errno(
-
r, "mkdir(%s) failed: %m", s->path);
+ log_warning_errno(r, "mkdir(%s) failed: %m", s->path);
}
static void path_spec_dump(PathSpec *s, FILE *f, const char *prefix) {
}
static void path_spec_dump(PathSpec *s, FILE *f, const char *prefix) {
@@
-562,8
+542,7
@@
static void path_enter_waiting(Path *p, bool initial, bool recheck) {
return;
fail:
return;
fail:
- log_warning("%s failed to enter waiting state: %s",
- UNIT(p)->id, strerror(-r));
+ log_warning_errno(r, "%s failed to enter waiting state: %m", UNIT(p)->id);
path_enter_dead(p, PATH_FAILURE_RESOURCES);
}
path_enter_dead(p, PATH_FAILURE_RESOURCES);
}