diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2017-10-03 15:34:33 +0200 |
---|---|---|
committer | Lukas Slebodnik <lslebodn@redhat.com> | 2017-10-04 06:36:07 +0200 |
commit | 82c36227e36de155b13e6eb7cfa3e80a25774157 (patch) | |
tree | 4baae2279717bcdc83bde0e08ea2596efe7ed075 | |
parent | f74408e37a3007aa41b19ab2afb693a91694da42 (diff) | |
download | sssd-82c36227e36de155b13e6eb7cfa3e80a25774157.tar.gz sssd-82c36227e36de155b13e6eb7cfa3e80a25774157.tar.xz sssd-82c36227e36de155b13e6eb7cfa3e80a25774157.zip |
intg: Fix execution with dbus-1.11.18
Since dbus-1.11.18 DBUS_COOKIE_SHA1 respect $HOME variable
and fallback to value returned from getpwnam only if env HOME
does not exist. It caused problem for dbus communication
between sssd processes because local user usually do not have
directory $HOME/.dbus-keyrings/. And directory created in cwrap
environment is problmatic
[build@host ~]$ ls -ld ~/.dbus-keyrings/
drw-------. 2 build build 6 Oct 3 10:44 /home/build/.dbus-keyrings/
[buildhost ~]$ ls -lna ~/.dbus-keyrings/
ls: cannot access '/home/build/.dbus-keyrings/.': Permission denied
ls: cannot access '/home/build/.dbus-keyrings/..': Permission denied
total 0
d????????? ? ? ? ? ? .
d????????? ? ? ? ? ? ..
[build@host ~]$ touch ~/.dbus-keyrings/test
touch: cannot touch '/home/build/.dbus-keyrings/test': Permission denied
Other alternative would be to set env variable HOME to the
same value as in fake passwd file:
HOME=$(abs_builddir)/root
Related dbus bug:
https://bugs.freedesktop.org/show_bug.cgi?id=101960
Resolves:
https://pagure.io/SSSD/sssd/issue/3531
Reviewed-by: Fabiano FidĂȘncio <fidencio@redhat.com>
-rw-r--r-- | src/tests/intg/Makefile.am | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/tests/intg/Makefile.am b/src/tests/intg/Makefile.am index 302825fc2..f30cc762c 100644 --- a/src/tests/intg/Makefile.am +++ b/src/tests/intg/Makefile.am @@ -74,6 +74,7 @@ intgcheck-installed: config.py passwd group cd "$(abs_srcdir)"; \ nss_wrapper=$$(pkg-config --libs nss_wrapper); \ uid_wrapper=$$(pkg-config --libs uid_wrapper); \ + unset HOME; \ PATH="$$(dirname -- $(SLAPD)):$$PATH" \ PATH="$(DESTDIR)$(sbindir):$(DESTDIR)$(bindir):$$PATH" \ PATH="$(abs_builddir):$(abs_srcdir):$$PATH" \ |