Re: [SSSD] [PATCH] Use unsigned long for conversion to id_t (sssd 1.2 only)

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/18/2010 03:16 PM, Jakub Hrozek wrote: > We used strtol() on a number of places to convert into uid_t or gid_t > from a string representation such as LDAP attribute, but on some > platforms, unsigned long might be necessary to store big id_t value

Re: [SSSD] [PATCHES] Netgroups for proxy provider

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/22/2010 08:41 AM, Sumit Bose wrote: > On Fri, Oct 22, 2010 at 08:10:10AM -0400, Stephen Gallagher wrote: > On 10/20/2010 05:34 AM, Sumit Bose wrote: Hi, there two patches add netgroups support for the proxy provider as request

Re: [SSSD] [PATCHES] Netgroups for proxy provider

2010-10-22 Thread Sumit Bose
On Fri, Oct 22, 2010 at 08:10:10AM -0400, Stephen Gallagher wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 10/20/2010 05:34 AM, Sumit Bose wrote: > > Hi, > > > > there two patches add netgroups support for the proxy provider as > > requested in ticket #630. > > > > Nack. > > I

Re: [SSSD] [PATCH] Download only enabled IPA HBAC rules

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/22/2010 07:47 AM, Stephen Gallagher wrote: > On 10/20/2010 08:00 AM, Sumit Bose wrote: >> and now with a patch attached ... > >> bye, >> Sumit > >> On Wed, Oct 20, 2010 at 01:02:48PM +0200, Sumit Bose wrote: >>> Hi, >>> >>> with this patch only

Re: [SSSD] [PATCHES] Netgroups for proxy provider

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/20/2010 05:34 AM, Sumit Bose wrote: > Hi, > > there two patches add netgroups support for the proxy provider as > requested in ticket #630. > Nack. If ctx->ops.getnetgrent_r() returns an error in get_netgroup(), you're leaking memory. Typo i

Re: [SSSD] [PATCH] Download only enabled IPA HBAC rules

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/20/2010 08:00 AM, Sumit Bose wrote: > and now with a patch attached ... > > bye, > Sumit > > On Wed, Oct 20, 2010 at 01:02:48PM +0200, Sumit Bose wrote: >> Hi, >> >> with this patch only enabled IPA HABC rules are loaded and saved in >> sysdb.

Re: [SSSD] [PATCH] Add ldap_deref option

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2010 07:07 AM, Jakub Hrozek wrote: > On 10/21/2010 11:25 AM, Sumit Bose wrote: >> On Thu, Oct 21, 2010 at 11:03:06AM +0200, Jakub Hrozek wrote: >> On 10/21/2010 10:19 AM, Sumit Bose wrote: > Hi, > > as requested by ticket #568 this

Re: [SSSD] [Transifex] File submitted via email to SSSD | master

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/21/2010 12:47 PM, ad...@transifex.net wrote: > Hello sssd, this is Transifex at http://www.transifex.net. > > The following attached files were submitted to SSSD | master by yurchor > > > Please, visit Transifex at http://www.transifex.net/p

Re: [SSSD] [PATCH] Add some missing ldap_memfree()

2010-10-22 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/22/2010 06:05 AM, Jakub Hrozek wrote: > On 10/22/2010 11:57 AM, Sumit Bose wrote: >> Hi, > >> I've found a few places where we do not free the error message returned >> by ldap_parse_result(). This patch should fix this. > >> bye, >> Sumit > >

Re: [SSSD] [PATCH] Add some missing ldap_memfree()

2010-10-22 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/22/2010 11:57 AM, Sumit Bose wrote: > Hi, > > I've found a few places where we do not free the error message returned > by ldap_parse_result(). This patch should fix this. > > bye, > Sumit > Ack -BEGIN PGP SIGNATURE- Version: GnuPG v1

[SSSD] [PATCH] Add some missing ldap_memfree()

2010-10-22 Thread Sumit Bose
Hi, I've found a few places where we do not free the error message returned by ldap_parse_result(). This patch should fix this. bye, Sumit From 4b1ef24cdc4c8857cb0f7bd754c29ffeb16263f1 Mon Sep 17 00:00:00 2001 From: Sumit Bose Date: Fri, 22 Oct 2010 11:53:51 +0200 Subject: [PATCH] Add some missi