]> git.ipfire.org Git - thirdparty/openldap.git/commitdiff
Update TLS codes (cert checking, fix crash condition)
authorKurt Zeilenga <kurt@openldap.org>
Wed, 17 Jan 2001 05:35:24 +0000 (05:35 +0000)
committerKurt Zeilenga <kurt@openldap.org>
Wed, 17 Jan 2001 05:35:24 +0000 (05:35 +0000)
Update ref chasing code

CHANGES
configure
include/ldap.h
include/ldap_pvt.h
libraries/libldap/ldap-int.h
libraries/libldap/request.c
libraries/libldap/result.c
libraries/libldap/url.c

diff --git a/CHANGES b/CHANGES
index 09b034938ecdd1e1098ad9a767f8c50348bdf07c..6b94b324d416b11f8557a79020e7bac6e6620567 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -4,15 +4,22 @@ OpenLDAP 2.0.8 Release Engineering
        Fixed slapd empty OID value bug
        Fixed slapd waitpid() bug
        Fixed slapd gai_strerror() bug
+       Fixed slapd unrecognized objectClass bug
        Fixed back-ldap protocol versioning bug
        Fixed back-passwd nbase bug
        Fixed back-shell unbind response bug
+       Fixed back-ldbm oldSuperior bug
+       Fixed libldap SASL scred handling
+       Fixed libldap TLS crash bug
+       Update libldap TLS certificate handling
+       Update libldap referral/reference handling
        Updated maildap
        Updated ldaptcl
        Build Environment
                Updated top.mk
                Added MacOS support
                Trim experimental options
+               Update schema files
        Documentation
                Updated ldapdelete(1)
                Updated ldapsearch(1)
index cd3cf3262c14b18e60547deeb0c67769cd5711a6..7be93238eac6389ca61c585c3f3fc20256abca5b 100755 (executable)
--- a/configure
+++ b/configure
@@ -1,6 +1,6 @@
 #! /bin/sh
 # $OpenLDAP$
-# from OpenLDAP: pkg/ldap/configure.in,v 1.223.2.21 2001/01/17 01:17:49 kurt Exp  
+# from OpenLDAP: pkg/ldap/configure.in,v 1.223.2.22 2001/01/17 01:24:30 kurt Exp  
 
 # Copyright 1998-2000 The OpenLDAP Foundation.  All Rights Reserved.
 # 
index c906919a811c525e2f9d3a8e6b90a9761b9ad18a..9741e576187591748bbf04eeb1df6eee5046ed44 100644 (file)
@@ -320,6 +320,7 @@ typedef struct ldapcontrol {
 #define LDAP_SUBSTRING_FINAL   ((ber_tag_t) 0x82U)     /* context specific */
 
 /* search scopes */
+#define LDAP_SCOPE_DEFAULT     ((ber_int_t) -1)
 #define LDAP_SCOPE_BASE                ((ber_int_t) 0x0000)
 #define LDAP_SCOPE_ONELEVEL    ((ber_int_t) 0x0001)
 #define LDAP_SCOPE_SUBTREE     ((ber_int_t) 0x0002)
index 12eda79f1724af53478ec72bb4bec320c7c660a1..3a80c18404c4dab02a0549885d6546ea991eae29 100644 (file)
@@ -32,6 +32,12 @@ LDAP_F ( int )
 ldap_pvt_url_scheme2tls LDAP_P((
        const char * ));
 
+struct ldap_url_desc; /* avoid pulling in <ldap.h> */
+
+LDAP_F( int )
+ldap_url_parse_ext LDAP_P((
+       LDAP_CONST char *url,
+       struct ldap_url_desc **ludpp ));
 
 LDAP_F ( int )
 ldap_pvt_domain2dn LDAP_P((
index 8247624d330b9f41984433d076bbc3ad42be9e23..1d218b8a8adec8fa60010f909852ef4fe99729a4 100644 (file)
@@ -439,8 +439,10 @@ LDAP_F (void) ldap_free_request( LDAP *ld, LDAPRequest *lr );
 LDAP_F (void) ldap_free_connection( LDAP *ld, LDAPConn *lc, int force, int unbind );
 LDAP_F (void) ldap_dump_connection( LDAP *ld, LDAPConn *lconns, int all );
 LDAP_F (void) ldap_dump_requests_and_responses( LDAP *ld );
-LDAP_F (int) ldap_chase_referrals( LDAP *ld, LDAPRequest *lr, char **errstrp, int *hadrefp );
-LDAP_F (int) ldap_chase_v3referrals( LDAP *ld, LDAPRequest *lr, char **refs, int sref, char **referralsp, int *hadrefp );
+LDAP_F (int) ldap_chase_referrals( LDAP *ld, LDAPRequest *lr,
+       char **errstrp, int sref, int *hadrefp );
+LDAP_F (int) ldap_chase_v3referrals( LDAP *ld, LDAPRequest *lr,
+       char **refs, int sref, char **referralsp, int *hadrefp );
 LDAP_F (int) ldap_append_referral( LDAP *ld, char **referralsp, char *s );
 
 /*
index d174308191c5416df3ff65a15d0faab9d6d6d342..b1e7e170c29033d79b6d72606801247bcf5d99a6 100644 (file)
 static LDAPConn *find_connection LDAP_P(( LDAP *ld, LDAPURLDesc *srv, int any ));
 static void use_connection LDAP_P(( LDAP *ld, LDAPConn *lc ));
 
-
-static BerElement *re_encode_request LDAP_P((
-       LDAP *ld,
+static BerElement *
+re_encode_request( LDAP *ld,
        BerElement *origber,
-    ber_int_t msgid,
-       char **dnp,
-       int      *type));
-
+       ber_int_t msgid,
+       int sref,
+       LDAPURLDesc *srv,
+       int *type );
 
 BerElement *
 ldap_alloc_ber_with_options( LDAP *ld )
@@ -628,7 +627,7 @@ ldap_chase_v3referrals( LDAP *ld, LDAPRequest *lr, char **refs, int sref, char *
        /* parse out & follow referrals */
        for( i=0; refarray[i] != NULL; i++) {
                /* Parse the referral URL */
-               if (( rc = ldap_url_parse( refarray[i], &srv)) != LDAP_SUCCESS) {
+               if (( rc = ldap_url_parse_ext( refarray[i], &srv)) != LDAP_SUCCESS) {
                        ld->ld_errno = rc;
                        rc = -1;
                        goto done;
@@ -657,7 +656,8 @@ ldap_chase_v3referrals( LDAP *ld, LDAPRequest *lr, char **refs, int sref, char *
                                 * if two search references come in one behind the other
                                 * for the same server with different contexts.
                                 */
-                               Debug( LDAP_DEBUG_TRACE, "ldap_chase_v3referrals: queue referral \"%s\"\n",
+                               Debug( LDAP_DEBUG_TRACE,
+                                       "ldap_chase_v3referrals: queue referral \"%s\"\n",
                                        refarray[i], 0, 0);
                                if( lc->lconn_rebind_queue == NULL ) {
                                        /* Create a referral list */
@@ -698,17 +698,21 @@ ldap_chase_v3referrals( LDAP *ld, LDAPRequest *lr, char **refs, int sref, char *
                 */
 
                /* For references we don't want old dn if new dn empty */
-               if ( sref && srv->lud_dn == NULL )
+               if ( sref && srv->lud_dn == NULL ) {
                        srv->lud_dn = LDAP_STRDUP( "" );
+               }
 
-               if (( ber = re_encode_request( ld, origreq->lr_ber,
-                           ++ld->ld_msgid, &srv->lud_dn, &rinfo.ri_request )) == NULL ) {
+               ber = re_encode_request( ld, origreq->lr_ber, ++ld->ld_msgid,
+                       sref, srv, &rinfo.ri_request );
+
+               if( ber == NULL ) {
                        ld->ld_errno = LDAP_ENCODING_ERROR;
                        rc = -1;
                        goto done;
                }
 
-               Debug( LDAP_DEBUG_TRACE, "ldap_chase_v3referral: msgid %d, url \"%s\"\n",
+               Debug( LDAP_DEBUG_TRACE,
+                       "ldap_chase_v3referral: msgid %d, url \"%s\"\n",
                        lr->lr_msgid, refarray[i], 0);
 
                /* Send the new request to the server - may require a bind */
@@ -782,10 +786,14 @@ done:
  * XXX merging of errors in this routine needs to be improved
  */
 int
-ldap_chase_referrals( LDAP *ld, LDAPRequest *lr, char **errstrp, int *hadrefp )
+ldap_chase_referrals( LDAP *ld,
+       LDAPRequest *lr,
+       char **errstrp,
+       int sref,
+       int *hadrefp )
 {
-       int             rc, count, len, newdn;
-       char            *p, *ports, *ref, *tmpref, *refdn, *unfollowed;
+       int             rc, count, len;
+       char            *p, *ref, *unfollowed;
        LDAPRequest     *origreq;
        LDAPURLDesc     *srv;
        BerElement      *ber;
@@ -824,7 +832,7 @@ ldap_chase_referrals( LDAP *ld, LDAPRequest *lr, char **errstrp, int *hadrefp )
        /* find original request */
        for ( origreq = lr; origreq->lr_parent != NULL;
             origreq = origreq->lr_parent ) {
-               ;
+               /* empty */;
        }
 
        unfollowed = NULL;
@@ -832,82 +840,50 @@ ldap_chase_referrals( LDAP *ld, LDAPRequest *lr, char **errstrp, int *hadrefp )
 
        /* parse out & follow referrals */
        for ( ref = p; rc == 0 && ref != NULL; ref = p ) {
-
                if (( p = strchr( ref, '\n' )) != NULL ) {
                        *p++ = '\0';
                } else {
                        p = NULL;
                }
 
-               /* copy the complete referral for rebind process */
-               rinfo.ri_url = LDAP_STRDUP( ref );
+               rc = ldap_url_parse_ext( ref, &srv );
 
-               ldap_pvt_hex_unescape( ref );
-               len = strlen( ref );
-
-               /* FIXME: we should use the URL Parser */
-
-               if ( len > LDAP_LDAP_REF_STR_LEN && strncasecmp( ref,
-                   LDAP_LDAP_REF_STR, LDAP_LDAP_REF_STR_LEN ) == 0 ) {
-                       Debug( LDAP_DEBUG_TRACE,
-                           "chasing LDAP referral: <%s>\n", ref, 0, 0 );
-                       tmpref = ref + LDAP_LDAP_REF_STR_LEN;
-               } else {
+               if ( rc != LDAP_URL_SUCCESS ) {
                        Debug( LDAP_DEBUG_TRACE,
                            "ignoring unknown referral <%s>\n", ref, 0, 0 );
                        rc = ldap_append_referral( ld, &unfollowed, ref );
                        *hadrefp = 1;
-                       LDAP_FREE( rinfo.ri_url );
-                       rinfo.ri_url = NULL;
                        continue;
                }
 
+               if( srv->lud_dn != NULL && srv->lud_dn == '\0' ) {
+                       LDAP_FREE( srv->lud_dn );
+                       srv->lud_dn = NULL;
+               }
+
+               Debug( LDAP_DEBUG_TRACE,
+                   "chasing LDAP referral: <%s>\n", ref, 0, 0 );
+
                *hadrefp = 1;
 
-               if (( refdn = strchr( tmpref, '/' )) != NULL ) {
-                       *refdn++ = '\0';
-                       newdn = refdn[0] != '?' && refdn[0] != '\0';
-                       if( !newdn ) refdn = NULL;
-               } else {
-                       newdn = 0;
-               }
+               ber = re_encode_request( ld, origreq->lr_ber,
+                   ++ld->ld_msgid, sref, srv, &rinfo.ri_request );
 
-               if (( ber = re_encode_request( ld, origreq->lr_ber,
-                   ++ld->ld_msgid, &refdn, &rinfo.ri_request )) == NULL ) {
-                       return( -1 );
+               if( ber == NULL ) {
+                       return -1 ;
                }
 
-                       if (( srv = (LDAPURLDesc *)LDAP_CALLOC( 1,
-                           sizeof( LDAPURLDesc ))) == NULL ) {
-                               ber_free( ber, 1 );
-                               ld->ld_errno = LDAP_NO_MEMORY;
-                               return( -1 );
-                       }
+               /* copy the complete referral for rebind process */
+               rinfo.ri_url = LDAP_STRDUP( ref );
 
-                       if (( srv->lud_scheme = LDAP_STRDUP("ldap")) == NULL ) {
-                               LDAP_FREE( (char *)srv );
-                               ber_free( ber, 1 );
-                               ld->ld_errno = LDAP_NO_MEMORY;
-                               return( -1 );
-                       }
+               rinfo.ri_msgid = origreq->lr_origid;
 
-                       if (( srv->lud_host = LDAP_STRDUP( tmpref )) == NULL ) {
-                               LDAP_FREE( (char *)srv );
-                               ber_free( ber, 1 );
-                               ld->ld_errno = LDAP_NO_MEMORY;
-                               return( -1 );
-                       }
+               rc = ldap_send_server_request( ld, ber, ld->ld_msgid,
+                   lr, srv, NULL, &rinfo );
 
-                       if (( ports = strchr( srv->lud_host, ':' )) != NULL ) {
-                               *ports++ = '\0';
-                               srv->lud_port = atoi( ports );
-                       } else {
-                               srv->lud_port = (LDAP_INT_GLOBAL_OPT())->ldo_defport;
-                       }
+               LDAP_FREE( rinfo.ri_url );
 
-               rinfo.ri_msgid = origreq->lr_origid;
-               if ( srv != NULL && ldap_send_server_request( ld, ber, ld->ld_msgid,
-                   lr, srv, NULL, &rinfo ) >= 0 ) {
+               if( rc >= 0 ) {
                        ++count;
                } else {
                        Debug( LDAP_DEBUG_ANY,
@@ -915,14 +891,8 @@ ldap_chase_referrals( LDAP *ld, LDAPRequest *lr, char **errstrp, int *hadrefp )
                            ldap_err2string( ld->ld_errno ), 0, 0 );
                        rc = ldap_append_referral( ld, &unfollowed, ref );
                }
-               LDAP_FREE( rinfo.ri_url);
 
-               if (srv != NULL)
-                       ldap_free_urllist(srv);
-
-               if ( !newdn && refdn != NULL ) {
-                       LDAP_FREE( refdn );
-               }
+               ldap_free_urllist(srv);
        }
 
        LDAP_FREE( *errstrp );
@@ -965,7 +935,12 @@ ldap_append_referral( LDAP *ld, char **referralsp, char *s )
 
 
 static BerElement *
-re_encode_request( LDAP *ld, BerElement *origber, ber_int_t msgid, char **dnp, int *type )
+re_encode_request( LDAP *ld,
+       BerElement *origber,
+       ber_int_t msgid,
+       int sref,
+       LDAPURLDesc *srv,
+       int *type )
 {
        /*
         * XXX this routine knows way too much about how the lber library works!
@@ -973,13 +948,16 @@ re_encode_request( LDAP *ld, BerElement *origber, ber_int_t msgid, char **dnp, i
        ber_int_t       along;
        ber_tag_t       tag;
        ber_int_t       ver;
+       ber_int_t       scope;
        int             rc;
        BerElement      tmpber, *ber;
        char            *orig_dn;
+       char            *dn;
 
        Debug( LDAP_DEBUG_TRACE,
            "re_encode_request: new msgid %ld, new dn <%s>\n",
-           (long) msgid, ( *dnp == NULL ) ? "NONE" : *dnp, 0 );
+           (long) msgid,
+               ( srv == NULL || srv->lud_dn == NULL) ? "NONE" : srv->lud_dn, 0 );
 
        tmpber = *origber;
 
@@ -1005,6 +983,22 @@ re_encode_request( LDAP *ld, BerElement *origber, ber_int_t msgid, char **dnp, i
                /* delete requests don't have a DN wrapping sequence */
                rc = ber_scanf( &tmpber, "a", &orig_dn );
 
+       } else if ( tag == LDAP_REQ_SEARCH ) {
+               /* search requests need to be re-scope-ed */
+               rc = ber_scanf( &tmpber, "{ae" /*"}"*/, &orig_dn, &scope );
+
+               if( srv->lud_scope != LDAP_SCOPE_DEFAULT ) {
+                       /* use the scope provided in reference */
+                       scope = srv->lud_scope;
+
+               } else if ( sref && scope != LDAP_SCOPE_SUBTREE ) {
+                       /* use scope implied by previous operation */
+                       /*   base -> base */
+                       /*   one -> base */
+                       /*   subtree -> subtree */
+                       scope = LDAP_SCOPE_BASE;
+               }
+
        } else {
                rc = ber_scanf( &tmpber, "{a" /*}*/, &orig_dn );
        }
@@ -1014,28 +1008,32 @@ re_encode_request( LDAP *ld, BerElement *origber, ber_int_t msgid, char **dnp, i
                return NULL;
        }
 
-       if ( *dnp == NULL ) {
-               *dnp = orig_dn;
-       } else {
-               LDAP_FREE( orig_dn );
+       if (( ber = ldap_alloc_ber_with_options( ld )) == NULL ) {
+               return NULL;
        }
 
-       if (( ber = ldap_alloc_ber_with_options( ld )) == NULL ) {
-               return( NULL );
+       if ( srv->lud_dn == NULL ) {
+               dn = orig_dn;
+       } else {
+               dn = srv->lud_dn;
        }
 
        if ( tag == LDAP_REQ_BIND ) {
-               rc = ber_printf( ber, "{it{is" /*}}*/, msgid, tag, ver, *dnp );
+               rc = ber_printf( ber, "{it{is" /*}}*/, msgid, tag, ver, dn );
        } else if ( tag == LDAP_REQ_DELETE ) {
-               rc = ber_printf( ber, "{itsN}", msgid, tag, *dnp );
+               rc = ber_printf( ber, "{itsN}", msgid, tag, dn );
+       } else if ( tag == LDAP_REQ_SEARCH ) {
+               rc = ber_printf( ber, "{it{se" /*}}*/, msgid, tag, dn, scope );
        } else {
-               rc = ber_printf( ber, "{it{s" /*}}*/, msgid, tag, *dnp );
+               rc = ber_printf( ber, "{it{s" /*}}*/, msgid, tag, dn );
        }
 
+       LDAP_FREE( orig_dn );
+
        if ( rc == -1 ) {
                ld->ld_errno = LDAP_ENCODING_ERROR;
                ber_free( ber, 1 );
-               return( NULL );
+               return NULL;
        }
 
        if ( tag != LDAP_REQ_DELETE && (
@@ -1045,7 +1043,7 @@ re_encode_request( LDAP *ld, BerElement *origber, ber_int_t msgid, char **dnp, i
        {
                ld->ld_errno = LDAP_ENCODING_ERROR;
                ber_free( ber, 1 );
-               return( NULL );
+               return NULL;
        }
 
 #ifdef LDAP_DEBUG
@@ -1057,7 +1055,7 @@ re_encode_request( LDAP *ld, BerElement *origber, ber_int_t msgid, char **dnp, i
 #endif /* LDAP_DEBUG */
 
        *type = tag;    /* return request type */
-       return( ber );
+       return ber;
 }
 
 
index 75f89e07473ab63982575577db1d03716388f5ee..f9e70a2cf2778aaea0b864543277ba5539b045b2 100644 (file)
@@ -334,7 +334,7 @@ try_read1msg(
        ber_int_t msgid,
        int all,
        Sockbuf *sb,
-    LDAPConn *lc,
+       LDAPConn *lc,
        LDAPMessage **result )
 {
        BerElement      *ber;
@@ -426,7 +426,8 @@ try_read1msg(
                return( -1 );
        }
 
-       Debug( LDAP_DEBUG_TRACE, "ldap_read: message type %s msgid %ld, original id %ld\n",
+       Debug( LDAP_DEBUG_TRACE,
+               "ldap_read: message type %s msgid %ld, original id %ld\n",
            ldap_int_msgtype2str( tag ),
                (long) lr->lr_msgid, (long) lr->lr_origid );
 
@@ -534,14 +535,13 @@ try_read1msg(
                        tmpber = *ber;  /* struct copy */
                        if ( v3ref == 1 ) {
                                ; /* V3 search reference or V3 referral sucessfully chased */
-                       } else
-                       if ( ber_scanf( &tmpber, "{iaa}", &lderr,
+                       } else if ( ber_scanf( &tmpber, "{iaa}", &lderr,
                            &lr->lr_res_matched, &lr->lr_res_error )
                            != LBER_ERROR ) {
                                if ( lderr != LDAP_SUCCESS ) {
                                        /* referrals are in error string */
                                        refer_cnt = ldap_chase_referrals( ld, lr,
-                                           &lr->lr_res_error, &hadref );
+                                               &lr->lr_res_error, -1, &hadref );
                                        lr->lr_status = LDAP_REQST_COMPLETED;
                                        Debug( LDAP_DEBUG_TRACE,
                                            "read1msg:  V2 referral chased, mark request completed, id = %d\n", lr->lr_msgid, 0, 0);
index cac3d31ed44146df323795d1f0ebea2879cb62f2..bc12e22719d997586822c174924d2e966a30cba4 100644 (file)
@@ -208,7 +208,7 @@ static int str2scope( const char *p )
 
 
 int
-ldap_url_parse( LDAP_CONST char *url_in, LDAPURLDesc **ludpp )
+ldap_url_parse_ext( LDAP_CONST char *url_in, LDAPURLDesc **ludpp )
 {
 /*
  *  Pick apart the pieces of an LDAP URL.
@@ -231,7 +231,7 @@ ldap_url_parse( LDAP_CONST char *url_in, LDAPURLDesc **ludpp )
         * because a call to LDAP_INT_GLOBAL_OPT() will try to allocate
         * the options and cause infinite recursion
         */
-       Debug( LDAP_DEBUG_TRACE, "ldap_url_parse(%s)\n", url_in, 0, 0 );
+       Debug( LDAP_DEBUG_TRACE, "ldap_url_parse_ext(%s)\n", url_in, 0, 0 );
 #endif
 
        *ludpp = NULL;  /* pessimistic */
@@ -275,7 +275,7 @@ ldap_url_parse( LDAP_CONST char *url_in, LDAPURLDesc **ludpp )
        ludp->lud_dn = NULL;
        ludp->lud_attrs = NULL;
        ludp->lud_filter = NULL;
-       ludp->lud_scope = LDAP_SCOPE_BASE;
+       ludp->lud_scope = LDAP_SCOPE_DEFAULT;
        ludp->lud_filter = NULL;
        ludp->lud_exts = NULL;
 
@@ -538,6 +538,27 @@ ldap_url_parse( LDAP_CONST char *url_in, LDAPURLDesc **ludpp )
        return LDAP_URL_SUCCESS;
 }
 
+int
+ldap_url_parse( LDAP_CONST char *url_in, LDAPURLDesc **ludpp )
+{
+       int rc = ldap_url_parse_ext( url_in, ludpp );
+
+       if( rc != LDAP_URL_SUCCESS ) {
+               return rc;
+       }
+
+       if ((*ludpp)->lud_scope == LDAP_SCOPE_DEFAULT) {
+               (*ludpp)->lud_scope = LDAP_SCOPE_BASE;
+       }
+
+       if ((*ludpp)->lud_host != NULL && *(*ludpp)->lud_host == '\0') {
+               LDAP_FREE( (*ludpp)->lud_host );
+               (*ludpp)->lud_host = NULL;
+       }
+
+       return rc;
+}
+
 LDAPURLDesc *
 ldap_url_dup ( LDAPURLDesc *ludp )
 {