Re: [SSSD] [PATCH] back end: periodic task API + refresh of expired records

2013-06-09 Thread Dmitri Pal
On 06/09/2013 08:05 AM, Jakub Hrozek wrote: > On Mon, Jun 03, 2013 at 10:15:49PM +0200, Pavel Březina wrote: >> On 06/03/2013 12:17 AM, Jakub Hrozek wrote: >>> On Thu, 2013-05-23 at 11:46 +0200, Pavel Březina wrote: On 05/21/2013 12:06 PM, Jakub Hrozek wrote: > In general it's not clear to

Re: [SSSD] [PATCH] back end: periodic task API + refresh of expired records

2013-06-09 Thread Jakub Hrozek
On Mon, Jun 03, 2013 at 10:15:49PM +0200, Pavel Březina wrote: > On 06/03/2013 12:17 AM, Jakub Hrozek wrote: > >On Thu, 2013-05-23 at 11:46 +0200, Pavel Březina wrote: > >>On 05/21/2013 12:06 PM, Jakub Hrozek wrote: > >>>In general it's not clear to me from the code who is responsible for > >>>free

Re: [SSSD] [PATCH] DB: Don't add empty ranges

2013-06-09 Thread Jakub Hrozek
On Sun, Jun 09, 2013 at 01:49:07PM +0200, Jakub Hrozek wrote: > On Tue, Jun 04, 2013 at 05:54:34PM +0200, Ondrej Kos wrote: > > On 06/04/2013 05:24 PM, Sumit Bose wrote: > > >if an idrange has both of these set it is invalid as well, can you add a > > >condition for this, too? > > > > > >bye, > > >

[SSSD] [PATCH] A new option krb5_use_kdcinfo

2013-06-09 Thread Jakub Hrozek
In order to test this patch, configure a domain that uses Kerberos (IPA for example) and set krb5_use_kdcinfo=False. Without this option a kdcinfo file (located in /var/lib/sss/pubconf) will be created at login time at latest. With the option set to False, the kdcinfo file will not get created and

Re: [SSSD] [PATCH] DB: Don't add empty ranges

2013-06-09 Thread Jakub Hrozek
On Tue, Jun 04, 2013 at 05:54:34PM +0200, Ondrej Kos wrote: > On 06/04/2013 05:24 PM, Sumit Bose wrote: > >if an idrange has both of these set it is invalid as well, can you add a > >condition for this, too? > > > >bye, > >Sumit > > After off-list discussion with Sumit, here's new patch. > > Ondr

[SSSD] [PATCH] man: document the need to set ldap_access_order

2013-06-09 Thread Jakub Hrozek
https://fedorahosted.org/sssd/ticket/1789 ldap_access_order must be set in order to non-default access control options to work. This patch amends the sssd-ldap man page to document this fact with all non-default ldap_access_order options. >From 28f17c871ad1119b40f352c5c8551cd8da3cb696 Mon Sep 17 0

[SSSD] [PATCH] dyndns: Fix NULL check

2013-06-09 Thread Jakub Hrozek
Found by Coverity. >From 879a78c4fe5ef4589a6b1bc53bcc74f06a66d25e Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Fri, 7 Jun 2013 07:19:27 +0200 Subject: [PATCH] dyndns: Fix NULL check --- src/providers/dp_dyndns.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/provide

Re: [SSSD] [PATCH] Don't test for NULL in nscd config check

2013-06-09 Thread Jakub Hrozek
On Fri, Jun 07, 2013 at 07:03:25PM +0200, Lukas Slebodnik wrote: > On (07/06/13 14:38), Ondrej Kos wrote: > >Hi, > > > >Attached patch fixes the following issue: > >https://fedorahosted.org/sssd/ticket/1971 > > > >This is coverity bug, CIDs: 11851,11852,11853 > > > >I tested this with NSCD configur