[SSSD] [PATCH] be_ptask and be_refresh: send, recv shadow a global declaration

2013-06-12 Thread Pavel Březina
Hi, Sumit found warning caused by my latest patch set. CC src/providers/dp_ptask.o ../src/providers/dp_ptask.c: In function 'be_ptask_create': ../src/providers/dp_ptask.c:249:41: warning: declaration of 'send' shadows a global declaration [-Wshadow] ../src/providers/dp_ptask.c:250:41: warn

Re: [SSSD] [PATCH] be_ptask and be_refresh: send, recv shadow a global declaration

2013-06-12 Thread Sumit Bose
On Wed, Jun 12, 2013 at 09:55:57AM +0200, Pavel Březina wrote: > Hi, > Sumit found warning caused by my latest patch set. > > CC src/providers/dp_ptask.o > ../src/providers/dp_ptask.c: In function 'be_ptask_create': > ../src/providers/dp_ptask.c:249:41: warning: declaration of 'send' > shado

Re: [SSSD] [PATCH] be_ptask and be_refresh: send, recv shadow a global declaration

2013-06-12 Thread Pavel Březina
On 06/12/2013 10:20 AM, Sumit Bose wrote: On Wed, Jun 12, 2013 at 09:55:57AM +0200, Pavel Březina wrote: Hi, Sumit found warning caused by my latest patch set. CC src/providers/dp_ptask.o ../src/providers/dp_ptask.c: In function 'be_ptask_create': ../src/providers/dp_ptask.c:249:41: warn

Re: [SSSD] [PATCH] be_ptask and be_refresh: send, recv shadow a global declaration

2013-06-12 Thread Sumit Bose
On Wed, Jun 12, 2013 at 10:37:44AM +0200, Pavel Březina wrote: > On 06/12/2013 10:20 AM, Sumit Bose wrote: > >On Wed, Jun 12, 2013 at 09:55:57AM +0200, Pavel Březina wrote: > >>Hi, > >>Sumit found warning caused by my latest patch set. > >> > >> CC src/providers/dp_ptask.o > >>../src/provider

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

2013-06-12 Thread Pavel Březina
On 05/27/2013 05:19 PM, Jan Cholasta wrote: Hi, the attached patches fix . Honza Hi, +if (state->count != 1) { +ret = sysdb_delete_ssh_host(state->domain->sysdb, state->domain, +state->name); +

Re: [SSSD] [PATCH] be_ptask and be_refresh: send, recv shadow a global declaration

2013-06-12 Thread Lukas Slebodnik
On (12/06/13 10:20), Sumit Bose wrote: >On Wed, Jun 12, 2013 at 09:55:57AM +0200, Pavel Březina wrote: >> Hi, >> Sumit found warning caused by my latest patch set. >> >> CC src/providers/dp_ptask.o >> ../src/providers/dp_ptask.c: In function 'be_ptask_create': >> ../src/providers/dp_ptask.c:

Re: [SSSD] [PATCH] be_ptask and be_refresh: send, recv shadow a global declaration

2013-06-12 Thread Sumit Bose
On Wed, Jun 12, 2013 at 12:04:59PM +0200, Lukas Slebodnik wrote: > On (12/06/13 10:20), Sumit Bose wrote: > >On Wed, Jun 12, 2013 at 09:55:57AM +0200, Pavel Březina wrote: > >> Hi, > >> Sumit found warning caused by my latest patch set. > >> > >> CC src/providers/dp_ptask.o > >> ../src/provi

[SSSD] [PATCH] Change order of libraries in linking process.

2013-06-12 Thread Lukas Slebodnik
ehlo, It seems that some linkers have problem with wrong order of libraries. This commit only change order. Attached patch should fix next issue: Library libsss_crypt.so is used in linking process, but there are undefined references. It seems that I changed order of libraries after rewriting lin

Re: [SSSD] [PATCH] Change order of libraries in linking process.

2013-06-12 Thread Timo Aaltonen
On 12.06.2013 14:39, Lukas Slebodnik wrote: > ehlo, > > It seems that some linkers have problem with wrong order of libraries. > This commit only change order. > > Attached patch should fix next issue: > Library libsss_crypt.so is used in linking process, but there are > undefined references. >

Re: [SSSD] [PATCH] be_ptask and be_refresh: send, recv shadow a global declaration

2013-06-12 Thread Jakub Hrozek
On Wed, Jun 12, 2013 at 11:04:31AM +0200, Sumit Bose wrote: > On Wed, Jun 12, 2013 at 10:37:44AM +0200, Pavel Březina wrote: > > On 06/12/2013 10:20 AM, Sumit Bose wrote: > > >On Wed, Jun 12, 2013 at 09:55:57AM +0200, Pavel Březina wrote: > > >>Hi, > > >>Sumit found warning caused by my latest patc

Re: [SSSD] [PATCH] Use the correct talloc context when creating AD subdomains

2013-06-12 Thread Jakub Hrozek
On Tue, Jun 11, 2013 at 12:51:16PM -0400, Stephen Gallagher wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 06/11/2013 08:44 AM, Jakub Hrozek wrote: > > This is the last Coverity warning in the current master. > > > Looks correct to me. Ack. Pushed to master.

Re: [SSSD] [PATCH] Fix minor typos in docs and comments

2013-06-12 Thread Jakub Hrozek
On Tue, Jun 11, 2013 at 01:04:07PM -0400, Stephen Gallagher wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 06/11/2013 12:57 PM, Dmitri Pal wrote: > > On 06/11/2013 12:16 PM, Yuri Chornoivan wrote: > >> Hi, > >> > >> Attached is a patch to fix minor typos in man pages and > >> com

Re: [SSSD] [PATCH] Change order of libraries in linking process.

2013-06-12 Thread Jakub Hrozek
On Wed, Jun 12, 2013 at 02:52:08PM +0300, Timo Aaltonen wrote: > On 12.06.2013 14:39, Lukas Slebodnik wrote: > > ehlo, > > > > It seems that some linkers have problem with wrong order of libraries. > > This commit only change order. > > > > Attached patch should fix next issue: > > Library libsss

Re: [SSSD] [PATCH] Every time return directory for krb5 cache collection.

2013-06-12 Thread Jakub Hrozek
On Wed, Jun 05, 2013 at 09:37:28AM +0200, Lukas Slebodnik wrote: > On (02/06/13 23:14), Jakub Hrozek wrote: > >On Sat, 2013-06-01 at 11:43 +0200, Lukas Slebodnik wrote: > >> On (31/05/13 21:29), Jakub Hrozek wrote: > >> >On Wed, 2013-05-29 at 16:09 +0200, Lukas Slebodnik wrote: > >> >> On (29/05/13

Re: [SSSD] [PATCH] Send error message from krb5_child to backend.

2013-06-12 Thread Jakub Hrozek
On Fri, May 24, 2013 at 04:52:49PM +0200, Lukas Slebodnik wrote: > ehlo, > > Commit c6872e79e8496fd075e20aec0343ade99cca725c caused that password migration > doesn't work using sssd. > > If pre authentication failed then we should send message to backend, > so password migration could be detected

Re: [SSSD] Design Discussion: IPA Server Mode

2013-06-12 Thread Jakub Hrozek
On Mon, Jun 10, 2013 at 11:45:08AM +0200, Sumit Bose wrote: > On Tue, May 28, 2013 at 01:20:15PM +0200, Sumit Bose wrote: > > Hi, > > > > I have created a design page for one of the next major features of SSSD > > at https://fedorahosted.org/sssd/wiki/DesignDocs/IPAServerMode . The > > basic idea