Need just a little help getting this LDAP addition going

I've tried to add the function: ldap_sort_entries to ldap.c but something is not working with the strcmp. The function partially works, just not correctly. If the ldap result returns: sn = Landon sn = Bob sn = Zeev sn = Rasmus sn = Andi sn = Rick sn = Colin when it sorts, it will return: sn = Landon sn = Rasmus sn = Rick sn = Zeev Andi, Bob and Colin are dropped, it will properly sort the entries that are (greater than) the first entry returned, which in this case was sn = Landon. Another Example: same list of people but sn = Zeev is the first Output: sn = Zeev So it is accessing and modifying the ldap_result that php has stored in its internal list, so I am kinda lost from here. It would be great if someone that actually knows what they are doing can make it work for the good of the people. Thanks if you decide you can look at it. I've attached the php3_ldap.h and ldap.c files with my mods, just look in ldap.c for php3_ldap_sort_entries on line 744. Bob Silva P.S. - I really did try and make this work, so Im not just copping out and asking someone else to do it.

Attachment: [application/octet-stream] ldap.c
/* +----------------------------------------------------------------------+ | PHP HTML Embedded Scripting Language Version 3.0 | +----------------------------------------------------------------------+ | Copyright (c) 1997-1999 PHP Development Team (See Credits file) | +----------------------------------------------------------------------+ | This program is free software; you can redistribute it and/or modify | | it under the terms of one of the following licenses: | | | | A) the GNU General Public License as published by the Free Software | | Foundation; either version 2 of the License, or (at your option) | | any later version. | | | | B) the PHP License as published by the PHP Development Team and | | included in the distribution in the file: LICENSE | | | | This program is distributed in the hope that it will be useful, | | but WITHOUT ANY WARRANTY; without even the implied warranty of | | MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | | GNU General Public License for more details. | | | | You should have received a copy of both licenses referred to here. | | If you did not, or have any questions about PHP licensing, please | | contact core@php.net. | +----------------------------------------------------------------------+ | Authors: Amitay Isaacs <amitay@w-o-i.com> | | Eric Warnke <ericw@albany.edu> | +----------------------------------------------------------------------+ */ /* $Id: php3_ldap.h,v 1.24 1999/01/01 17:59:15 zeev Exp $ */ #ifndef _PHP3_LDAP_H #define _PHP3_LDAP_H #if COMPILE_DL #undef HAVE_LDAP #define HAVE_LDAP 1 #endif #if HAVE_LDAP #include <lber.h> #include <ldap.h> extern php3_module_entry ldap_module_entry; #define ldap_module_ptr &ldap_module_entry /* LDAP functions */ extern int php3_minit_ldap(INIT_FUNC_ARGS); extern int php3_mshutdown_ldap(void); extern void php3_info_ldap(void); extern void php3_ldap_connect(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_bind(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_unbind(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_read(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_list(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_search(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_free_result(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_count_entries(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_sort_entries(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_first_entry(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_next_entry(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_get_entries(INTERNAL_FUNCTION_PARAMETERS); #if 0 extern void php3_ldap_free_entry(INTERNAL_FUNCTION_PARAMETERS); #endif extern void php3_ldap_first_attribute(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_next_attribute(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_get_attributes(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_get_values(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ber_free(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_get_dn(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_explode_dn(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_dn2ufn(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_add(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_delete(INTERNAL_FUNCTION_PARAMETERS); extern void php3_ldap_modify(INTERNAL_FUNCTION_PARAMETERS); typedef struct { long default_link; long num_links, max_links; char *base_dn; int le_result, le_result_entry, le_ber_entry; int le_link; /* I just found out that the thread safe features of the netscape ldap library are only required if multiple threads are accessing the same LDAP structure. Since we are not doing that, we do not need to use this feature. I am leaving the code here anyway just in case. smc */ #if 0 struct ldap_thread_fns tfns; int le_errno; /* Corresponds to the LDAP error code */ char *le_matched; /* Matching components of the DN, if an NO_SUCH_OBJECT error occurred */ char *le_errmsg; /* Error message */ #endif } ldap_module; #ifndef THREAD_SAFE extern ldap_module php3_ldap_module; #endif #else #define ldap_module_ptr NULL #endif #endif /* _PHP3_LDAP_H */