diff options
author | Lennart Poettering <lennart@poettering.net> | 2015-05-11 22:28:52 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2015-05-11 22:28:52 +0200 |
commit | 8b4305c7354f572fa0390b7580ba30af741aa0a5 (patch) | |
tree | ed8f076f3f2104a82d8b88713b349ff0c1043b3d /src/core | |
parent | f2341e0a87cab1558c84c933956e9181d5fb6c52 (diff) |
unit: move unit_warn_if_dir_nonempty() and friend to unit.c
The call is only used by the mount and automount unit types, but that's
already enough to consider it generic unit functionality, hence move it
out of mount.c and into unit.c.
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/mount.c | 46 | ||||
-rw-r--r-- | src/core/mount.h | 3 | ||||
-rw-r--r-- | src/core/unit.c | 46 | ||||
-rw-r--r-- | src/core/unit.h | 3 |
4 files changed, 49 insertions, 49 deletions
diff --git a/src/core/mount.c b/src/core/mount.c index 155cd2df1d..6501be96db 100644 --- a/src/core/mount.c +++ b/src/core/mount.c @@ -812,52 +812,6 @@ fail: mount_enter_dead(m, MOUNT_FAILURE_RESOURCES); } -void unit_warn_if_dir_nonempty(Unit *u, const char* where) { - int r; - - assert(u); - assert(where); - - r = dir_is_empty(where); - if (r > 0) - return; - if (r < 0) { - log_unit_warning_errno(u, r, "Failed to check directory %s: %m", where); - return; - } - - log_struct(LOG_NOTICE, - LOG_MESSAGE_ID(SD_MESSAGE_OVERMOUNTING), - LOG_UNIT_ID(u), - LOG_UNIT_MESSAGE(u, "Directory %s to mount over is not empty, mounting anyway.", where), - "WHERE=%s", where, - NULL); -} - -int unit_fail_if_symlink(Unit *u, const char* where) { - int r; - - assert(u); - assert(where); - - r = is_symlink(where); - if (r < 0) { - log_unit_debug_errno(u, r, "Failed to check symlink %s, ignoring: %m", where); - return 0; - } - if (r == 0) - return 0; - - log_struct(LOG_ERR, - LOG_MESSAGE_ID(SD_MESSAGE_OVERMOUNTING), - LOG_UNIT_ID(u), - LOG_UNIT_MESSAGE(u, "Mount on symlink %s not allowed.", where), - "WHERE=%s", where, - NULL); - - return -ELOOP; -} - static void mount_enter_unmounting(Mount *m) { int r; diff --git a/src/core/mount.h b/src/core/mount.h index 79cdc8dbfc..a01e6da194 100644 --- a/src/core/mount.h +++ b/src/core/mount.h @@ -128,6 +128,3 @@ MountExecCommand mount_exec_command_from_string(const char *s) _pure_; const char* mount_result_to_string(MountResult i) _const_; MountResult mount_result_from_string(const char *s) _pure_; - -void unit_warn_if_dir_nonempty(Unit *u, const char* where); -int unit_fail_if_symlink(Unit *u, const char* where); diff --git a/src/core/unit.c b/src/core/unit.c index 25d0859bf3..0a1278c2a1 100644 --- a/src/core/unit.c +++ b/src/core/unit.c @@ -3672,6 +3672,52 @@ bool unit_type_supported(UnitType t) { return unit_vtable[t]->supported(); } +void unit_warn_if_dir_nonempty(Unit *u, const char* where) { + int r; + + assert(u); + assert(where); + + r = dir_is_empty(where); + if (r > 0) + return; + if (r < 0) { + log_unit_warning_errno(u, r, "Failed to check directory %s: %m", where); + return; + } + + log_struct(LOG_NOTICE, + LOG_MESSAGE_ID(SD_MESSAGE_OVERMOUNTING), + LOG_UNIT_ID(u), + LOG_UNIT_MESSAGE(u, "Directory %s to mount over is not empty, mounting anyway.", where), + "WHERE=%s", where, + NULL); +} + +int unit_fail_if_symlink(Unit *u, const char* where) { + int r; + + assert(u); + assert(where); + + r = is_symlink(where); + if (r < 0) { + log_unit_debug_errno(u, r, "Failed to check symlink %s, ignoring: %m", where); + return 0; + } + if (r == 0) + return 0; + + log_struct(LOG_ERR, + LOG_MESSAGE_ID(SD_MESSAGE_OVERMOUNTING), + LOG_UNIT_ID(u), + LOG_UNIT_MESSAGE(u, "Mount on symlink %s not allowed.", where), + "WHERE=%s", where, + NULL); + + return -ELOOP; +} + static const char* const unit_active_state_table[_UNIT_ACTIVE_STATE_MAX] = { [UNIT_ACTIVE] = "active", [UNIT_RELOADING] = "reloading", diff --git a/src/core/unit.h b/src/core/unit.h index 094510ba5d..62257c403e 100644 --- a/src/core/unit.h +++ b/src/core/unit.h @@ -607,6 +607,9 @@ static inline bool unit_supported(Unit *u) { return unit_type_supported(u->type); } +void unit_warn_if_dir_nonempty(Unit *u, const char* where); +int unit_fail_if_symlink(Unit *u, const char* where); + const char *unit_active_state_to_string(UnitActiveState i) _const_; UnitActiveState unit_active_state_from_string(const char *s) _pure_; |