summaryrefslogtreecommitdiffstats
path: root/src/shared/vpick.c
diff options
context:
space:
mode:
authorFrantisek Sumsal <frantisek@sumsal.cz>2024-02-01 14:00:48 +0100
committerFrantisek Sumsal <frantisek@sumsal.cz>2024-02-01 14:54:06 +0100
commitd049bffc50bf68aef4e12add1877e5fccfda5cca (patch)
treecf57845b2787780d90a1bba5ff9555296973359c /src/shared/vpick.c
parentvpick: make a working copy of the current dname (diff)
downloadsystemd-d049bffc50bf68aef4e12add1877e5fccfda5cca.tar.xz
systemd-d049bffc50bf68aef4e12add1877e5fccfda5cca.zip
vpick: use prefix_roota() to avoid double slash in log messages
If the toplevel_path is empty we end up with doubled leading slash, which looks weird: [ 4737.028985] testsuite-74.sh[102]: Inode '//var/lib/machines/mytree.v/mytree_37.0_arm64+2-3' has wrong type, found 'dir'. [ 4737.028985] testsuite-74.sh[102]: Failed to pick version for '/var/lib/machines/mytree.v': Is a directory ... [ 4316.957536] testsuite-74.sh[99]: Failed to open '//var/lib/machines/mytree.v/mytree_37.0': No such file or directory ...
Diffstat (limited to '')
-rw-r--r--src/shared/vpick.c14
1 files changed, 7 insertions, 7 deletions
diff --git a/src/shared/vpick.c b/src/shared/vpick.c
index 614708133b..6f15a7070d 100644
--- a/src/shared/vpick.c
+++ b/src/shared/vpick.c
@@ -159,14 +159,14 @@ static int pin_choice(
struct stat st;
if (fstat(inode_fd, &st) < 0)
- return log_debug_errno(errno, "Failed to stat discovered inode '%s/%s': %m", toplevel_path, inode_path);
+ return log_debug_errno(errno, "Failed to stat discovered inode '%s': %m", prefix_roota(toplevel_path, inode_path));
if (filter->type_mask != 0 &&
!FLAGS_SET(filter->type_mask, UINT32_C(1) << IFTODT(st.st_mode)))
return log_debug_errno(
SYNTHETIC_ERRNO(errno_from_mode(filter->type_mask, st.st_mode)),
- "Inode '%s/%s' has wrong type, found '%s'.",
- toplevel_path, inode_path,
+ "Inode '%s' has wrong type, found '%s'.",
+ prefix_roota(toplevel_path, inode_path),
inode_type_to_string(st.st_mode));
_cleanup_(pick_result_done) PickResult result = {
@@ -293,7 +293,7 @@ static int make_choice(
r = chaseat(toplevel_fd, p, CHASE_AT_RESOLVE_IN_ROOT, &object_path, &object_fd);
if (r < 0) {
if (r != -ENOENT)
- return log_debug_errno(r, "Failed to open '%s/%s': %m", toplevel_path, p);
+ return log_debug_errno(r, "Failed to open '%s': %m", prefix_roota(toplevel_path, p));
*ret = PICK_RESULT_NULL;
return 0;
@@ -318,11 +318,11 @@ static int make_choice(
/* Convert O_PATH to a regular directory fd */
dir_fd = fd_reopen(inode_fd, O_DIRECTORY|O_RDONLY|O_CLOEXEC);
if (dir_fd < 0)
- return log_debug_errno(dir_fd, "Failed to reopen '%s/%s' as directory: %m", toplevel_path, inode_path);
+ return log_debug_errno(dir_fd, "Failed to reopen '%s' as directory: %m", prefix_roota(toplevel_path, inode_path));
r = readdir_all(dir_fd, 0, &de);
if (r < 0)
- return log_debug_errno(r, "Failed to read directory '%s/%s': %m", toplevel_path, inode_path);
+ return log_debug_errno(r, "Failed to read directory '%s': %m", prefix_roota(toplevel_path, inode_path));
if (filter->architecture < 0) {
architectures = local_architectures;
@@ -466,7 +466,7 @@ static int make_choice(
object_fd = openat(dir_fd, best_filename, O_CLOEXEC|O_PATH);
if (object_fd < 0)
- return log_debug_errno(errno, "Failed to open '%s/%s': %m", toplevel_path, p);
+ return log_debug_errno(errno, "Failed to open '%s': %m", prefix_roota(toplevel_path, p));
return pin_choice(
toplevel_path,