diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2023-04-20 17:42:57 +0200 |
---|---|---|
committer | Mike Yuan <me@yhndnzj.com> | 2023-04-21 03:20:24 +0800 |
commit | 08c2f9c626e0f0052d505b1b7e52f335c0fbfa1d (patch) | |
tree | c432fb99ab9e236ac2a128fb5efb761b7502abf4 /src/basic/stat-util.c | |
parent | 6b8e90545e918a4653281b3672a873e948f12b65 (diff) | |
download | systemd-08c2f9c626e0f0052d505b1b7e52f335c0fbfa1d.tar.gz |
detect-virt: add message at debug level
Normal users do not have permissions to access /proc/1/root, so
'systemd-detect-virt -r' fails, but the output, even at debug level
is cryptic:
$ SYSTEMD_LOG_LEVEL=debug build/systemd-detect-virt -r
Failed to check for chroot() environment: Permission denied
Let's make this a bit easier to figure out:
$ SYSTEMD_LOG_LEVEL=debug build/systemd-detect-virt -r
Cannot stat /proc/1/root: Permission denied
Failed to check for chroot() environment: Permission denied
I looked over other users of files_same(), and I think in general the message
at debug level is OK for them too.
Diffstat (limited to 'src/basic/stat-util.c')
-rw-r--r-- | src/basic/stat-util.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/basic/stat-util.c b/src/basic/stat-util.c index e97770707f..335daca234 100644 --- a/src/basic/stat-util.c +++ b/src/basic/stat-util.c @@ -190,10 +190,10 @@ int files_same(const char *filea, const char *fileb, int flags) { assert(fileb); if (fstatat(AT_FDCWD, filea, &a, flags) < 0) - return -errno; + return log_debug_errno(errno, "Cannot stat %s: %m", filea); if (fstatat(AT_FDCWD, fileb, &b, flags) < 0) - return -errno; + return log_debug_errno(errno, "Cannot stat %s: %m", fileb); return stat_inode_same(&a, &b); } |