sssd:sssd-1-13

Last commit made on 2018-08-03
Get this branch:
git clone -b sssd-1-13 https://git.launchpad.net/sssd

Branch merges

Branch information

Name:
sssd-1-13
Repository:
lp:sssd

Recent commits

c7636e4... by Lukas Slebodnik <email address hidden>

CRYPTO: Save prefix in s3crypt_sha512

Since commit 2951a9a84bd85f384213a3e071ffc167907df2d7 where we switched from
stpncpy to memcpy the salt prefix "$6$" is not stored at all.
This broke offline authentication if someone upgraded from old version
that stored the prefix to one that doesn't store it.

Resolves:
https://pagure.io/SSSD/sssd/issue/3791

Merges: https://pagure.io/SSSD/sssd/pull-request/3792

Reviewed-by: Jakub Hrozek <email address hidden>
(cherry picked from commit f62d2af0c270a4f6143264a6cf3ce618f4a9ba80)
(cherry picked from commit 89a05bdaec89a8ee651836f11872d810bf0641fa)

5f234f2... by =?utf-8?q?Pavel_B=C5=99ezina?= <email address hidden>

sudo: allow use of default domain suffix

Reviewed-by: Pavel Březina <email address hidden>

b061451... by Jakub Hrozek <email address hidden>

SUDO: Create the socket with stricter permissions

This patch switches the sudo responder from being created as a public
responder where the permissions are open and not checked by the sssd
deaamon to a private socket. In this case, sssd creates the pipes with
strict permissions (see the umask in the call to create_pipe_fd() in
set_unix_socket()) and additionaly checks the permissions with every read
via the tevent integrations (see accept_fd_handler()).

Resolves:
https://pagure.io/SSSD/sssd/issue/3766 (CVE-2018-10852)

Reviewed-by: Sumit Bose <email address hidden>
Reviewed-by: Pavel Březina <email address hidden>

8cf14ee... by Jakub Hrozek <email address hidden>

AUTOFS: remove timed event if related object is removed

autofs_map_result_timeout() is called as a timed event to free the
autofs map data is the cache lifetime is exceeded. If the data is freed
earlier the timed event should be removed as well to avoid a double
free issue.

Since talloc is used here the most easy way to achieve this is to allocate
the timed event on the map object itself.

Resolves:
https://pagure.io/SSSD/sssd/issue/3752

Reviewed-by: Fabiano Fidêncio <email address hidden>
(cherry picked from commit 7567215ca0e180414f76469699025f281b8ac6dc)
(cherry picked from commit 1199a0d94b37d942739958d7875ae4082a2dee06)

377b622... by Sumit Bose <email address hidden>

NSS: remove timed event if related object is removed

setnetgrent_result_timeout() is called as a timed event to free the
netgr data is the cache lifetime is exceeded. If the data is freed
earlier the timed event should be removed as well to avoid a double
free issue.

Since talloc is used here the most easy way to achieve this is to
allocate the timed event on the netgr object itself.

Resolves:
https://pagure.io/SSSD/sssd/issue/3737

Reviewed-by: Fabiano Fidêncio <email address hidden>
(cherry picked from commit 83dfa4ded19c73db50b7d3b909109139c752ae28)

e51c63f... by Sumit Bose <email address hidden>

test_ca: add empty index.txt.attr file

Although is does not harm because 'openssl ca' creates the
index.tx.tattr file with a suitable content automatically this patch
adds the file to the test_CA directory to silence a message like:

Can't open ./index.txt.attr for reading, No such file or directory
139867607979840:error:02001002:system library:fopen:No such file or
directory:crypto/bio/bss_file.c:74:fopen('./index.txt.attr','r')
139867607979840:error:2006D080:BIO routines:BIO_new_file:no such
file:crypto/bio/bss_file.c:81:

which is show by recent versions of OpenSSL.

Related to https://pagure.io/SSSD/sssd/issue/3436

(cherry picked from commit 7581dbc58f9bcdab35f309d33f6ff0ebab03a343)
(cherry picked from commit fcd477aaafd9700e39d68babc1244e45e9d3f783)

Reviewed-by: Fabiano Fidêncio <email address hidden>

86dee30... by Sumit Bose <email address hidden>

TESTS: remove NSS test databases

NSS databases with the certificates from the test CA will be
automatically generated. The static databases are not needed anymore.

Related to https://pagure.io/SSSD/sssd/issue/3436

Reviewed-by: Lukáš Slebodník <email address hidden>
(cherry picked from commit cbcb2dab1ba06c65d64910b733f4480b5cf5d090
with fixes)

(cherry picked from commit aa922a0bdd6835f2538afea32391038e825fcad0
with fixes)

5c2d6dc... by Sumit Bose <email address hidden>

TESTS: replace hardcoded certificates

Since the hardcoded certificates have a limited lifetime they are
replaces by certificates from the test CA.

Related to https://pagure.io/SSSD/sssd/issue/3436

Reviewed-by: Lukáš Slebodník <email address hidden>
(cherry picked from commit 0dc7f90667df6420bc9e93ae2c8bacd6ea148f0f
with fixes)

(cherry picked from commit 40822eb1c6baad7fcc9000bcc2b9e2347002096d
with fixes)

1df9ccd... by Sumit Bose <email address hidden>

TESTS: simple CA to generate certificates for test

To avoid issue with certificate lifetimes a simple OpenSSL based CA is
used to generate certificates for tests.

To make management easy all related data is kept in
src/tests/test_CA. Since some header files will be generated the
generation of the needed files is added to BUILT_SOURCES as other
generated code.

Related to https://pagure.io/SSSD/sssd/issue/3436

Reviewed-by: Lukáš Slebodník <email address hidden>
(cherry picked from commit 19f5dd0b8dc4eff3373a0ac9ea17c2440628fd4c)
(cherry picked from commit 9458d2abb4a200989cbb6305046eabc23e670463
with fixes)

158689b... by Jakub Hrozek <email address hidden>

TESTS: Add regression tests to try if resolving root and ID 0 fails as expected

Reviewed-by: Fabiano Fidêncio <email address hidden>
(cherry picked from commit 6c3841099addb84bf3e9a2f85e96dffae1b94623)
(cherry picked from commit dd40167673d003e58b29c8478a4c8b755726da85)