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

Re: commit: ldap/servers/slapd/overlays syncprov.c



On Tue, 2005-12-06 at 17:00 -0800, Howard Chu wrote:
> ando@OpenLDAP.org wrote:
> > Update of /repo/OpenLDAP/pkg/ldap/servers/slapd/overlays
> >
> > Modified Files:
> > 	syncprov.c  1.132 -> 1.133
> >
> > Log Message:
> > don't search max CSN if don't have a value; try the operational_attrs if entryCSN is not in attrs
> >   
> 
> The "don't search max CSN" patch here is a bad idea, why do you need it? 
> Currently syncprov will work even if you forget to use "-w" on the 
> slapadd, your patch will break that feature.

Because if none is found in the entry, it searches with "(entryCSN>=)"
which creates some problems.  I'm trying to make syncprov work with
back-sql and with relaying backends in general...

In this case, I guess we should move the slap_get_csn() __before__ the
search.

p.

p.




Ing. Pierangelo Masarati
Responsabile Open Solution

SysNet s.n.c.
Via Dossi, 8 - 27100 Pavia - ITALIA
http://www.sys-net.it
------------------------------------------
Office:   +39.02.23998309          
Mobile:   +39.333.4963172
Email:    pierangelo.masarati@sys-net.it
------------------------------------------