Re: [SSSD] [PATCH] nss_check_name_of_well_known_sid() improve name splitting

2015-07-20 Thread Lukas Slebodnik
On (15/07/15 21:11), Jakub Hrozek wrote: On Wed, Jul 15, 2015 at 07:27:08PM +0200, Lukas Slebodnik wrote: On (15/07/15 10:34), Sumit Bose wrote: On Tue, Jul 14, 2015 at 04:33:58PM +0200, Lukas Slebodnik wrote: On (14/07/15 15:02), Sumit Bose wrote: Hi, this patch should fix

Re: [SSSD] [PATCH] nss_check_name_of_well_known_sid() improve name splitting

2015-07-16 Thread Jakub Hrozek
On Wed, Jul 15, 2015 at 09:11:55PM +0200, Jakub Hrozek wrote: On Wed, Jul 15, 2015 at 07:27:08PM +0200, Lukas Slebodnik wrote: On (15/07/15 10:34), Sumit Bose wrote: On Tue, Jul 14, 2015 at 04:33:58PM +0200, Lukas Slebodnik wrote: On (14/07/15 15:02), Sumit Bose wrote: Hi, this

Re: [SSSD] [PATCH] nss_check_name_of_well_known_sid() improve name splitting

2015-07-15 Thread Sumit Bose
On Tue, Jul 14, 2015 at 04:33:58PM +0200, Lukas Slebodnik wrote: On (14/07/15 15:02), Sumit Bose wrote: Hi, this patch should fix https://fedorahosted.org/sssd/ticket/2717 . As you can see I added a new entry ipa_ad_default_names to the global nss context so that the regular expression

Re: [SSSD] [PATCH] nss_check_name_of_well_known_sid() improve name splitting

2015-07-15 Thread Jakub Hrozek
On Wed, Jul 15, 2015 at 07:27:08PM +0200, Lukas Slebodnik wrote: On (15/07/15 10:34), Sumit Bose wrote: On Tue, Jul 14, 2015 at 04:33:58PM +0200, Lukas Slebodnik wrote: On (14/07/15 15:02), Sumit Bose wrote: Hi, this patch should fix https://fedorahosted.org/sssd/ticket/2717 . As

Re: [SSSD] [PATCH] nss_check_name_of_well_known_sid() improve name splitting

2015-07-15 Thread Lukas Slebodnik
On (15/07/15 10:34), Sumit Bose wrote: On Tue, Jul 14, 2015 at 04:33:58PM +0200, Lukas Slebodnik wrote: On (14/07/15 15:02), Sumit Bose wrote: Hi, this patch should fix https://fedorahosted.org/sssd/ticket/2717 . As you can see I added a new entry ipa_ad_default_names to the global nss

[SSSD] [PATCH] nss_check_name_of_well_known_sid() improve name splitting

2015-07-14 Thread Sumit Bose
Hi, this patch should fix https://fedorahosted.org/sssd/ticket/2717 . As you can see I added a new entry ipa_ad_default_names to the global nss context so that the regular expression string is only evaluated once. Since it is currently only used in nss_check_name_of_well_known_sid() I do the

Re: [SSSD] [PATCH] nss_check_name_of_well_known_sid() improve name splitting

2015-07-14 Thread Lukas Slebodnik
On (14/07/15 15:02), Sumit Bose wrote: Hi, this patch should fix https://fedorahosted.org/sssd/ticket/2717 . As you can see I added a new entry ipa_ad_default_names to the global nss context so that the regular expression string is only evaluated once. Since it is currently only used in