]> git.ipfire.org Git - thirdparty/systemd.git/commit - src/test/test-fs-util.c
tree-wide: warn when a directory path already exists but has bad mode/owner/type 8552/head
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Thu, 22 Mar 2018 12:03:41 +0000 (13:03 +0100)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Fri, 23 Mar 2018 09:26:38 +0000 (10:26 +0100)
commit37c1d5e97dbc869edd8fc178427714e2d9428d2b
tree8aa80dc807df5b4452fc25094fcf5ac7a8b6e0bc
parentd50b5839b0d6f1e66860a6d8e125fd3fe96b76c8
tree-wide: warn when a directory path already exists but has bad mode/owner/type

When we are attempting to create directory somewhere in the bowels of /var/lib
and get an error that it already exists, it can be quite hard to diagnose what
is wrong (especially for a user who is not aware that the directory must have
the specified owner, and permissions not looser than what was requested). Let's
print a warning in most cases. A warning is appropriate, because such state is
usually a sign of borked installation and needs to be resolved by the adminstrator.

$ build/test-fs-util

Path "/tmp/test-readlink_and_make_absolute" already exists and is not a directory, refusing.
   (or)
Directory "/tmp/test-readlink_and_make_absolute" already exists, but has mode 0775 that is too permissive (0755 was requested), refusing.
   (or)
Directory "/tmp/test-readlink_and_make_absolute" already exists, but is owned by 1001:1000 (1000:1000 was requested), refusing.

Assertion 'mkdir_safe(tempdir, 0755, getuid(), getgid(), MKDIR_WARN_MODE) >= 0' failed at ../src/test/test-fs-util.c:320, function test_readlink_and_make_absolute(). Aborting.

No functional change except for the new log lines.
14 files changed:
src/basic/mkdir.c
src/basic/mkdir.h
src/core/execute.c
src/login/logind-dbus.c
src/login/logind-inhibit.c
src/login/logind-seat.c
src/login/logind-session.c
src/login/logind-user.c
src/machine/machine.c
src/network/networkd.c
src/nspawn/nspawn-setuid.c
src/resolve/resolved.c
src/test/test-fs-util.c
src/timesync/timesyncd.c