sssd-devel September 2012

sssd-devel@lists.fedorahosted.org

16 participants
77 discussions

[PATCH] NSS: Fix off-by-one error in parse_getservbyname
by Jakub Hrozek 13 Sep '12

13 Sep '12

https://fedorahosted.org/sssd/ticket/1438 This off-by-one bug would hit us especially when using the negative cache, I guess it was because we would strdup and later retrieve something else than intended.. The same bug does not exist in the similar parse_getservbyport function. I managed to reproduce the bug with by queriying the SSSD for services quite fast. The services/protocols were based on data from the crash dump: for s in smtp/dccp rpcbind/dccp rpcbind/udplite rpcbind/sctp rpcbind/sctp 47117/udp rpcbind/dccp; do getent services $s; sleep 1; done

2 2

[PATCHES] backend: initialize sudo only when it is enabled in services
by Pavel Březina 12 Sep '12

12 Sep '12

0001 fixes https://fedorahosted.org/sssd/ticket/1458 0002 fixes a memory leak when be_process_init() fails. I think it should be fixed, even though the backend is in this case immediately terminated

3 9

[PATCH] netgroup: resolve hostgroup membership correctly
by Pavel Březina 12 Sep '12

12 Sep '12

See the commit message.

2 2

[PATCH] KRB5: Add a missing string argument
by Jakub Hrozek 12 Sep '12

12 Sep '12

I noticed this while debugging a rhbz. Pushed under the one-liner rule.

1 0

[PATCH] Remove obsolete comment
by Simo Sorce 12 Sep '12

12 Sep '12

okos on IRC pointed out this comment does not reflect the code anymore. The comment is old and need to be removed as the logic it refers to has been removed. Simo. -- Simo Sorce * Red Hat, Inc * New York

3 2

[PATCH] tools_util.h provides signal_sssd function.
by Michal Židek 10 Sep '12

10 Sep '12

I was looking for some reusable code and found static function send_sighup in sss_debuglevel.c. This function can be useful in other tools as well, so I renamed it to signal_sssd, added signum parameter and put it to tools_util.c/h. It can be used to send any signal to sssd monitor. Patch is in attachment. Thanks Michal

2 1

[PATCH] RPM: BuildRequire selinux-policy-targeted
by Jakub Hrozek 10 Sep '12

10 Sep '12

We check the presence of the directory /etc/selinux/targeted/logins to determine if the platform supports per-user SELinux logins. However, I forgot to BuildRequire that directory, so RPMs built in mock failed to detect it completely unlike my test RPMs built locally. The other alternative I was considering was a simple switch that would be disabled on old platforms and enabled on new platforms but a) I don't lke the idea of yet another switch and b) I'd like the feature to work correctly on all platforms automatically, even on other distributions.

2 3

[PATCH] SYSDB: NULL-terminate the output of sysdb_get_{ranges, subdomains}
by Jakub Hrozek 10 Sep '12

10 Sep '12

I noticed (using valgrind) that the sysdb tests access memory outside the array allocated as a result of the functions in subject. The attached patch NULL-terminates the array so that it's safe to loop over the results.

2 4

[PATCH] KRB5: cancel the sysdb transaction on one place only
by Jakub Hrozek 10 Sep '12

10 Sep '12

Found by Coverity.

2 2

[PATCH] KRB5: Return PAM_AUTH_ERR on incorrect password
by Jakub Hrozek 10 Sep '12

10 Sep '12

https://fedorahosted.org/sssd/ticket/1515 The patch fixes handling of wrong password during both auth and chpass, in the chpass case, the return value was fixed in general.

2 2

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

sssd-devel September 2012