chiark / gitweb /
Make systemctl --root look for files in the proper places
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Thu, 24 Apr 2014 05:44:10 +0000 (01:44 -0400)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Thu, 15 May 2014 13:29:58 +0000 (15:29 +0200)
commit12ed81d9c88406234c20e9261ae8c8b992d8bc4d
treea18bc3403e26772f3d00c3d01c9c33cd05770e48
parent62b002337727093c21d020c730bd65971f7783a7
Make systemctl --root look for files in the proper places

Running systemctl enable/disable/set-default/... with the --root
option under strace reveals that it accessed various files and
directories in the main fs, and not underneath the specified root.
This can lead to correct results only when the layout and
configuration in the container are identical, which often is not the
case. Fix this by adding the specified root to all file access
operations.

This patch does not handle some corner cases: symlinks which point
outside of the specified root might be interpreted differently than
they would be by the kernel if the specified root was the real root.
But systemctl does not create such symlinks by itself, and I think
this is enough of a corner case not to be worth the additional
complexity of reimplementing link chasing in systemd.

Also, simplify the code in a few places and remove an hypothetical
memory leak on error.
TODO
src/core/manager.c
src/shared/install.c
src/shared/path-lookup.c
src/shared/path-lookup.h
src/shared/path-util.c
src/systemctl/systemctl.c