chiark
/
gitweb
/
~ianmdlvl
/
elogind.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
coredump: simplify a few things by allocating small fields on the stack rather than...
[elogind.git]
/
src
/
core
/
load-dropin.c
diff --git
a/src/core/load-dropin.c
b/src/core/load-dropin.c
index 21c991526c7fdb804c3427e18eb7c6574cf01765..8afaf45fe6acb1a55603341e7c57dc0584130a98 100644
(file)
--- a/
src/core/load-dropin.c
+++ b/
src/core/load-dropin.c
@@
-58,22
+58,18
@@
static int iterate_dir(
if (errno == ENOENT)
return 0;
if (errno == ENOENT)
return 0;
- log_error
(
"Failed to open directory %s: %m", path);
+ log_error
_errno(errno,
"Failed to open directory %s: %m", path);
return -errno;
}
for (;;) {
struct dirent *de;
_cleanup_free_ char *f = NULL;
return -errno;
}
for (;;) {
struct dirent *de;
_cleanup_free_ char *f = NULL;
- int k;
errno = 0;
de = readdir(d);
errno = 0;
de = readdir(d);
- if (!de && errno != 0) {
- k = errno;
- log_error("Failed to read directory %s: %s", path, strerror(k));
- return -k;
- }
+ if (!de && errno != 0)
+ return log_error_errno(errno, "Failed to read directory %s: %m", path);
if (!de)
break;
if (!de)
break;
@@
-87,7
+83,7
@@
static int iterate_dir(
r = unit_add_dependency_by_name(u, dependency, de->d_name, f, true);
if (r < 0)
r = unit_add_dependency_by_name(u, dependency, de->d_name, f, true);
if (r < 0)
- log_error
("Cannot add dependency %s to %s, ignoring: %s", de->d_name, u->id, strerror(-r)
);
+ log_error
_errno(r, "Cannot add dependency %s to %s, ignoring: %m", de->d_name, u->id
);
}
return 0;
}
return 0;
@@
-155,7
+151,7
@@
char **unit_find_dropin_paths(Unit *u) {
r = conf_files_list_strv(&configs, ".conf", NULL, (const char**) strv);
if (r < 0) {
r = conf_files_list_strv(&configs, ".conf", NULL, (const char**) strv);
if (r < 0) {
- log_error
("Failed to get list of configuration files: %s", strerror(-r)
);
+ log_error
_errno(r, "Failed to get list of configuration files: %m"
);
strv_free(configs);
return NULL;
}
strv_free(configs);
return NULL;
}
@@
-181,7
+177,7
@@
int unit_load_dropin(Unit *u) {
}
u->dropin_paths = unit_find_dropin_paths(u);
}
u->dropin_paths = unit_find_dropin_paths(u);
- if (!
u->dropin_paths)
+ if (!u->dropin_paths)
return 0;
STRV_FOREACH(f, u->dropin_paths) {
return 0;
STRV_FOREACH(f, u->dropin_paths) {