Re: [SSSD] [PATCH] PAC: do not expect that sysdb_search_object_by_sid() return ENOENT

2013-06-18 Thread Simo Sorce
On Tue, 2013-06-18 at 19:45 +0200, Jakub Hrozek wrote: > On Tue, Jun 18, 2013 at 05:30:09PM +0200, Sumit Bose wrote: > > Hi, > > > > one of the issues reported in > > https://bugzilla.redhat.com/show_bug.cgi?id=975001 is a core dump in the > > PAC responder. I thought that I can find missing resul

Re: [SSSD] [PATCH] PAC: do not expect that sysdb_search_object_by_sid() return ENOENT

2013-06-18 Thread Jakub Hrozek
On Tue, Jun 18, 2013 at 05:30:09PM +0200, Sumit Bose wrote: > Hi, > > one of the issues reported in > https://bugzilla.redhat.com/show_bug.cgi?id=975001 is a core dump in the > PAC responder. I thought that I can find missing results by checking for > ENOENT which is wrong in this case. This patch

[SSSD] [PATCH] PAC: do not expect that sysdb_search_object_by_sid() return ENOENT

2013-06-18 Thread Sumit Bose
Hi, one of the issues reported in https://bugzilla.redhat.com/show_bug.cgi?id=975001 is a core dump in the PAC responder. I thought that I can find missing results by checking for ENOENT which is wrong in this case. This patch properly checks the number of results. bye, Sumit From c6943b2331a3d2a

Re: [SSSD] [PATCH] Add ldap_autofs_map_master_name option

2013-06-18 Thread Lukas Slebodnik
On (18/06/13 11:15), Jakub Hrozek wrote: >On Tue, Jun 18, 2013 at 10:29:36AM +0200, Lukas Slebodnik wrote: >> On (17/06/13 19:47), Jakub Hrozek wrote: >> >On Mon, Jun 17, 2013 at 05:28:17PM +0200, Lukas Slebodnik wrote: >> >> On (03/06/13 22:41), Jakub Hrozek wrote: >> >> >On Sat, Jun 01, 2013 at 1

Re: [SSSD] [PATCHES] SSH: When host is removed from LDAP remove it from the, cache as well

2013-06-18 Thread Jan Cholasta
On 12.6.2013 11:46, Pavel Březina wrote: +if (state->count != 1) { +ret = sysdb_delete_ssh_host(state->domain->sysdb, state->domain, +state->name); +if (ret != EOK && ret != ENOENT) { +goto done; +} + +if (state->

Re: [SSSD] [PATCH] Use meta server if there isn't another server after meta.

2013-06-18 Thread Pavel Březina
On 06/10/2013 02:22 PM, Lukas Slebodnik wrote: On (10/06/13 10:54), Pavel Březina wrote: On 06/10/2013 10:32 AM, Pavel Březina wrote: On 06/03/2013 10:28 PM, Jakub Hrozek wrote: On Mon, Jun 03, 2013 at 03:54:34PM +0200, Lukas Slebodnik wrote: On (03/06/13 12:50), Lukas Slebodnik wrote: ehlo

[SSSD] [PATCHES] fix SRV expansion

2013-06-18 Thread Pavel Březina
https://fedorahosted.org/sssd/ticket/1947 A fast explanation how _srv_ expansion works. _srv_ is inserted into server list as so called meta server. Let us consider following configuration: *Setup* ipa_server = _srv_, ipa.pb server list will contain: meta -> ipa.pb *Expansion* meta -> ipa.pb

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/18/2013 06:12 AM, Jakub Hrozek wrote: > On Tue, Jun 18, 2013 at 12:05:44PM +0200, steve wrote: >> You should not run autoreconf on the tarball. If you do, it will >> not build. > > That was a bug that should be fixed now. As a matter of fact, we

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread Jakub Hrozek
On Tue, Jun 18, 2013 at 12:05:44PM +0200, steve wrote: > On 18/06/13 11:54, Jakub Hrozek wrote: > >On Tue, Jun 18, 2013 at 09:43:04AM +0200, steve wrote: > >>On Tue, 2013-06-18 at 09:02 +0200, Jakub Hrozek wrote: > >>>On Fri, Apr 12, 2013 at 10:58:54AM -0500, Ariel Barria wrote: > done. > T

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread steve
On 18/06/13 11:54, Jakub Hrozek wrote: On Tue, Jun 18, 2013 at 09:43:04AM +0200, steve wrote: On Tue, 2013-06-18 at 09:02 +0200, Jakub Hrozek wrote: On Fri, Apr 12, 2013 at 10:58:54AM -0500, Ariel Barria wrote: done. Thank you for comments. I must admit I forgot about this patch (it wasn't e

Re: [SSSD] [PATCH] BUILD.txt: Add samba-devel to dependencies for build

2013-06-18 Thread Jakub Hrozek
On Fri, Jun 14, 2013 at 02:06:03PM +0200, Jakub Hrozek wrote: > On Fri, Jun 14, 2013 at 01:42:44PM +0200, Ondrej Kos wrote: > > Hi, > > > > While working on one issue, I created clean minimal fedora virtual machine > > and while installing dependencies for building SSSD (from BUILD.txt), I > > not

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread Jakub Hrozek
On Tue, Jun 18, 2013 at 09:43:04AM +0200, steve wrote: > On Tue, 2013-06-18 at 09:02 +0200, Jakub Hrozek wrote: > > On Fri, Apr 12, 2013 at 10:58:54AM -0500, Ariel Barria wrote: > > > done. > > > Thank you for comments. > > > > I must admit I forgot about this patch (it wasn't even found by > > Co

Re: [SSSD] [PATCH] nested groups patches

2013-06-18 Thread Jakub Hrozek
On Mon, Jun 17, 2013 at 03:25:40PM -0400, Stephen Gallagher wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 06/17/2013 03:21 PM, Pavel Brezina wrote: > > > > > > - Original Message - > >> From: "Stephen Gallagher" To: "Development > >> of the System Security Services Dae

Re: [SSSD] [PATCH] Add ldap_autofs_map_master_name option

2013-06-18 Thread Jakub Hrozek
On Tue, Jun 18, 2013 at 10:29:36AM +0200, Lukas Slebodnik wrote: > On (17/06/13 19:47), Jakub Hrozek wrote: > >On Mon, Jun 17, 2013 at 05:28:17PM +0200, Lukas Slebodnik wrote: > >> On (03/06/13 22:41), Jakub Hrozek wrote: > >> >On Sat, Jun 01, 2013 at 12:30:47AM -0700, C. S. wrote: > >> >> Hi Jakub

Re: [SSSD] [PATCH] Add ldap_autofs_map_master_name option

2013-06-18 Thread Lukas Slebodnik
On (17/06/13 19:47), Jakub Hrozek wrote: >On Mon, Jun 17, 2013 at 05:28:17PM +0200, Lukas Slebodnik wrote: >> On (03/06/13 22:41), Jakub Hrozek wrote: >> >On Sat, Jun 01, 2013 at 12:30:47AM -0700, C. S. wrote: >> >> Hi Jakub, >> >> >> >> Enclosed is the squashed patch. >> >> >> >> No worries, re-

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread Jakub Hrozek
On Tue, Jun 18, 2013 at 09:39:20AM +0200, Pavel Březina wrote: > On 06/18/2013 09:02 AM, Jakub Hrozek wrote: > >On Fri, Apr 12, 2013 at 10:58:54AM -0500, Ariel Barria wrote: > >>done. > >>Thank you for comments. > > > >I must admit I forgot about this patch (it wasn't even found by > >Coverity), I'

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread steve
On Tue, 2013-06-18 at 09:02 +0200, Jakub Hrozek wrote: > On Fri, Apr 12, 2013 at 10:58:54AM -0500, Ariel Barria wrote: > > done. > > Thank you for comments. > > I must admit I forgot about this patch (it wasn't even found by > Coverity), I'm sorry. > > Attached is a version with slightly reworded

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread Pavel Březina
On 06/18/2013 09:02 AM, Jakub Hrozek wrote: On Fri, Apr 12, 2013 at 10:58:54AM -0500, Ariel Barria wrote: done. Thank you for comments. I must admit I forgot about this patch (it wasn't even found by Coverity), I'm sorry. Why would it be found by Coverity? Attached is a version with sligh

Re: [SSSD] [PATCH] Removing BUILD.txt content

2013-06-18 Thread Jakub Hrozek
On Fri, Apr 12, 2013 at 10:58:54AM -0500, Ariel Barria wrote: > done. > Thank you for comments. I must admit I forgot about this patch (it wasn't even found by Coverity), I'm sorry. Attached is a version with slightly reworded BUILD.txt contents. The attribution still belongs to Ariel, of course.