On 08/08/2016 04:09 PM, Lukas Slebodnik wrote:
On (08/08/16 15:36), Petr Cech wrote:
On 08/08/2016 03:14 PM, Petr Cech wrote:
On 08/08/2016 02:40 PM, Lukas Slebodnik wrote:
ehlo,
The simple reproducer is to use getent passwd,group with non-existing
entry.
Without the patch you will see that "/var/lib/sss/mc/passwd" is opened
twice. Onece with mode "rw" opened by sssd_nss and the 2nd time
with "r-" opened by sssd-client (getpwnam, getpwuid, getgrnam, getgrgid)
LS
Hello Lukas,
it looks good to me. I am waiting for CI tests.
Hi Lukas,
CI failed:
http://sssd-ci.duckdns.org/logs/job/51/05/summary.html
On F24 during valgrind negcache-tests
http://sssd-ci.duckdns:.org/logs/job/51/05/fedora24/ci-build-debug/ci-make-check-valgrind.log
I didn't find more descriptive log. Is there any?
Yes,
http://sssd-ci.duckdns.org/logs/job/51/05/fedora24/ci-build-debug/src/tests/cwrap/negcache-tests.log
http://sssd-ci.duckdns.org/logs/job/51/05/fedora24/ci-build-debug/src/tests/cwrap/responder_common-tests.1122.valgrind.log
Anyway,
attached is a patch which does not require changes in unit test.
LS
Great,
this looks better to me. (LBTM?)
I am waiting for CI tests.
Regards
--
Petr^4 Čech
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org