]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/result.c
Rework ac/socket.h for HAVE_WINSOCK:
[openldap] / servers / slapd / result.c
index 1e9384f51ff2e45449568840f0770207aa58f98f..311a355f18a94d64bb7fa1b9d989838f8856761a 100644 (file)
@@ -13,7 +13,6 @@
 
 #include "slap.h"
 
-
 static void
 send_ldap_result2(
     Connection *conn,
@@ -25,7 +24,7 @@ send_ldap_result2(
 )
 {
        BerElement      *ber;
-       int             rc, sd;
+       int             rc, tmp;
        unsigned long   tag, bytes;
 
        if ( err == LDAP_PARTIAL_RESULTS && (text == NULL || *text == '\0') )
@@ -52,12 +51,14 @@ send_ldap_result2(
                break;
        }
 
+
 #ifdef LDAP_COMPAT30
        if ( (ber = ber_alloc_t( conn->c_version == 30 ? 0 : LBER_USE_DER ))
-           == NULLBER ) {
+           == NULLBER )
 #else
-       if ( (ber = der_alloc()) == NULLBER ) {
+       if ( (ber = der_alloc()) == NULLBER )
 #endif
+       {
                Debug( LDAP_DEBUG_ANY, "ber_alloc failed\n", 0, 0, 0 );
                return;
        }
@@ -83,14 +84,15 @@ send_ldap_result2(
        }
 
        /* write only one pdu at a time - wait til it's our turn */
-       ldap_pvt_thread_mutex_lock( &conn->c_pdumutex );
+       ldap_pvt_thread_mutex_lock( &conn->c_write_mutex );
+
+       /* lock the connection */
+       ldap_pvt_thread_mutex_lock( &conn->c_mutex );
 
        /* write the pdu */
        bytes = ber->ber_ptr - ber->ber_buf;
-       ldap_pvt_thread_mutex_lock( &new_conn_mutex );
-       while ( conn->c_connid == op->o_connid && ber_flush( &conn->c_sb, ber,
-           1 ) != 0 ) {
-               ldap_pvt_thread_mutex_unlock( &new_conn_mutex );
+
+       while ( ber_flush( &conn->c_sb, ber, 1 ) != 0 ) {
                /*
                 * we got an error.  if it's ewouldblock, we need to
                 * wait on the socket being writable.  otherwise, figure
@@ -102,38 +104,30 @@ send_ldap_result2(
                    : "unknown", 0 );
 
                if ( errno != EWOULDBLOCK && errno != EAGAIN ) {
-                       close_connection( conn, op->o_connid, op->o_opid );
+                       connection_closing( conn );
 
-                       ldap_pvt_thread_mutex_unlock( &conn->c_pdumutex );
+                       ldap_pvt_thread_mutex_unlock( &conn->c_mutex );
+                       ldap_pvt_thread_mutex_unlock( &conn->c_write_mutex );
                        return;
                }
 
                /* wait for socket to be write-ready */
-               ldap_pvt_thread_mutex_lock( &active_threads_mutex );
-               active_threads--;
                conn->c_writewaiter = 1;
+               slapd_set_write( conn->c_sb.sb_sd, 1 );
 
-               ldap_pvt_thread_kill( listener_tid, LDAP_SIGUSR1 );
-
-               ldap_pvt_thread_cond_wait( &conn->c_wcv, &active_threads_mutex );
-
-               if( active_threads < 1 ) {
-                       ldap_pvt_thread_cond_signal(&active_threads_cond);
-               }
-               ldap_pvt_thread_mutex_unlock( &active_threads_mutex );
-
-               ldap_pvt_thread_yield();
-               ldap_pvt_thread_mutex_lock( &new_conn_mutex );
+               ldap_pvt_thread_cond_wait( &conn->c_write_cv, &conn->c_mutex );
+               conn->c_writewaiter = 0;
        }
-       ldap_pvt_thread_mutex_unlock( &new_conn_mutex );
-       ldap_pvt_thread_mutex_unlock( &conn->c_pdumutex );
+
+       ldap_pvt_thread_mutex_unlock( &conn->c_mutex );
+       ldap_pvt_thread_mutex_unlock( &conn->c_write_mutex );
 
        ldap_pvt_thread_mutex_lock( &num_sent_mutex );
        num_bytes_sent += bytes;
        ldap_pvt_thread_mutex_unlock( &num_sent_mutex );
 
        Statslog( LDAP_DEBUG_STATS,
-           "conn=%d op=%d RESULT err=%d tag=%d nentries=%d\n", conn->c_connid,
+           "conn=%d op=%d RESULT err=%d tag=%lu nentries=%d\n", conn->c_connid,
            op->o_opid, err, tag, nentries );
 
        return;
@@ -150,12 +144,11 @@ send_ldap_result(
 {
 #ifdef LDAP_CONNECTIONLESS
        if ( op->o_cldap ) {
-               SAFEMEMCPY( (char *)conn->c_sb.sb_useaddr, &op->o_clientaddr,
-                   sizeof( struct sockaddr ));
+               lber_pvt_sb_udp_set_dst( &conn->c_sb, &op->o_clientaddr );
                Debug( LDAP_DEBUG_TRACE, "UDP response to %s port %d\n", 
                    inet_ntoa(((struct sockaddr_in *)
-                   conn->c_sb.sb_useaddr)->sin_addr ),
-                   ((struct sockaddr_in *) conn->c_sb.sb_useaddr)->sin_port,
+                   &op->o_clientaddr)->sin_addr ),
+                   ((struct sockaddr_in *) &op->o_clientaddr)->sin_port,
                    0 );
        }
 #endif
@@ -187,7 +180,7 @@ send_search_entry(
 {
        BerElement      *ber;
        Attribute       *a;
-       int             i, rc, bytes, sd;
+       int             i, rc=-1, bytes;
        struct acl      *acl;
        char            *edn;
 
@@ -262,7 +255,7 @@ send_search_entry(
                        continue;
                }
 
-               if ( ber_printf( ber, "{s[", a->a_type ) == -1 ) {
+               if (( rc = ber_printf( ber, "{s[", a->a_type )) == -1 ) {
                        Debug( LDAP_DEBUG_ANY, "ber_printf failed\n", 0, 0, 0 );
                        ber_free( ber, 1 );
                        send_ldap_result( conn, op, LDAP_OPERATIONS_ERROR,
@@ -279,9 +272,9 @@ send_search_entry(
                                        continue;
                                }
 
-                               if ( ber_printf( ber, "o",
+                               if (( rc = ber_printf( ber, "o",
                                    a->a_vals[i]->bv_val,
-                                   a->a_vals[i]->bv_len ) == -1 )
+                                   a->a_vals[i]->bv_len )) == -1 )
                                {
                                        Debug( LDAP_DEBUG_ANY,
                                            "ber_printf failed\n", 0, 0, 0 );
@@ -294,7 +287,7 @@ send_search_entry(
                        }
                }
 
-               if ( ber_printf( ber, "]}" ) == -1 ) {
+               if (( rc = ber_printf( ber, "]}" )) == -1 ) {
                        Debug( LDAP_DEBUG_ANY, "ber_printf failed\n", 0, 0, 0 );
                        ber_free( ber, 1 );
                        send_ldap_result( conn, op, LDAP_OPERATIONS_ERROR,
@@ -318,14 +311,16 @@ send_search_entry(
                return( 1 );
        }
 
+       bytes = ber->ber_ptr - ber->ber_buf;
+
        /* write only one pdu at a time - wait til it's our turn */
-       ldap_pvt_thread_mutex_lock( &conn->c_pdumutex );
+       ldap_pvt_thread_mutex_lock( &conn->c_write_mutex );
 
-       bytes = ber->ber_ptr - ber->ber_buf;
-       ldap_pvt_thread_mutex_lock( &new_conn_mutex );
-       while ( conn->c_connid == op->o_connid && ber_flush( &conn->c_sb, ber,
-           1 ) != 0 ) {
-               ldap_pvt_thread_mutex_unlock( &new_conn_mutex );
+       /* lock the connection */ 
+       ldap_pvt_thread_mutex_lock( &conn->c_mutex );
+
+       /* write the pdu */
+       while ( ber_flush( &conn->c_sb, ber, 1 ) != 0 ) {
                /*
                 * we got an error.  if it's ewouldblock, we need to
                 * wait on the socket being writable.  otherwise, figure
@@ -337,51 +332,38 @@ send_search_entry(
                    : "unknown", 0 );
 
                if ( errno != EWOULDBLOCK && errno != EAGAIN ) {
-                       close_connection( conn, op->o_connid, op->o_opid );
+                       connection_closing( conn );
 
-                       ldap_pvt_thread_mutex_unlock( &conn->c_pdumutex );
+                       ldap_pvt_thread_mutex_unlock( &conn->c_mutex );
+                       ldap_pvt_thread_mutex_unlock( &conn->c_write_mutex );
                        return( -1 );
                }
 
                /* wait for socket to be write-ready */
-               ldap_pvt_thread_mutex_lock( &active_threads_mutex );
-               active_threads--;
                conn->c_writewaiter = 1;
-               ldap_pvt_thread_kill( listener_tid, LDAP_SIGUSR1 );
-               ldap_pvt_thread_cond_wait( &conn->c_wcv, &active_threads_mutex );
-
-               if( active_threads < 1 ) {
-                       ldap_pvt_thread_cond_signal(&active_threads_cond);
-               }
-               ldap_pvt_thread_mutex_unlock( &active_threads_mutex );
+               slapd_set_write( conn->c_sb.sb_sd, 1 );
 
-               ldap_pvt_thread_yield();
-               ldap_pvt_thread_mutex_lock( &new_conn_mutex );
+               ldap_pvt_thread_cond_wait( &conn->c_write_cv, &conn->c_mutex );
+               conn->c_writewaiter = 0;
        }
-       ldap_pvt_thread_mutex_unlock( &new_conn_mutex );
-       ldap_pvt_thread_mutex_unlock( &conn->c_pdumutex );
+
+       ldap_pvt_thread_mutex_unlock( &conn->c_mutex );
+       ldap_pvt_thread_mutex_unlock( &conn->c_write_mutex );
 
        ldap_pvt_thread_mutex_lock( &num_sent_mutex );
        num_bytes_sent += bytes;
        num_entries_sent++;
        ldap_pvt_thread_mutex_unlock( &num_sent_mutex );
 
-       ldap_pvt_thread_mutex_lock( &new_conn_mutex );
-       if ( conn->c_connid == op->o_connid ) {
-               rc = 0;
-               Statslog( LDAP_DEBUG_STATS2, "conn=%d op=%d ENTRY dn=\"%s\"\n",
-                   conn->c_connid, op->o_opid, e->e_dn, 0, 0 );
-       } else {
-               rc = -1;
-       }
-       ldap_pvt_thread_mutex_unlock( &new_conn_mutex );
+       Statslog( LDAP_DEBUG_STATS2, "conn=%d op=%d ENTRY dn=\"%s\"\n",
+           conn->c_connid, op->o_opid, e->e_dn, 0, 0 );
 
        Debug( LDAP_DEBUG_TRACE, "<= send_search_entry\n", 0, 0, 0 );
 
-       return( rc );
+       rc = 0;
 
 error_return:;
-       return( 1 );
+       return( rc );
 }
 
 int
@@ -437,25 +419,3 @@ str2result(
 
        return( rc );
 }
-
-/*
- * close_connection - close a connection. takes the connection to close,
- * the connid associated with the operation generating the close (so we
- * don't accidentally close a connection that's not ours), and the opid
- * of the operation generating the close (for logging purposes).
- */
-void
-close_connection( Connection *conn, int opconnid, int opid )
-{
-       ldap_pvt_thread_mutex_lock( &new_conn_mutex );
-       if ( conn->c_sb.sb_sd != -1 && conn->c_connid == opconnid ) {
-               Statslog( LDAP_DEBUG_STATS,
-                   "conn=%d op=%d fd=%d closed errno=%d\n", conn->c_connid,
-                   opid, conn->c_sb.sb_sd, errno, 0 );
-               close( conn->c_sb.sb_sd );
-               conn->c_sb.sb_sd = -1;
-               conn->c_version = 0;
-               conn->c_protocol = 0;
-       }
-       ldap_pvt_thread_mutex_unlock( &new_conn_mutex );
-}