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

Re: ITS#5724 (Re: commit: ldap/servers/slapd backover.c)

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/