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

Re: commit: ldap/servers/slapd result.c slap.h



On Fri, 2005-11-25 at 11:54 -0800, Howard Chu wrote:
> ando@OpenLDAP.org wrote:
> > Update of /repo/OpenLDAP/pkg/ldap/servers/slapd
> >
> > Modified Files:
> > 	result.c  1.276 -> 1.277
> > 	slap.h  1.731 -> 1.732
> >
> > Log Message:
> > check sizelimit in frontend (ITS#4213)
> >
> > CVS Web URLs:
> >   http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/
> >     http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/result.c
> >     http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/slap.h
> >
> > Changes are generally available on cvs.openldap.org (and CVSweb)
> > within 30 minutes of being committed.
> 
> Is there any reason why we need a special SLAPD_SIZELIMIT result code, 
> instead of just returning LDAP_SIZELIMIT_EXCEEDED?

I had the same idea initially, but I was afraid it could be misleading
to have specific error codes returned by a function that's not supposed
to return protocol error codes; it seems reasonable, though.

I'd rather change those -1, 0, 1 that are currently returned by
send_search_entry(), to make them a bit more readable and ready to be
used by the caller.  Something like LDAP_UNAVAILABLE, LDAP_SUCCESS,
LDAP_OTHER?

p.