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

Re: exop client API



At 03:09 PM 11/14/2005, Pierangelo Masarati wrote:
>I see a slight inconsistency in the client API for parsing exops:
>
>- ldap_parse_extended_result() takes a struct berval ** for the resdata;

This is because the caller can pass NULL if it doesn't
expect the resdata.  Of course, it the caller passes
NULL and there is resdata, the resdata is improperly
ignored.

>- same for ldap_parse_whoami() for the authzDN

This likely should be just struct berval * as there is
no point in calling it with NULL.

>- but ldap_parse_passwd takes a struct berval * for the newpasswd
>
>I think ldap_parse_passwd() has the best interface.  Yet another
>(minimal, I understand) argument in favor of an API redesign...
>
>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
>------------------------------------------