]> git.sur5r.net Git - openldap/blobdiff - libraries/libldap/result.c
s/exit(1)/exit(EXIT_FAILURE)/
[openldap] / libraries / libldap / result.c
index 182a13ccd7d4b19c9048861ad79fe7d8dbf40143..e517a059ab23a8348741ff47208b468ddc2e988f 100644 (file)
@@ -1,18 +1,31 @@
 /*
+ * Copyright 1998-1999 The OpenLDAP Foundation, All Rights Reserved.
+ * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
+ */
+/*  Portions
  *  Copyright (c) 1990 Regents of the University of Michigan.
  *  All rights reserved.
  *
  *  result.c - wait for an ldap result
  */
 
-#include "portable.h"
+/*
+ * LDAPv3 (RFC2251)
+ *     LDAPResult ::= SEQUENCE {
+ *             resultCode              ENUMERATED { ... },
+ *             matchedDN               LDAPDN,
+ *             errorMessage    LDAPString,
+ *             referral                Referral OPTIONAL
+ *     }
+ *     Referral ::= SEQUENCE OF LDAPURL        (one or more)
+ *     LDAPURL ::= LDAPString                          (limited to URL chars)
+ */
 
-#ifndef lint 
-static char copyright[] = "@(#) Copyright (c) 1990 Regents of the University of Michigan.\nAll rights reserved.\n";
-#endif
+#include "portable.h"
 
 #include <stdio.h>
-#include <stdlib.h>
+
+#include <ac/stdlib.h>
 
 #include <ac/errno.h>
 #include <ac/socket.h>
@@ -23,22 +36,14 @@ static char copyright[] = "@(#) Copyright (c) 1990 Regents of the University of
 #include "ldap-int.h"
 
 
-static int ldap_abandoned LDAP_P(( LDAP *ld, int msgid ));
-static int ldap_mark_abandoned LDAP_P(( LDAP *ld, int msgid ));
-static int wait4msg LDAP_P(( LDAP *ld, int msgid, int all, struct timeval *timeout,
-       LDAPMessage **result ));
-#ifdef LDAP_REFERRALS
-static int read1msg LDAP_P(( LDAP *ld, int msgid, int all, Sockbuf *sb, LDAPConn *lc,
+static int ldap_abandoned LDAP_P(( LDAP *ld, ber_int_t msgid ));
+static int ldap_mark_abandoned LDAP_P(( LDAP *ld, ber_int_t msgid ));
+static int wait4msg LDAP_P(( LDAP *ld, ber_int_t msgid, int all, struct timeval *timeout,
        LDAPMessage **result ));
-static unsigned long build_result_ber LDAP_P(( LDAP *ld, BerElement *ber, LDAPRequest *lr ));
+static ber_tag_t try_read1msg LDAP_P(( LDAP *ld, ber_int_t msgid,
+       int all, Sockbuf *sb, LDAPConn *lc, LDAPMessage **result ));
+static ber_tag_t build_result_ber LDAP_P(( LDAP *ld, BerElement **bp, LDAPRequest *lr ));
 static void merge_error_info LDAP_P(( LDAP *ld, LDAPRequest *parentr, LDAPRequest *lr ));
-#else /* LDAP_REFERRALS */
-static int read1msg LDAP_P(( LDAP *ld, int msgid, int all, Sockbuf *sb,
-       LDAPMessage **result ));
-#endif /* LDAP_REFERRALS */
-#if defined( LDAP_CONNECTIONLESS ) || !defined( LDAP_REFERRALS )
-static int ldap_select1 LDAP_P(( LDAP *ld, struct timeval *timeout ));
-#endif
 
 
 /*
@@ -56,11 +61,29 @@ static int ldap_select1 LDAP_P(( LDAP *ld, struct timeval *timeout ));
  *     ldap_result( s, msgid, all, timeout, result )
  */
 int
-ldap_result( LDAP *ld, int msgid, int all, struct timeval *timeout,
+ldap_result(
+       LDAP *ld,
+       int msgid,
+       int all,
+       struct timeval *timeout,
        LDAPMessage **result )
 {
        LDAPMessage     *lm, *lastlm, *nextlm;
 
+       assert( ld != NULL );
+       assert( result != NULL );
+
+       Debug( LDAP_DEBUG_TRACE, "ldap_result\n", 0, 0, 0 );
+
+       if( ld == NULL ) {
+               return -1;
+       }
+
+       if( result == NULL ) {
+               ld->ld_errno = LDAP_PARAM_ERROR;
+               return -1;
+       }
+
        /*
         * First, look through the list of responses we have received on
         * this association and see if the response we're interested in
@@ -68,17 +91,15 @@ ldap_result( LDAP *ld, int msgid, int all, struct timeval *timeout,
         * wait until it arrives or timeout occurs.
         */
 
-       Debug( LDAP_DEBUG_TRACE, "ldap_result\n", 0, 0, 0 );
-
-       *result = NULLMSG;
-       lastlm = NULLMSG;
-       for ( lm = ld->ld_responses; lm != NULLMSG; lm = nextlm ) {
+       *result = NULL;
+       lastlm = NULL;
+       for ( lm = ld->ld_responses; lm != NULL; lm = nextlm ) {
                nextlm = lm->lm_next;
 
                if ( ldap_abandoned( ld, lm->lm_msgid ) ) {
                        ldap_mark_abandoned( ld, lm->lm_msgid );
 
-                       if ( lastlm == NULLMSG ) {
+                       if ( lastlm == NULL ) {
                                ld->ld_responses = lm->lm_next;
                        } else {
                                lastlm->lm_next = nextlm;
@@ -94,37 +115,37 @@ ldap_result( LDAP *ld, int msgid, int all, struct timeval *timeout,
 
                        if ( all == 0
                            || (lm->lm_msgtype != LDAP_RES_SEARCH_RESULT
+                           && lm->lm_msgtype != LDAP_RES_SEARCH_REFERENCE      /* LDAPv3 */
                            && lm->lm_msgtype != LDAP_RES_SEARCH_ENTRY) )
                                break;
 
-                       for ( tmp = lm; tmp != NULLMSG; tmp = tmp->lm_chain ) {
+                       for ( tmp = lm; tmp != NULL; tmp = tmp->lm_chain ) {
                                if ( tmp->lm_msgtype == LDAP_RES_SEARCH_RESULT )
                                        break;
                        }
 
-                       if ( tmp == NULLMSG ) {
-                               return( wait4msg( ld, msgid, all, timeout,
-                                   result ) );
+                       if ( tmp == NULL ) {
+                               lm = NULL;
                        }
 
                        break;
                }
                lastlm = lm;
        }
-       if ( lm == NULLMSG ) {
+       if ( lm == NULL ) {
                return( wait4msg( ld, msgid, all, timeout, result ) );
        }
 
-       if ( lastlm == NULLMSG ) {
-               ld->ld_responses = (all == 0 && lm->lm_chain != NULLMSG
+       if ( lastlm == NULL ) {
+               ld->ld_responses = (all == 0 && lm->lm_chain != NULL
                    ? lm->lm_chain : lm->lm_next);
        } else {
-               lastlm->lm_next = (all == 0 && lm->lm_chain != NULLMSG
+               lastlm->lm_next = (all == 0 && lm->lm_chain != NULL
                    ? lm->lm_chain : lm->lm_next);
        }
        if ( all == 0 )
-               lm->lm_chain = NULLMSG;
-       lm->lm_next = NULLMSG;
+               lm->lm_chain = NULL;
+       lm->lm_next = NULL;
 
        *result = lm;
        ld->ld_errno = LDAP_SUCCESS;
@@ -132,16 +153,21 @@ ldap_result( LDAP *ld, int msgid, int all, struct timeval *timeout,
 }
 
 static int
-wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
+wait4msg(
+       LDAP *ld,
+       ber_int_t msgid,
+       int all,
+       struct timeval *timeout,
        LDAPMessage **result )
 {
        int             rc;
        struct timeval  tv, *tvp;
        time_t          start_time = 0;
        time_t          tmp_time;
-#ifdef LDAP_REFERRALS
        LDAPConn        *lc, *nextlc;
-#endif /* LDAP_REFERRALS */
+
+       assert( ld != NULL );
+       assert( result != NULL );
 
 #ifdef LDAP_DEBUG
        if ( timeout == NULL ) {
@@ -149,7 +175,7 @@ wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
                    0, 0, 0 );
        } else {
                Debug( LDAP_DEBUG_TRACE, "wait4msg (timeout %ld sec, %ld usec)\n",
-                   timeout->tv_sec, timeout->tv_usec, 0 );
+                      (long) timeout->tv_sec, (long) timeout->tv_usec, 0 );
        }
 #endif /* LDAP_DEBUG */
 
@@ -163,27 +189,6 @@ wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
                    
        rc = -2;
        while ( rc == -2 ) {
-#ifndef LDAP_REFERRALS
-               /* hack attack */
-               if ( ld->ld_sb.sb_ber.ber_ptr >= ld->ld_sb.sb_ber.ber_end ) {
-                       rc = ldap_select1( ld, tvp );
-
-                       if ( rc == 0 || ( rc == -1 && (
-                               ( LDAP_BOOL_GET(&ld->ld_options, LDAP_BOOL_RESTART)
-                                       == LDAP_OPT_OFF )
-                           || errno != EINTR ))) {
-                               ld->ld_errno = (rc == -1 ? LDAP_SERVER_DOWN :
-                                   LDAP_TIMEOUT);
-                               return( rc );
-                       }
-
-               }
-               if ( rc == -1 ) {
-                       rc = -2;        /* select interrupted: loop */
-               } else {
-                       rc = read1msg( ld, msgid, all, &ld->ld_sb, result );
-               }
-#else /* !LDAP_REFERRALS */
 #ifdef LDAP_DEBUG
                if ( ldap_debug & LDAP_DEBUG_TRACE ) {
                        ldap_dump_connection( ld, ld->ld_conns, 1 );
@@ -191,9 +196,8 @@ wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
                }
 #endif /* LDAP_DEBUG */
                for ( lc = ld->ld_conns; lc != NULL; lc = lc->lconn_next ) {
-                       if ( lc->lconn_sb->sb_ber.ber_ptr <
-                           lc->lconn_sb->sb_ber.ber_end ) {
-                               rc = read1msg( ld, msgid, all, lc->lconn_sb,
+                       if ( ber_pvt_sb_data_ready(lc->lconn_sb) ) {
+                               rc = try_read1msg( ld, msgid, all, lc->lconn_sb,
                                    lc, result );
                                break;
                        }
@@ -212,8 +216,7 @@ wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
 #endif
 
                        if ( rc == 0 || ( rc == -1 && (
-                               ( LDAP_BOOL_GET(&ld->ld_options, LDAP_BOOL_RESTART)
-                                       == LDAP_OPT_OFF )
+                               !LDAP_BOOL_GET(&ld->ld_options, LDAP_BOOL_RESTART)
                                || errno != EINTR )))
                        {
                                ld->ld_errno = (rc == -1 ? LDAP_SERVER_DOWN :
@@ -232,13 +235,12 @@ wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
                                            LDAP_CONNST_CONNECTED &&
                                            ldap_is_read_ready( ld,
                                            lc->lconn_sb )) {
-                                               rc = read1msg( ld, msgid, all,
+                                               rc = try_read1msg( ld, msgid, all,
                                                    lc->lconn_sb, lc, result );
                                        }
                                }
                        }
                }
-#endif /* !LDAP_REFERRALS */
 
                if ( rc == -2 && tvp != NULL ) {
                        tmp_time = time( NULL );
@@ -249,7 +251,7 @@ wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
                        }
 
                        Debug( LDAP_DEBUG_TRACE, "wait4msg:  %ld secs to go\n",
-                               tv.tv_sec, 0, 0 );
+                              (long) tv.tv_sec, 0, 0 );
                        start_time = tmp_time;
                }
        }
@@ -258,71 +260,104 @@ wait4msg( LDAP *ld, int msgid, int all, struct timeval *timeout,
 }
 
 
-static int
-read1msg( LDAP *ld, int msgid, int all, Sockbuf *sb,
-#ifdef LDAP_REFERRALS
+static ber_tag_t
+try_read1msg(
+       LDAP *ld,
+       ber_int_t msgid,
+       int all,
+       Sockbuf *sb,
     LDAPConn *lc,
-#endif /* LDAP_REFERRALS */
-    LDAPMessage **result )
+       LDAPMessage **result )
 {
-       BerElement      ber;
+       BerElement      *ber;
        LDAPMessage     *new, *l, *prev, *tmp;
-       long            id;
-       unsigned long   tag, len;
+       ber_int_t       id;
+       ber_tag_t       tag;
+       ber_len_t       len;
        int             foundit = 0;
-#ifdef LDAP_REFERRALS
        LDAPRequest     *lr;
        BerElement      tmpber;
        int             rc, refer_cnt, hadref, simple_request;
-       unsigned long   lderr;
-#endif /* LDAP_REFERRALS */
+       ber_int_t       lderr;
 
+       assert( ld != NULL );
+       assert( lc != NULL );
+       
        Debug( LDAP_DEBUG_TRACE, "read1msg\n", 0, 0, 0 );
 
-       ber_init_w_nullc( &ber, 0 );
-       ldap_set_ber_options( ld, &ber );
+    if ( lc->lconn_ber == NULL ) {
+               lc->lconn_ber = ldap_alloc_ber_with_options(ld);
+
+               if( lc->lconn_ber == NULL ) {
+                       return -1;
+               }
+    }
+
+       ber = lc->lconn_ber;
+       assert( BER_VALID (ber) );
 
        /* get the next message */
-       if ( (tag = ber_get_next( sb, &len, &ber ))
+       errno = 0;
+       if ( (tag = ber_get_next( sb, &len, ber ))
            != LDAP_TAG_MESSAGE ) {
-               ld->ld_errno = (tag == LBER_DEFAULT ? LDAP_SERVER_DOWN :
-                   LDAP_LOCAL_ERROR);
-               return( -1 );
+               if ( tag == LBER_DEFAULT) {
+#ifdef LDAP_DEBUG                 
+                       Debug( LDAP_DEBUG_CONNS,
+                             "ber_get_next failed.\n", 0, 0, 0 );
+#endif            
+#ifdef EWOULDBLOCK                     
+                       if (errno==EWOULDBLOCK) return -2;
+#endif
+#ifdef EAGAIN
+                       if (errno == EAGAIN) return -2;
+#endif
+                       ld->ld_errno = LDAP_SERVER_DOWN;
+                       return -1;
+               }
+               ld->ld_errno = LDAP_LOCAL_ERROR;
+               return -1;
        }
 
+       /*
+     * We read a complete message.
+        * The connection should no longer need this ber.
+        */
+    lc->lconn_ber = NULL;
+
        /* message id */
-       if ( ber_get_int( &ber, &id ) == LBER_ERROR ) {
+       if ( ber_get_int( ber, &id ) == LBER_ERROR ) {
+               ber_free( ber, 1 );
                ld->ld_errno = LDAP_DECODING_ERROR;
                return( -1 );
        }
 
        /* if it's been abandoned, toss it */
-       if ( ldap_abandoned( ld, (int)id ) ) {
-               free( ber.ber_buf );    /* gack! */
+       if ( ldap_abandoned( ld, id ) ) {
+               ber_free( ber, 1 );
                return( -2 );   /* continue looking */
        }
 
-#ifdef LDAP_REFERRALS
        if (( lr = ldap_find_request_by_msgid( ld, id )) == NULL ) {
                Debug( LDAP_DEBUG_ANY,
                    "no request for response with msgid %ld (tossing)\n",
                    id, 0, 0 );
-               free( ber.ber_buf );    /* gack! */
+               ber_free( ber, 1 );
                return( -2 );   /* continue looking */
        }
-       Debug( LDAP_DEBUG_TRACE, "got %s msgid %ld, original id %d\n",
-           ( tag == LDAP_RES_SEARCH_ENTRY ) ? "entry" : "result", id,
-           lr->lr_origid );
-       id = lr->lr_origid;
-#endif /* LDAP_REFERRALS */
 
        /* the message type */
-       if ( (tag = ber_peek_tag( &ber, &len )) == LBER_ERROR ) {
+       if ( (tag = ber_peek_tag( ber, &len )) == LBER_ERROR ) {
                ld->ld_errno = LDAP_DECODING_ERROR;
+               ber_free( ber, 1 );
                return( -1 );
        }
 
-#ifdef LDAP_REFERRALS
+       Debug( LDAP_DEBUG_TRACE, "ldap_read: %s msgid %ld, original id %d\n",
+           ( tag == LDAP_RES_SEARCH_ENTRY ) ? "entry" : 
+               ( tag == LDAP_RES_SEARCH_REFERENCE ) ? "reference" : "result",
+               id, lr->lr_origid );
+
+       id = lr->lr_origid;
        refer_cnt = 0;
        hadref = simple_request = 0;
        rc = -2;        /* default is to keep looking (no response found) */
@@ -331,10 +366,9 @@ read1msg( LDAP *ld, int msgid, int all, Sockbuf *sb,
        if ( tag != LDAP_RES_SEARCH_ENTRY ) {
                if ( ld->ld_version >= LDAP_VERSION2 &&
                        ( lr->lr_parent != NULL ||
-                       ( LDAP_BOOL_GET(&ld->ld_options, LDAP_BOOL_REFERRALS)
-                               != LDAP_OPT_OFF ) ) )
+                       LDAP_BOOL_GET(&ld->ld_options, LDAP_BOOL_REFERRALS) ) )
                {
-                       tmpber = ber;   /* struct copy */
+                       tmpber = *ber;  /* struct copy */
                        if ( ber_scanf( &tmpber, "{iaa}", &lderr,
                            &lr->lr_res_matched, &lr->lr_res_error )
                            != LBER_ERROR ) {
@@ -365,8 +399,8 @@ Debug( LDAP_DEBUG_TRACE,
                    "read1msg:  %d new referrals\n", refer_cnt, 0, 0 );
 
                if ( refer_cnt != 0 ) { /* chasing referrals */
-                       free( ber.ber_buf );    /* gack! */
-                       ber.ber_buf = NULL;
+                       ber_free( ber, 1 );
+                       ber = NULL;
                        if ( refer_cnt < 0 ) {
                                return( -1 );   /* fatal error */
                        }
@@ -377,8 +411,8 @@ Debug( LDAP_DEBUG_TRACE,
                                simple_request = ( hadref ? 0 : 1 );
                        } else {
                                /* request with referrals or child request */
-                               free( ber.ber_buf );    /* gack! */
-                               ber.ber_buf = NULL;
+                               ber_free( ber, 1 );
+                               ber = NULL;
                        }
 
                        while ( lr->lr_parent != NULL ) {
@@ -400,13 +434,10 @@ Debug( LDAP_DEBUG_TRACE,
 lr->lr_res_errno, lr->lr_res_error ? lr->lr_res_error : "",
 lr->lr_res_matched ? lr->lr_res_matched : "" );
                                if ( !simple_request ) {
-                                       if ( ber.ber_buf != NULL ) {
-                                               free( ber.ber_buf ); /* gack! */
-                                               ber.ber_buf = NULL;
-                                       }
+                                       ber_free( ber, 1 );
+                                       ber = NULL;
                                        if ( build_result_ber( ld, &ber, lr )
                                            == LBER_ERROR ) {
-                                               ld->ld_errno = LDAP_NO_MEMORY;
                                                rc = -1; /* fatal error */
                                        }
                                }
@@ -420,20 +451,19 @@ lr->lr_res_matched ? lr->lr_res_matched : "" );
                }
        }
 
-       if ( ber.ber_buf == NULL ) {
+       if ( ber == NULL ) {
                return( rc );
        }
 
-#endif /* LDAP_REFERRALS */
        /* make a new ldap message */
-       if ( (new = (LDAPMessage *) calloc( 1, sizeof(LDAPMessage) ))
+       if ( (new = (LDAPMessage *) LDAP_CALLOC( 1, sizeof(LDAPMessage) ))
            == NULL ) {
                ld->ld_errno = LDAP_NO_MEMORY;
                return( -1 );
        }
        new->lm_msgid = (int)id;
        new->lm_msgtype = tag;
-       new->lm_ber = ber_dup( &ber );
+       new->lm_ber = ber;
 
 #ifndef LDAP_NOCACHE
                if ( ld->ld_cache != NULL ) {
@@ -460,15 +490,15 @@ lr->lr_res_matched ? lr->lr_res_matched : "" );
         * search response.
         */
 
-       prev = NULLMSG;
-       for ( l = ld->ld_responses; l != NULLMSG; l = l->lm_next ) {
+       prev = NULL;
+       for ( l = ld->ld_responses; l != NULL; l = l->lm_next ) {
                if ( l->lm_msgid == new->lm_msgid )
                        break;
                prev = l;
        }
 
        /* not part of an existing search response */
-       if ( l == NULLMSG ) {
+       if ( l == NULL ) {
                if ( foundit ) {
                        *result = new;
                        ld->ld_errno = LDAP_SUCCESS;
@@ -484,7 +514,7 @@ lr->lr_res_matched ? lr->lr_res_matched : "" );
            new->lm_msgid, new->lm_msgtype, 0 );
 
        /* part of a search response - add to end of list of entries */
-       for ( tmp = l; tmp->lm_chain != NULLMSG &&
+       for ( tmp = l; tmp->lm_chain != NULL &&
            tmp->lm_chain->lm_msgtype == LDAP_RES_SEARCH_ENTRY;
            tmp = tmp->lm_chain )
                ;       /* NULL */
@@ -492,7 +522,7 @@ lr->lr_res_matched ? lr->lr_res_matched : "" );
 
        /* return the whole chain if that's what we were looking for */
        if ( foundit ) {
-               if ( prev == NULLMSG )
+               if ( prev == NULL )
                        ld->ld_responses = l->lm_next;
                else
                        prev->lm_next = l->lm_next;
@@ -516,32 +546,56 @@ lr->lr_res_matched ? lr->lr_res_matched : "" );
 }
 
 
-#ifdef LDAP_REFERRALS
-static unsigned long
-build_result_ber( LDAP *ld, BerElement *ber, LDAPRequest *lr )
+static ber_tag_t
+build_result_ber( LDAP *ld, BerElement **bp, LDAPRequest *lr )
 {
-       unsigned long   len;
-       long            along;
+       ber_len_t       len;
+       ber_int_t       tag;
+       ber_int_t       along;
+       BerElement *ber;
+
+       *bp = NULL;
+       ber = ldap_alloc_ber_with_options( ld );
+
+       if( ber == NULL ) {
+               ld->ld_errno = LDAP_NO_MEMORY;
+               return LBER_ERROR;
+       }
 
-       ber_init_w_nullc( ber, 0 );
-       ldap_set_ber_options( ld, ber );
        if ( ber_printf( ber, "{it{ess}}", lr->lr_msgid,
-           (long)lr->lr_res_msgtype, lr->lr_res_errno,
+           lr->lr_res_msgtype, lr->lr_res_errno,
            lr->lr_res_matched ? lr->lr_res_matched : "",
            lr->lr_res_error ? lr->lr_res_error : "" ) == -1 ) {
+
+               ld->ld_errno = LDAP_ENCODING_ERROR;
+               ber_free(ber, 1);
                return( LBER_ERROR );
        }
 
        ber_reset( ber, 1 );
+
        if ( ber_skip_tag( ber, &len ) == LBER_ERROR ) {
+               ld->ld_errno = LDAP_DECODING_ERROR;
+               ber_free(ber, 1);
                return( LBER_ERROR );
        }
 
        if ( ber_get_int( ber, &along ) == LBER_ERROR ) {
+               ld->ld_errno = LDAP_DECODING_ERROR;
+               ber_free(ber, 1);
                return( LBER_ERROR );
        }
 
-       return( ber_peek_tag( ber, &len ));
+       tag = ber_peek_tag( ber, &len );
+
+       if ( tag == LBER_ERROR ) {
+               ld->ld_errno = LDAP_DECODING_ERROR;
+               ber_free(ber, 1);
+               return( LBER_ERROR );
+       }
+
+       *bp = ber;
+       return tag;
 }
 
 
@@ -561,13 +615,13 @@ merge_error_info( LDAP *ld, LDAPRequest *parentr, LDAPRequest *lr )
            parentr->lr_res_errno == LDAP_SUCCESS ) {
                parentr->lr_res_errno = lr->lr_res_errno;
                if ( parentr->lr_res_error != NULL ) {
-                       free( parentr->lr_res_error );
+                       LDAP_FREE( parentr->lr_res_error );
                }
                parentr->lr_res_error = lr->lr_res_error;
                lr->lr_res_error = NULL;
-               if ( NAME_ERROR( lr->lr_res_errno )) {
+               if ( LDAP_NAME_ERROR( lr->lr_res_errno )) {
                        if ( parentr->lr_res_matched != NULL ) {
-                               free( parentr->lr_res_matched );
+                               LDAP_FREE( parentr->lr_res_matched );
                        }
                        parentr->lr_res_matched = lr->lr_res_matched;
                        lr->lr_res_matched = NULL;
@@ -581,52 +635,23 @@ merge_error_info( LDAP *ld, LDAPRequest *parentr, LDAPRequest *lr )
            parentr->lr_res_error : "", parentr->lr_res_matched ?
            parentr->lr_res_matched : "" );
 }
-#endif /* LDAP_REFERRALS */
-
-
 
-#if defined( LDAP_CONNECTIONLESS ) || !defined( LDAP_REFERRALS )
-
-static int
-ldap_select1( LDAP *ld, struct timeval *timeout )
-{
-       fd_set          readfds;
-       static int      tblsize;
-
-       if ( tblsize == 0 ) {
-#ifdef HAVE_SYSCONF
-               tblsize = sysconf( _SC_OPEN_MAX );
-#elif HAVE_GETDTABLESIZE
-               tblsize = getdtablesize();
-#else
-               tblsize = FD_SETSIZE;
-#endif
-#ifdef FD_SETSIZE
-               if ( tblsize > FD_SETSIZE ) {
-                       tblsize = FD_SETSIZE;
-               }
-#endif /* FD_SETSIZE */
-       }
-
-       FD_ZERO( &readfds );
-       FD_SET( ld->ld_sb.sb_sd, &readfds );
-
-       return( select( tblsize, &readfds, 0, 0, timeout ) );
-}
-
-#endif /* !LDAP_REFERRALS */
 
 
 int
 ldap_msgtype( LDAPMessage *lm )
 {
-       return( lm ? lm->lm_msgtype : -1 );
+       assert( lm != NULL );
+       return ( lm == NULL ) ? lm->lm_msgtype : -1;
 }
 
+
 int
 ldap_msgid( LDAPMessage *lm )
 {
-       return( lm ? lm->lm_msgid : -1 );
+       assert( lm != NULL );
+
+       return ( lm == NULL ) ? lm->lm_msgid : -1;
 }
 
 
@@ -636,13 +661,15 @@ ldap_msgfree( LDAPMessage *lm )
        LDAPMessage     *next;
        int             type = 0;
 
+       assert( lm != NULL );
+
        Debug( LDAP_DEBUG_TRACE, "ldap_msgfree\n", 0, 0, 0 );
 
-       for ( ; lm != NULLMSG; lm = next ) {
+       for ( ; lm != NULL; lm = next ) {
                next = lm->lm_chain;
                type = lm->lm_msgtype;
                ber_free( lm->lm_ber, 1 );
-               free( (char *) lm );
+               LDAP_FREE( (char *) lm );
        }
 
        return( type );
@@ -658,19 +685,21 @@ ldap_msgdelete( LDAP *ld, int msgid )
 {
        LDAPMessage     *lm, *prev;
 
+       assert( ld != NULL );
+
        Debug( LDAP_DEBUG_TRACE, "ldap_msgdelete\n", 0, 0, 0 );
 
-       prev = NULLMSG;
-       for ( lm = ld->ld_responses; lm != NULLMSG; lm = lm->lm_next ) {
+       prev = NULL;
+       for ( lm = ld->ld_responses; lm != NULL; lm = lm->lm_next ) {
                if ( lm->lm_msgid == msgid )
                        break;
                prev = lm;
        }
 
-       if ( lm == NULLMSG )
+       if ( lm == NULL )
                return( -1 );
 
-       if ( prev == NULLMSG )
+       if ( prev == NULL )
                ld->ld_responses = lm->lm_next;
        else
                prev->lm_next = lm->lm_next;
@@ -686,7 +715,7 @@ ldap_msgdelete( LDAP *ld, int msgid )
  * return 1 if message msgid is waiting to be abandoned, 0 otherwise
  */
 static int
-ldap_abandoned( LDAP *ld, int msgid )
+ldap_abandoned( LDAP *ld, ber_int_t msgid )
 {
        int     i;
 
@@ -702,7 +731,7 @@ ldap_abandoned( LDAP *ld, int msgid )
 
 
 static int
-ldap_mark_abandoned( LDAP *ld, int msgid )
+ldap_mark_abandoned( LDAP *ld, ber_int_t msgid )
 {
        int     i;
 
@@ -728,11 +757,19 @@ ldap_mark_abandoned( LDAP *ld, int msgid )
 int
 cldap_getmsg( LDAP *ld, struct timeval *timeout, BerElement *ber )
 {
-       int             rc;
-       unsigned long   tag, len;
+       int     rc;
+       ber_tag_t       tag;
+       ber_len_t       len;
+
+       if ( ! ber_pvt_sb_data_ready(&ld->ld_sb) ) {
+               /* restored from ldap_select1() in result.c version 1.24 */
+               fd_set  readfds;
+               if ( ldap_int_tblsize == 0 )
+                       ldap_int_ip_init();
+               FD_ZERO( &readfds );
+               FD_SET( ber_pvt_sb_get_desc(&ld->ld_sb), &readfds );
+               rc = select( ldap_int_tblsize, &readfds, 0, 0, timeout );
 
-       if ( ld->ld_sb.sb_ber.ber_ptr >= ld->ld_sb.sb_ber.ber_end ) {
-               rc = ldap_select1( ld, timeout );
                if ( rc == -1 || rc == 0 ) {
                        ld->ld_errno = (rc == -1 ? LDAP_SERVER_DOWN :
                            LDAP_TIMEOUT);
@@ -748,6 +785,6 @@ cldap_getmsg( LDAP *ld, struct timeval *timeout, BerElement *ber )
                return( -1 );
        }
 
-       return( tag );
+       return( 0 );
 }
 #endif /* LDAP_CONNECTIONLESS */