Re: [SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2009 04:26 PM, Martin Nagy wrote: > Martin Nagy wrote: >> On Tue, 2009-10-20 at 23:36 -0400, Simo Sorce wrote: >>> On Wed, 2009-10-21 at 20:21 +0200, Martin Nagy wrote: On Tue, 2009-10-20 at 22:16 -0400, Simo Sorce wrote: > Why did yo

[SSSD] [PATCH] fix offline detection/enumeration behavior in ldap_id

2009-10-21 Thread Simo Sorce
I was seeing the ipa backend blocking on start trying to contact the KDC and when failing not going offline, causing some applications (su -) to take a looong time to complete as they make many nss calls, and where waiting the KDC timeout for each call. Simo. 0001-Better-offline-enumeration-beha

Re: [SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Martin Nagy
Martin Nagy wrote: > On Tue, 2009-10-20 at 23:36 -0400, Simo Sorce wrote: > > On Wed, 2009-10-21 at 20:21 +0200, Martin Nagy wrote: > > > On Tue, 2009-10-20 at 22:16 -0400, Simo Sorce wrote: > > > > Why did you move config.h below talloc.h etc.. ? > > > > > > > > Simo. > > > > > > No functional r

[SSSD] [PATCH] Bring SSSDConfig API options up-to-date

2009-10-21 Thread Stephen Gallagher
I discovered several incorrect entries in the sssd.api.conf (and plugins) as well as several missing ones (including some added recently). I updated the unit tests accordingly. -- Stephen Gallagher RHCE 804006346421761 Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 1550233df

Re: [SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Martin Nagy
On Tue, 2009-10-20 at 23:36 -0400, Simo Sorce wrote: > On Wed, 2009-10-21 at 20:21 +0200, Martin Nagy wrote: > > On Tue, 2009-10-20 at 22:16 -0400, Simo Sorce wrote: > > > Why did you move config.h below talloc.h etc.. ? > > > > > > Simo. > > > > No functional reason, I think it's better if <> he

Re: [SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Simo Sorce
On Wed, 2009-10-21 at 20:21 +0200, Martin Nagy wrote: > On Tue, 2009-10-20 at 22:16 -0400, Simo Sorce wrote: > > Why did you move config.h below talloc.h etc.. ? > > > > Simo. > > No functional reason, I think it's better if <> headers are first and "" > headers are last so since I was changing

Re: [SSSD] [PATCHES] Fix resolv bugs and improve test coverage

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2009 12:24 PM, Martin Nagy wrote: > Hi, > the first patch breaks the test, the third one fixes them. The second > one contains couple of functions that I found handy for test cases, I > would like to use them in other tests as well sometime la

Re: [SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Martin Nagy
On Tue, 2009-10-20 at 22:16 -0400, Simo Sorce wrote: > Why did you move config.h below talloc.h etc.. ? > > Simo. No functional reason, I think it's better if <> headers are first and "" headers are last so since I was changing those lines anyway I decided to move them up as well. This shouldn't

Re: [SSSD] [PATCH] New option code for krb5 and ipa auth

2009-10-21 Thread Sumit Bose
On Tue, Oct 20, 2009 at 07:20:51PM -0400, Simo Sorce wrote: > > On Tue, 2009-10-20 at 14:24 +0200, Sumit Bose wrote: > > +dummy = dp_opt_get_cstring(opts, KRB5_CHANGEPW_PRINC); > > +if (strchr(dummy, '@') == NULL) { > > +value = talloc_asprintf(opts, "%...@%s", dummy, realm); > > +

Re: [SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Simo Sorce
Why did you move config.h below talloc.h etc.. ? Simo. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel

Re: [SSSD] [PATCH] Resolver: Use talloc_get_type() for type safety

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2009 12:19 PM, Martin Nagy wrote: > Martin > > > > ___ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://fedorahosted.org/mailman/listinfo/sssd-devel Ack. - -- Stephen Gal

Re: [SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2009 12:19 PM, Martin Nagy wrote: > Martin > > > > ___ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://fedorahosted.org/mailman/listinfo/sssd-devel Ack - -- Stephen Gall

Re: [SSSD] [PATCH] Fix potential memory leaks in the data provider

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2009 12:18 PM, Martin Nagy wrote: > Martin > > > > ___ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://fedorahosted.org/mailman/listinfo/sssd-devel Ack - -- Stephen Gall

Re: [SSSD] [PATCH] Use correct talloc context in sss_names_init()

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2009 12:17 PM, Martin Nagy wrote: > Martin > > > > ___ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://fedorahosted.org/mailman/listinfo/sssd-devel Ack. - -- Stephen Gal

[SSSD] [PATCHES] Fix resolv bugs and improve test coverage

2009-10-21 Thread Martin Nagy
Hi, the first patch breaks the test, the third one fixes them. The second one contains couple of functions that I found handy for test cases, I would like to use them in other tests as well sometime later. Look into the patch files for more detailed description. Martin >From 67f61eb0764eed07f33b82

[SSSD] [PATCH] Resolver: Use talloc_get_type() for type safety

2009-10-21 Thread Martin Nagy
Martin >From 5bf49d0fe5d1ea92a837039bbffe4a238d8bc4f0 Mon Sep 17 00:00:00 2001 From: Martin Nagy Date: Thu, 6 Aug 2009 08:55:27 +0200 Subject: [PATCH 4/7] Resolver: Use talloc_get_type() for type safety --- server/resolv/async_resolv.c | 10 +- 1 files changed, 5 insertions(+), 5 delet

[SSSD] [PATCH] Fix up header file names in util.h and add sssd-i18n.h

2009-10-21 Thread Martin Nagy
Martin >From f4e238fa95306a7ca5aee4160dfc64a29c38d331 Mon Sep 17 00:00:00 2001 From: Martin Nagy Date: Thu, 8 Oct 2009 21:18:30 +0200 Subject: [PATCH 3/7] Fix up header file names in util.h and add sssd-i18n.h --- server/util/util.h |9 ++--- 1 files changed, 6 insertions(+), 3 deletions

[SSSD] [PATCH] Fix potential memory leaks in the data provider

2009-10-21 Thread Martin Nagy
Martin >From 24ce0899d4c0d71f5e1045d726d7461fdc6e4776 Mon Sep 17 00:00:00 2001 From: Martin Nagy Date: Thu, 17 Sep 2009 18:10:10 +0200 Subject: [PATCH 2/7] Fix potential memory leaks in the data provider --- server/providers/data_provider_be.c |6 -- 1 files changed, 4 insertions(+), 2 d

[SSSD] [PATCH] Use correct talloc context in sss_names_init()

2009-10-21 Thread Martin Nagy
Martin >From f48c6acde37d24dc1cd5cb3d09df76457e8aecfa Mon Sep 17 00:00:00 2001 From: Martin Nagy Date: Tue, 13 Oct 2009 13:36:28 +0200 Subject: [PATCH 1/7] Use correct talloc context in sss_names_init() --- server/util/usertools.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) di

Re: [SSSD] [PATCH] add store/search/delete interface for custom sysdb objects

2009-10-21 Thread Simo Sorce
On Wed, 2009-10-21 at 17:15 +0200, Sumit Bose wrote: > On Tue, Oct 20, 2009 at 07:46:02PM -0400, Simo Sorce wrote: > > On Mon, 2009-10-19 at 16:42 +0200, Sumit Bose wrote: > > > Hi, > > > > > > this patch adds a store/search/delete sysdb API for data not related > > > to > > > users of groups. The

Re: [SSSD] [PATCH] add store/search/delete interface for custom sysdb objects

2009-10-21 Thread Sumit Bose
On Tue, Oct 20, 2009 at 07:46:02PM -0400, Simo Sorce wrote: > On Mon, 2009-10-19 at 16:42 +0200, Sumit Bose wrote: > > Hi, > > > > this patch adds a store/search/delete sysdb API for data not related > > to > > users of groups. The data is stored in cn=custom,cn=domain,cn=sysdb. > > The > > client

Re: [SSSD] [PATCH] add store/search/delete interface for custom sysdb objects

2009-10-21 Thread Simo Sorce
On Mon, 2009-10-19 at 16:42 +0200, Sumit Bose wrote: > Hi, > > this patch adds a store/search/delete sysdb API for data not related > to > users of groups. The data is stored in cn=custom,cn=domain,cn=sysdb. > The > client must specify a subtree_name and an object_name to save the data > in cn=obj

Re: [SSSD] [PATCH] added generic LDAP search sdap_get_generic_send/_recv

2009-10-21 Thread Simo Sorce
On Mon, 2009-10-19 at 14:45 +0200, Sumit Bose wrote: > > done, new patch attached. ack Simo. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel

Re: [SSSD] [PATCH] New option code for krb5 and ipa auth

2009-10-21 Thread Simo Sorce
On Tue, 2009-10-20 at 14:24 +0200, Sumit Bose wrote: > +dummy = dp_opt_get_cstring(opts, KRB5_CHANGEPW_PRINC); > +if (strchr(dummy, '@') == NULL) { > +value = talloc_asprintf(opts, "%...@%s", dummy, realm); > +if (value == NULL) { > +DEBUG(7, ("talloc_asprintf f

Re: [SSSD] [PATCH] New option code for krb5 and ipa auth

2009-10-21 Thread Simo Sorce
On Tue, 2009-10-20 at 14:24 +0200, Sumit Bose wrote: > + > +dummy = dp_opt_get_cstring(opts, KRB5_CCNAME_TMPL); > +if (dummy[0] != '/' && strncmp(dummy, "FILE:", 5) != 0) { > +DEBUG(1, ("Currently only file based credential caches are > supported " > + "and krb5ccn

Re: [SSSD] [PATCH] New option code for krb5 and ipa auth

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2009 09:58 AM, Sumit Bose wrote: > On Wed, Oct 21, 2009 at 08:51:21AM -0400, Stephen Gallagher wrote: >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On 10/20/2009 08:24 AM, Sumit Bose wrote: >>> Hi, >>> >>> this set of patches adds

Re: [SSSD] [PATCH] New option code for krb5 and ipa auth

2009-10-21 Thread Sumit Bose
On Wed, Oct 21, 2009 at 08:51:21AM -0400, Stephen Gallagher wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 10/20/2009 08:24 AM, Sumit Bose wrote: > > Hi, > > > > this set of patches adds the new option handling to krb5 and ipa auth: > > > > 0001: new option handling for krb5 > >

Re: [SSSD] [PATCH] New option code for krb5 and ipa auth

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/20/2009 08:24 AM, Sumit Bose wrote: > Hi, > > this set of patches adds the new option handling to krb5 and ipa auth: > > 0001: new option handling for krb5 > 0002: new option handling for ipa auth > 0003: fix a compiler warning that was introdu

Re: [SSSD] [PATCH] New option code for krb5 and ipa auth

2009-10-21 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/20/2009 08:24 AM, Sumit Bose wrote: > Hi, > > this set of patches adds the new option handling to krb5 and ipa auth: > > 0001: new option handling for krb5 > 0002: new option handling for ipa auth > 0003: fix a compiler warning that was introdu

Re: [SSSD] [PATCH] User home directories management

2009-10-21 Thread Martin Nagy
Hi, sorry, I accidentally sent the email too early. I'll continue where I left, see below. On Wed, 2009-10-21 at 06:39 +0200, Martin Nagy wrote: > On Tue, 2009-10-20 at 13:33 +0200, Jakub Hrozek wrote: > > -BEGIN PGP SIGNED MESSAGE- > > Hash: SHA1 > > > > On 10/19/2009 11:32 AM, Jakub Hro