[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: commit: ldap/clients/tools common.c common.h
At 03:23 AM 11/5/2005, Pierangelo Masarati wrote:
>On Fri, 2005-11-04 at 23:53 +0000, kurt@OpenLDAP.org wrote:
>> Update of /repo/OpenLDAP/pkg/ldap/clients/tools
>>
>> Modified Files:
>> common.c 1.51 -> 1.52
>> common.h 1.16 -> 1.17
>>
>> Log Message:
>> ITS#4147 fix
>> Should replace all ldap_perror calls with tool_perror calls to
>> avoid use of deprecated function.
>
>there is some (erroneous?) behavior since tool_error() is called also
>with LDAP_SUCCESS when matched == "" or info == "" (I'm fixing it); but
>this likely always happens, because the client library is always
>mallocing a number of "" when parsing results. Wouldn't it be better to
>leave the pointers NULL when there's nothing to return?
Yes. Given that matched and DN come from non-optional
protocol fields where empty indicates no value (ugh),
we should return NULL instead of ->"".
>I know it's
>going to be a significant API change, but it's going to save a lot of
>mallocs and of memory fragmentation, since successful results are most
>likely.