[SSSD] Re: [PATCH] REFACTOR: global_talloc_context push/pop remove

2016-04-12 Thread Lukas Slebodnik
On (12/04/16 07:50), Petr Cech wrote: >On 04/11/2016 08:51 AM, Lukas Slebodnik wrote: >>On (11/04/16 06:34), Petr Cech wrote: >>>Hello list, >>> >>>push/pop global_talloc_context into check_leaks is redudant. It is done >>>in leak_check_setup() and leak_check_teardown() functions in >>>src/tests/ch

[SSSD] Re: [PATCH] REFACTOR: global_talloc_context push/pop remove

2016-04-12 Thread Lukas Slebodnik
On (12/04/16 09:58), Lukas Slebodnik wrote: >On (12/04/16 07:50), Petr Cech wrote: >>On 04/11/2016 08:51 AM, Lukas Slebodnik wrote: >>>On (11/04/16 06:34), Petr Cech wrote: Hello list, push/pop global_talloc_context into check_leaks is redudant. It is done in leak_check_setup() and

[SSSD] [PATCHES] sss_override: do not generate DN, search object

2016-04-12 Thread Sumit Bose
Hi, I'm working on adding certificates to overrides and came across some issues with the local overrides. The main cause was that instead you searching the original object in the cache, the DN of the object was constructed. This failed for users form sub-domain which have the fully-qualified name

[SSSD] Re: [PATCH] IPA: Check RDN in ipa_add_ad_memberships_get_next

2016-04-12 Thread Pavel Březina
On 04/11/2016 05:58 PM, Lukas Slebodnik wrote: On (11/04/16 15:01), Lukas Slebodnik wrote: ehlo, attached patch fix crash in #2980 LS From 422abe6e6263c3c907611a8611fa3f28d6e93ae0 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Mon, 11 Apr 2016 14:46:47 +0200 Subject: [PATCH] IPA: Che

[SSSD] Re: [PATCH] IPA: Check RDN in ipa_add_ad_memberships_get_next

2016-04-12 Thread Sumit Bose
On Tue, Apr 12, 2016 at 11:08:54AM +0200, Pavel Březina wrote: > On 04/11/2016 05:58 PM, Lukas Slebodnik wrote: > >On (11/04/16 15:01), Lukas Slebodnik wrote: > >>ehlo, > >> > >>attached patch fix crash in #2980 > >> > >>LS > > > >>From 422abe6e6263c3c907611a8611fa3f28d6e93ae0 Mon Sep 17 00:00:00 2

[SSSD] Re: [PATCH] IPA: Check RDN in ipa_add_ad_memberships_get_next

2016-04-12 Thread Lukas Slebodnik
On (12/04/16 11:08), Pavel Březina wrote: >On 04/11/2016 05:58 PM, Lukas Slebodnik wrote: >>On (11/04/16 15:01), Lukas Slebodnik wrote: >>>ehlo, >>> >>>attached patch fix crash in #2980 >>> >>>LS >> >>>From 422abe6e6263c3c907611a8611fa3f28d6e93ae0 Mon Sep 17 00:00:00 2001 >>>From: Lukas Slebodnik

[SSSD] Re: [PATCH] IPA: Check RDN in ipa_add_ad_memberships_get_next

2016-04-12 Thread Lukas Slebodnik
On (12/04/16 12:07), Sumit Bose wrote: >On Tue, Apr 12, 2016 at 11:08:54AM +0200, Pavel Březina wrote: >> On 04/11/2016 05:58 PM, Lukas Slebodnik wrote: >> >On (11/04/16 15:01), Lukas Slebodnik wrote: >> >>ehlo, >> >> >> >>attached patch fix crash in #2980 >> >> >> >>LS >> > >> >>From 422abe6e6263c

[SSSD] Re: Design document - sssctl

2016-04-12 Thread Jakub Hrozek
On Tue, Mar 22, 2016 at 12:42:28PM +0100, Pavel Reichl wrote: > Hello, > > Pavel Březina and I have prepared the 1st draft of design document. We mostly > focused on summing up its future functionality and its interface. > > Please comment if you miss some essential functionality or if you would

[SSSD] Re: [PATCHES] PAC: only save PAC blob into the cache

2016-04-12 Thread Pavel Březina
On 04/01/2016 01:12 PM, Pavel Březina wrote: On 03/18/2016 05:09 PM, Sumit Bose wrote: On Tue, Mar 15, 2016 at 04:50:48PM +0100, Lukas Slebodnik wrote: On (15/03/16 10:04), Pavel Březina wrote: On 03/14/2016 12:12 PM, Sumit Bose wrote: Hi, this patch-set should solve https://fedorahosted.org

[SSSD] Re: Design document - sssctl

2016-04-12 Thread Pavel Březina
On 04/12/2016 12:52 PM, Jakub Hrozek wrote: On Tue, Mar 22, 2016 at 12:42:28PM +0100, Pavel Reichl wrote: Hello, Pavel Březina and I have prepared the 1st draft of design document. We mostly focused on summing up its future functionality and its interface. Please comment if you miss some esse

[SSSD] Re: Design document - sssctl

2016-04-12 Thread Pavel Březina
On 04/06/2016 10:54 AM, Jakub Hrozek wrote: On Tue, Mar 22, 2016 at 12:42:28PM +0100, Pavel Reichl wrote: Hello, Pavel Březina and I have prepared the 1st draft of design document. We mostly focused on summing up its future functionality and its interface. Please comment if you miss some esse

[SSSD] Re: Design document - sssctl

2016-04-12 Thread Jakub Hrozek
On Tue, Apr 12, 2016 at 01:01:03PM +0200, Pavel Březina wrote: > On 04/12/2016 12:52 PM, Jakub Hrozek wrote: > >On Tue, Mar 22, 2016 at 12:42:28PM +0100, Pavel Reichl wrote: > >>Hello, > >> > >>Pavel Březina and I have prepared the 1st draft of design document. We > >>mostly focused on summing up

[SSSD] Re: Design document - sssctl

2016-04-12 Thread Jakub Hrozek
On Tue, Apr 12, 2016 at 01:02:30PM +0200, Pavel Březina wrote: > On 04/06/2016 10:54 AM, Jakub Hrozek wrote: > >On Tue, Mar 22, 2016 at 12:42:28PM +0100, Pavel Reichl wrote: > >>Hello, > >> > >>Pavel Březina and I have prepared the 1st draft of design document. We > >>mostly focused on summing up

[SSSD] Re: [PATCH SET] TOOL: Invalidation of sudo rules at sss_cache

2016-04-12 Thread Pavel Březina
On 04/06/2016 04:55 PM, Petr Cech wrote: On 03/31/2016 03:29 PM, Pavel Březina wrote: On 03/29/2016 03:17 PM, Petr Cech wrote: Hi, I am going to take three days of PTO. There is my work. Tests don't leak and they really test :-) It is possible to use compact form of them, maybe more help func

[SSSD] Re: [PATCH] GPO: Process GPOS in offline mode if ldap search failed

2016-04-12 Thread Jakub Hrozek
On Mon, Apr 11, 2016 at 12:49:35PM +0200, Lukas Slebodnik wrote: > ehlo, > > attached patch should fix #2964. > BTW it worked previously because there was a bug in marking > domain as offline. > > So this bug was revealed with fix of another bug :-) > 0561d532cf76b035b73cfed929a6896071dac407 > >

[SSSD] [PATCH] dp_ptask: Fix memory leak in synchronous ptask

2016-04-12 Thread Lukas Slebodnik
ehlo, the 1st patch fixes memory leak and the second one is change in unit test to prevent such leaks in be_ptask in feature. LS >From 97196e2df84ec84f9f047c20e125017ba67aa32e Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Tue, 12 Apr 2016 12:41:50 +0200 Subject: [PATCH 1/3] dp_ptask: Fix

[SSSD] Re: [PATCH] GPO: Process GPOS in offline mode if ldap search failed

2016-04-12 Thread Jakub Hrozek
On Tue, Apr 12, 2016 at 02:10:55PM +0200, Jakub Hrozek wrote: > On Mon, Apr 11, 2016 at 12:49:35PM +0200, Lukas Slebodnik wrote: > > ehlo, > > > > attached patch should fix #2964. > > BTW it worked previously because there was a bug in marking > > domain as offline. > > > > So this bug was reveal

[SSSD] Re: [PATCH] GPO: Process GPOS in offline mode if ldap search failed

2016-04-12 Thread Lukas Slebodnik
On (12/04/16 14:51), Jakub Hrozek wrote: >On Tue, Apr 12, 2016 at 02:10:55PM +0200, Jakub Hrozek wrote: >> On Mon, Apr 11, 2016 at 12:49:35PM +0200, Lukas Slebodnik wrote: >> > ehlo, >> > >> > attached patch should fix #2964. >> > BTW it worked previously because there was a bug in marking >> > do

[SSSD] Re: [PATCH] AD: Recognize Windows Server 2016

2016-04-12 Thread Lukas Slebodnik
On (07/04/16 11:55), Jakub Hrozek wrote: >On Thu, Apr 07, 2016 at 09:12:17AM +0200, Lukas Slebodnik wrote: >> On (06/04/16 15:30), Jakub Hrozek wrote: >> >Hi, >> > >> >I was doing a smoke-test with Windows Server 2016 after receiving >> >several requests from users (mainly about IDMU deprecation) a

[SSSD] Re: [PATCH] memberof: Fix a memory leak when removing ghost users

2016-04-12 Thread Jakub Hrozek
On Fri, Apr 08, 2016 at 11:52:18AM +0200, Jakub Hrozek wrote: > From c69aff1284c72d1e5bcb633c2499e9fb74f16335 Mon Sep 17 00:00:00 2001 > From: Jakub Hrozek > Date: Wed, 6 Apr 2016 18:35:39 +0200 > Subject: [PATCH 1/3] memberof: Fix a memory leak when removing ghost users > From 660413914a9f3061d5

[SSSD] Re: Config file merging in SSSD

2016-04-12 Thread Jakub Hrozek
On Tue, Mar 22, 2016 at 04:19:48PM +0100, Michal Židek wrote: > On 03/22/2016 03:29 PM, Sumit Bose wrote: > >On Tue, Mar 22, 2016 at 12:29:39PM +0100, Michal Židek wrote: > >>Hi, > >> > >>I would like to write a patch that will > >>allow SSSD to use the config file merging > >>feature from libini.

[SSSD] Re: [PATCH] sudo: use cache_req interface

2016-04-12 Thread Jakub Hrozek
On Fri, Apr 08, 2016 at 01:05:54PM +0200, Jakub Hrozek wrote: > On Fri, Apr 08, 2016 at 12:15:25PM +0200, Pavel Březina wrote: > > On 03/01/2016 01:53 PM, Pavel Březina wrote: > > >On 02/25/2016 02:08 PM, Jakub Hrozek wrote: > > >>On Tue, Feb 09, 2016 at 02:07:21PM +0100, Pavel Březina wrote: > > >

[SSSD] Re: [PATCH SET] TOOL: Invalidation of sudo rules at sss_cache

2016-04-12 Thread Petr Cech
On 04/12/2016 01:27 PM, Pavel Březina wrote: On 04/06/2016 04:55 PM, Petr Cech wrote: On 03/31/2016 03:29 PM, Pavel Březina wrote: On 03/29/2016 03:17 PM, Petr Cech wrote: Hi, I am going to take three days of PTO. There is my work. Tests don't leak and they really test :-) It is possible t

[SSSD] Re: [PATCH] IPA: Check RDN in ipa_add_ad_memberships_get_next

2016-04-12 Thread Sumit Bose
On Tue, Apr 12, 2016 at 12:46:00PM +0200, Lukas Slebodnik wrote: > On (12/04/16 12:07), Sumit Bose wrote: > >On Tue, Apr 12, 2016 at 11:08:54AM +0200, Pavel Březina wrote: > >> On 04/11/2016 05:58 PM, Lukas Slebodnik wrote: > >> >On (11/04/16 15:01), Lukas Slebodnik wrote: > >> >>ehlo, > >> >> > >>

[SSSD] Re: [PATCH] IPA: Check RDN in ipa_add_ad_memberships_get_next

2016-04-12 Thread Lukas Slebodnik
On (12/04/16 17:52), Sumit Bose wrote: >On Tue, Apr 12, 2016 at 12:46:00PM +0200, Lukas Slebodnik wrote: >> On (12/04/16 12:07), Sumit Bose wrote: >> >On Tue, Apr 12, 2016 at 11:08:54AM +0200, Pavel Březina wrote: >> >> On 04/11/2016 05:58 PM, Lukas Slebodnik wrote: >> >> >On (11/04/16 15:01), Luka