[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: ITS#8233

Michael Ströder wrote:
> Howard Chu wrote:
>> Howard Chu wrote:
>>> Michael Ströder wrote:
>>>> But then I would expect slapd to remove the backslash(es) used for quoting:
>>> Good point. OK, there's some more work needed in here somewhere.
>> Fixed. Closing this ITS. If you have any other problems regarding this,
>> followup to ITS#8233. We don't open new ITSs for unreleased code.
> Sorry, but still I see the same problem with commit
> 23953716c76ab36fab7d5f6dea335bf9bdea6323.
> Example from ITS#8251 repeated here:
> In slapd.conf:
> ---------------------------------- snip ----------------------------------
> attributetype (
>        DESC 'Test attribute type with \"double quotes\" in DESC'
>        SYNTAX )
> ---------------------------------- snip ----------------------------------
> Returned via LDAP in subschema subentry (as LDIF):
> ---------------------------------- snip ----------------------------------
> attributeTypes: ( DESC 'Test attribute type with
>    \"double quotes\" in DESC' SYNTAX )
> ---------------------------------- snip ----------------------------------

True, and irrelevant. This behavior is unchanged from any previous OpenLDAP 

The regression you reported has been fixed, that is all.

> But regarding your comment in [1] I wonder what counts as an "argument"?
> Reading the section slapd.conf(5) more carefully it could mean that also all
> schema descriptions (containing spaces) count as one argument and therefore
> should be enclosed in double quotes (which is not the case also for all
> .schema files installed by OpenLDAP).
> Another example is:
> index foo,bar eq,sub
> Does the the config parser handle "foo,bar" and "eq,sub" as two separate
> arguments for directive "index"? Does the argument parsing depend on the
> configuration directive?

Yes, the argument parsing depends on the config directive. All of the 
schema-related elements (attributetype, objectclass, syntax, ditcontentrule) 
have their own parsers and (some of) the normal slapd.conf rules don't apply 
to them.

> [1] https://www.openldap.org/its/index.cgi?findid=8251#followup9
> Ciao, Michael.

   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/