[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: ITS#5724 (Re: commit: ldap/servers/slapd backover.c)
- To: openldap-its@OpenLDAP.org
- Subject: Re: ITS#5724 (Re: commit: ldap/servers/slapd backover.c)
- From: hyc@symas.com
- Date: Thu, 19 Feb 2009 23:17:09 GMT
- Auto-submitted: auto-generated (OpenLDAP-ITS)
ando@sys-net.it wrote:
> ando@sys-net.it wrote:
>
>> Sharing the pointer to the supported controls would allow to use the
>> actual set. Something along the line
>>
>> struct BackendDB {
>> char be_ctrlsbuf[SLAP_MAX_CIDS + 1];
>> char *be_ctrls;
>> };
>>
>> All backends would have be_ctrls = be_ctrlsbuf, but back-relay would set
>> it to NULL, and temporarily set it to the relayed database's be_ctrlsbuf
>> for the duration of an operation. Modules wouldn't be able to register
>> new controls because the pointer is null otherwise.
>
> Of course, this wouldn't work since it implies modifying the BackendDB
> structure, which is read-only :(
backover.c 1.91 appears to partially revert rev 1.84, thus breaking ITS#5346
again. Investigating now...
--
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/