Issue 7626 - [Patch] Fix config_build_schema_inc
Summary: [Patch] Fix config_build_schema_inc
Status: VERIFIED FIXED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: slapd (show other issues)
Version: unspecified
Hardware: All All
: --- normal
Target Milestone: ---
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-17 06:08 UTC by jsynacek@redhat.com
Modified: 2014-08-01 21:04 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description jsynacek@redhat.com 2013-06-17 06:08:45 UTC 
Full_Name: Jan Synacek
Version: master
OS: Linux - Fedora 18
URL: http://jsynacek.fedorapeople.org/openldap/0001-Fix-config_build_schema_inc.patch
Submission from: (NULL) (209.132.186.34)


When converting a slapd.conf into the DIT configuration, slaptest incorrectly
handles 'include' directives containing a custom file instead of a schema file.

Example:
  custom.file:
    include /etc/ldap/schema/cosine.ldif

  slapd.conf
    ...
    include /path/to/custom.file
    ...

Slaptest does not properly include cosine.ldif into the new configuration.
Comment 1 Howard Chu 2013-06-17 12:41:40 UTC 
jsynacek@redhat.com wrote:
> Full_Name: Jan Synacek
> Version: master
> OS: Linux - Fedora 18
> URL: http://jsynacek.fedorapeople.org/openldap/0001-Fix-config_build_schema_inc.patch
> Submission from: (NULL) (209.132.186.34)
>
>
> When converting a slapd.conf into the DIT configuration, slaptest incorrectly
> handles 'include' directives containing a custom file instead of a schema file.
>
> Example:
>    custom.file:
>      include /etc/ldap/schema/cosine.ldif
>
>    slapd.conf
>      ...
>      include /path/to/custom.file
>      ...
>
> Slaptest does not properly include cosine.ldif into the new configuration.

Your example is invalid since slapd.conf format doesn't support including LDIF 
files.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/
Comment 2 jsynacek@redhat.com 2013-06-17 12:45:25 UTC 
On 06/17/2013 02:41 PM, Howard Chu wrote:
> jsynacek@redhat.com wrote:
>> Full_Name: Jan Synacek
>> Version: master
>> OS: Linux - Fedora 18
>> URL:
>> http://jsynacek.fedorapeople.org/openldap/0001-Fix-config_build_schema_inc.patch
>> Submission from: (NULL) (209.132.186.34)
>>
>>
>> When converting a slapd.conf into the DIT configuration, slaptest incorrectly
>> handles 'include' directives containing a custom file instead of a schema file.
>>
>> Example:
>>    custom.file:
>>      include /etc/ldap/schema/cosine.ldif
>>
>>    slapd.conf
>>      ...
>>      include /path/to/custom.file
>>      ...
>>
>> Slaptest does not properly include cosine.ldif into the new configuration.
> 
> Your example is invalid since slapd.conf format doesn't support including LDIF
> files.
> 

Sorry, I meant "schema" instead of "ldif".

s/cosine.ldif/cosine.schema
Comment 3 Howard Chu 2013-11-18 02:59:53 UTC 
changed notes
changed state Open to Closed
Comment 4 OpenLDAP project 2014-08-01 21:04:02 UTC 
Fixed in ITS#7739
(patch provided here was actually correct)