Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-12 Thread Jakub Hrozek
On Wed, Sep 12, 2012 at 02:23:41PM +0200, Sumit Bose wrote: > On Wed, Sep 12, 2012 at 10:19:57AM +0200, Pavel Březina wrote: > > On 09/11/2012 06:54 PM, Sumit Bose wrote: > > >On Mon, Sep 10, 2012 at 01:52:53PM +0200, Pavel Březina wrote: > > >>On 09/10/2012 10:48 AM, Sumit Bose wrote: > > >>>On Mo

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-12 Thread Sumit Bose
On Wed, Sep 12, 2012 at 10:19:57AM +0200, Pavel Březina wrote: > On 09/11/2012 06:54 PM, Sumit Bose wrote: > >On Mon, Sep 10, 2012 at 01:52:53PM +0200, Pavel Březina wrote: > >>On 09/10/2012 10:48 AM, Sumit Bose wrote: > >>>On Mon, Sep 10, 2012 at 09:35:25AM +0200, Pavel Březina wrote: > On 09/

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-12 Thread Pavel Březina
On 09/11/2012 06:54 PM, Sumit Bose wrote: On Mon, Sep 10, 2012 at 01:52:53PM +0200, Pavel Březina wrote: On 09/10/2012 10:48 AM, Sumit Bose wrote: On Mon, Sep 10, 2012 at 09:35:25AM +0200, Pavel Březina wrote: On 09/07/2012 11:31 AM, Sumit Bose wrote: On Wed, Sep 05, 2012 at 03:05:49PM +0200,

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-11 Thread Sumit Bose
On Mon, Sep 10, 2012 at 01:52:53PM +0200, Pavel Březina wrote: > On 09/10/2012 10:48 AM, Sumit Bose wrote: > >On Mon, Sep 10, 2012 at 09:35:25AM +0200, Pavel Březina wrote: > >>On 09/07/2012 11:31 AM, Sumit Bose wrote: > >>>On Wed, Sep 05, 2012 at 03:05:49PM +0200, Pavel Březina wrote: > 0001 f

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-10 Thread Pavel Březina
On 09/10/2012 10:48 AM, Sumit Bose wrote: On Mon, Sep 10, 2012 at 09:35:25AM +0200, Pavel Březina wrote: On 09/07/2012 11:31 AM, Sumit Bose wrote: On Wed, Sep 05, 2012 at 03:05:49PM +0200, Pavel Březina wrote: 0001 fixes https://fedorahosted.org/sssd/ticket/1458 0002 fixes a memory leak when b

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-10 Thread Sumit Bose
On Mon, Sep 10, 2012 at 09:35:25AM +0200, Pavel Březina wrote: > On 09/07/2012 11:31 AM, Sumit Bose wrote: > >On Wed, Sep 05, 2012 at 03:05:49PM +0200, Pavel Březina wrote: > >>0001 fixes https://fedorahosted.org/sssd/ticket/1458 > >>0002 fixes a memory leak when be_process_init() fails. I think it

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-10 Thread Jakub Hrozek
On Mon, Sep 10, 2012 at 09:35:25AM +0200, Pavel Březina wrote: > On 09/07/2012 11:31 AM, Sumit Bose wrote: > >On Wed, Sep 05, 2012 at 03:05:49PM +0200, Pavel Březina wrote: > >>0001 fixes https://fedorahosted.org/sssd/ticket/1458 > >>0002 fixes a memory leak when be_process_init() fails. I think it

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-10 Thread Pavel Březina
On 09/07/2012 11:31 AM, Sumit Bose wrote: On Wed, Sep 05, 2012 at 03:05:49PM +0200, Pavel Březina wrote: 0001 fixes https://fedorahosted.org/sssd/ticket/1458 0002 fixes a memory leak when be_process_init() fails. I think it should be fixed, even though the backend is in this case immediately ter

Re: [SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-07 Thread Sumit Bose
On Wed, Sep 05, 2012 at 03:05:49PM +0200, Pavel Březina wrote: > 0001 fixes https://fedorahosted.org/sssd/ticket/1458 > 0002 fixes a memory leak when be_process_init() fails. I think it > should be fixed, even though the backend is in this case immediately > terminated Hi, the patches compile wit

[SSSD] [PATCHES] backend: initialize sudo only when it is enabled in services

2012-09-05 Thread Pavel Březina
0001 fixes https://fedorahosted.org/sssd/ticket/1458 0002 fixes a memory leak when be_process_init() fails. I think it should be fixed, even though the backend is in this case immediately terminated From 26f1d475af75572a9deffdec73360fb65e787417 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C