[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: (ITS#8434) Config entry cleanup on Add should be run unconditionally
- To: openldap-its@OpenLDAP.org
- Subject: Re: (ITS#8434) Config entry cleanup on Add should be run unconditionally
- From: hyc@symas.com
- Date: Wed, 08 Jun 2016 22:15:13 +0000
- Auto-submitted: auto-generated (OpenLDAP-ITS)
okuznik@symas.com wrote:
> Full_Name: Ondrej Kuznik
> Version: master
> OS:
> URL: ftp://ftp.openldap.org/incoming/
> Submission from: (NULL) (2a02:c7f:221f:c00:a3e:8eff:fe52:dac5)
>
>
> With cn=config the ConfigLDAPadd function might have to allocate resources that
> are needed by the time the attributes/config directives are being processed
> since there is no indication which directives there are and what order they come
> in.
When using LDAPAdd, attributes are processed in the order of their schema
definitions.
> If any of the config values fail validation, the configuration code has to react
> and clean up - which is possible using ca->cleanup.
No, that's not its purpose. If anything fails, the entire config entry must be
wiped out. LDAPAdd is all-or-nothing.
> While Modify handling (config_modify_internal) already calls ca->cleanup in each
> case, this is not true for config_add_internal, there it is only run on success.
--
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/