[SSSD]Re: [PATCHES] ldap: skip sdap_save_grpmem() if ignore_group_members is set

2015-11-27 Thread Jakub Hrozek
On Fri, Nov 27, 2015 at 11:00:36AM +0100, Pavel Reichl wrote: > > > On 11/25/2015 01:16 PM, Sumit Bose wrote: > > > > >Hi Pavel, > > > >thank you for the suggestions. I did both changes, although I did the > >first a bit differently. I didn't like the separate return as well but > >originally

[SSSD]Re: [PATCH] AD: Add autofs provider

2015-11-27 Thread Jakub Hrozek
On Fri, Nov 27, 2015 at 11:41:16AM +0100, Jakub Hrozek wrote: > On Fri, Nov 27, 2015 at 10:48:29AM +0100, Pavel Březina wrote: > > Attached. > > > > > From 845fa8d3cf904f4d237527f371666ce4a5e21400 Mon Sep 17 00:00:00 2001 > > From: =?UTF-8?q?Pavel=20B=C5=99ezina?= > > Date:

[SSSD]Re: [PATCH] NSS: Fix memory leak netgroup

2015-11-27 Thread Pavel Reichl
On 11/27/2015 10:06 AM, Jakub Hrozek wrote: On Wed, Nov 18, 2015 at 03:06:03PM +0100, Pavel Reichl wrote: Hello, attached patch proposes solution for leaking memory when non-existing netgroup is looked up. 1st patch is just for testing - just call 'pkill -SIGUSR1 sssd_nss' and talloc

[SSSD]Re: [PATCH] IPA_PROVIDER: Explicit no handle of services

2015-11-27 Thread Jakub Hrozek
On Thu, Nov 19, 2015 at 05:57:56PM +0100, Petr Cech wrote: > On 11/12/2015 06:04 PM, Jakub Hrozek wrote: > >On Thu, Nov 12, 2015 at 04:54:21PM +0100, Petr Cech wrote: > >>>On 11/11/2015 02:42 PM, Jakub Hrozek wrote: > >Hi, I think it's a good idea to only say we don't handle services for >

[SSSD]Re: [PATCH] AD: Add autofs provider

2015-11-27 Thread Pavel Březina
On 11/27/2015 09:31 AM, Jakub Hrozek wrote: On Thu, Nov 26, 2015 at 10:57:49AM +0100, Pavel Březina wrote: On 11/25/2015 09:48 PM, Jakub Hrozek wrote: On Wed, Nov 25, 2015 at 04:18:18PM +0100, Jakub Hrozek wrote: On Wed, Nov 25, 2015 at 03:09:41PM -, Ondrej Valousek wrote: Note: man page

[SSSD]Re: [PATCHES] ldap: skip sdap_save_grpmem() if ignore_group_members is set

2015-11-27 Thread Pavel Reichl
On 11/25/2015 01:16 PM, Sumit Bose wrote: Hi Pavel, thank you for the suggestions. I did both changes, although I did the first a bit differently. I didn't like the separate return as well but originally was too lazy to rename 'fail' to 'done'. New version attached. bye, Sumit Thanks,

[SSSD]Re: [PATCH] NSS: Fix memory leak netgroup

2015-11-27 Thread Jakub Hrozek
On Wed, Nov 18, 2015 at 03:06:03PM +0100, Pavel Reichl wrote: > Hello, > > attached patch proposes solution for leaking memory when non-existing > netgroup is looked up. > > 1st patch is just for testing - just call 'pkill -SIGUSR1 sssd_nss' and > talloc report will be generated in

[SSSD]Re: [PATCH] AD: Add autofs provider

2015-11-27 Thread Jakub Hrozek
On Thu, Nov 26, 2015 at 10:57:49AM +0100, Pavel Březina wrote: > On 11/25/2015 09:48 PM, Jakub Hrozek wrote: > >On Wed, Nov 25, 2015 at 04:18:18PM +0100, Jakub Hrozek wrote: > >>On Wed, Nov 25, 2015 at 03:09:41PM -, Ondrej Valousek wrote: > >>>Note: man page says: > >>>... > >>>When autofs

[SSSD]Re: Patch for Invalid link in BUILD.txt

2015-11-27 Thread Jakub Hrozek
On Fri, Nov 20, 2015 at 01:41:36PM -0500, Dan Lavu wrote: > I think linking contribute makes more sense as it actually contains build > instructions, but I don't see any harm in adding both. > I still had trouble applying the patch from the mail, but hopefully I reconstructed it well:

[SSSD]Re: man page patch https://fedorahosted.org/sssd/ticket/2683

2015-11-27 Thread Jakub Hrozek
On Fri, Nov 20, 2015 at 01:31:10PM -0500, Dan Lavu wrote: > Arguably just 'git diff' is the git formatted way, =D > but here you go. We prefer to attach the output of git-format-patch to the e-mail :-) > > From 8b2d160abbf6eb6ff894ddb2e687983168d1da51 Mon Sep 17 00:00:00 2001 > From: Dan Lavu

[SSSD]Re: [PATCH] AD: Add autofs provider

2015-11-27 Thread Jakub Hrozek
On Fri, Nov 27, 2015 at 10:48:29AM +0100, Pavel Březina wrote: > Attached. > > From 845fa8d3cf904f4d237527f371666ce4a5e21400 Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Pavel=20B=C5=99ezina?= > Date: Fri, 27 Nov 2015 10:47:58 +0100 > Subject: [PATCH] man sssd-ad: fix typo >