Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-10-05 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/02/2009 02:47 PM, Stephen Gallagher wrote: > On 10/02/2009 12:52 PM, Dmitri Pal wrote: >> Dmitri Pal wrote: >>> Stephen Gallagher wrote: >>> On 09/28/2009 01:46 PM, Dmitri Pal wrote: > Stephen Gallagher wrote: >

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-10-02 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/02/2009 12:52 PM, Dmitri Pal wrote: > Dmitri Pal wrote: >> Stephen Gallagher wrote: >> >>> On 09/28/2009 01:46 PM, Dmitri Pal wrote: >>> >>> Stephen Gallagher wrote: > On 09/28/2009 09:55 AM, Simo Sorce w

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-10-02 Thread Dmitri Pal
Dmitri Pal wrote: > Stephen Gallagher wrote: > >> On 09/28/2009 01:46 PM, Dmitri Pal wrote: >> >> >>> Stephen Gallagher wrote: >>> >>> On 09/28/2009 09:55 AM, Simo Sorce wrote: > On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher w

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Dmitri Pal
Stephen Gallagher wrote: > On 09/28/2009 01:46 PM, Dmitri Pal wrote: > >> Stephen Gallagher wrote: >> >>> On 09/28/2009 09:55 AM, Simo Sorce wrote: >>> >>> On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher wrote: > The SSSD needs a config_from_f

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Sumit Bose
On Mon, Sep 28, 2009 at 02:50:30PM -0400, Stephen Gallagher wrote: > On 09/28/2009 01:46 PM, Dmitri Pal wrote: > > Stephen Gallagher wrote: > >> On 09/28/2009 09:55 AM, Simo Sorce wrote: > >> > >>> On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher wrote: > >>> > The SSSD needs a con

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Stephen Gallagher
On 09/28/2009 01:46 PM, Dmitri Pal wrote: > Stephen Gallagher wrote: >> On 09/28/2009 09:55 AM, Simo Sorce wrote: >> >>> On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher wrote: >>> The SSSD needs a config_from_fd() variant of the config_from_file() call so that we can pre

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Dmitri Pal
Stephen Gallagher wrote: > On 09/28/2009 09:55 AM, Simo Sorce wrote: > >> On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher wrote: >> >>> The SSSD needs a config_from_fd() variant of the config_from_file() >>> call >>> so that we can preopen a config file and perform some verification on

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Sumit Bose
On Mon, Sep 28, 2009 at 10:33:22AM -0400, Stephen Gallagher wrote: > On 09/28/2009 09:55 AM, Simo Sorce wrote: > > On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher wrote: > >> The SSSD needs a config_from_fd() variant of the config_from_file() > >> call > >> so that we can preopen a config file

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Stephen Gallagher
On 09/28/2009 09:55 AM, Simo Sorce wrote: > On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher wrote: >> The SSSD needs a config_from_fd() variant of the config_from_file() >> call >> so that we can preopen a config file and perform some verification on >> it >> before parsing it. The config_from

Re: [SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Simo Sorce
On Mon, 2009-09-28 at 09:38 -0400, Stephen Gallagher wrote: > The SSSD needs a config_from_fd() variant of the config_from_file() > call > so that we can preopen a config file and perform some verification on > it > before parsing it. The config_from_fd() call is used to avoid race > conditions bet

[SSSD] [PATCH] Add config_from_fd() to ini_config

2009-09-28 Thread Stephen Gallagher
The SSSD needs a config_from_fd() variant of the config_from_file() call so that we can preopen a config file and perform some verification on it before parsing it. The config_from_fd() call is used to avoid race conditions between testing the file and reading it in. Note: the *_from_fd() function