[Date Prev][Date Next]
Re: (ITS#3720) back meta sizelimit exceeded
On Thu, 2005-05-12 at 22:36 +0200, Pierangelo Masarati wrote:
> firstname.lastname@example.org wrote:
> >Full_Name: Martin Evans
> >Version: 2.2.23
> >OS: Solaris 8
> >Submission from: (NULL) (22.214.171.124)
> >There's some strange looking logic in back_meta/search.c I think it is
> >incorrectly reporting size limit exceeded for some searches. e.g. if you do
> >ldapsearch -z1 with a filter to return only 1 entry from a proxied tree.
> >Heres a patch, which fixes a short term problem that I have. is it reasonable?
> I confirm the problem, but your analysis appears incorrect. In fact,
> your fix works in your case, but in cases where the filter matches
> exactly two entries it would return both and then sizelimit exceeded for
> a -z1 sizelimit, which is incorrect. I believe the test must move
> somewhere else, and apply if yet another entry is about to be returned
> by any target. I'm providing a new fix to both HEAD and RE22.
> In any cases, thanks for reporting and please test. p.
I've tested your RE22 patch (assuming the only changes were to search.c)
and am happy with it.
-- Dr MDT Evans, Computing Services, Queen Mary, University of London