Tweak connection error logging.

Do not log when receiving the last bytes on a connection. Log failed
writes.

diff --git a/servers/lloadd/client.c b/servers/lloadd/client.c
index 6a5801f34d984344ac902b4f23daf2df1a21dbf0..2dfdf13960a4d92dbc912e9d2230bf4fc68eaad4 100644 (file)
--- a/servers/lloadd/client.c
+++ b/servers/lloadd/client.c
@@ -59,10 +59,13 @@ client_read_cb( evutil_socket_t s, short what, void *arg )
         int err = sock_errno();
 
         if ( err != EWOULDBLOCK && err != EAGAIN ) {
-            char ebuf[128];
-            Debug( LDAP_DEBUG_ANY, "client_read_cb: "
-                    "ber_get_next on fd %d failed errno=%d (%s)\n",
-                    c->c_fd, err, sock_errstr( err, ebuf, sizeof(ebuf) ) );
+            if ( err ) {
+                char ebuf[128];
+                Debug( LDAP_DEBUG_ANY, "client_read_cb: "
+                        "ber_get_next on fd %d failed errno=%d (%s)\n",
+                        c->c_fd, err,
+                        sock_errstr( err, ebuf, sizeof(ebuf) ) );
+            }
 
             c->c_currentber = NULL;
             ber_free( ber, 1 );
@@ -130,11 +133,13 @@ handle_requests( void *ctx, void *arg )
             int err = sock_errno();
 
             if ( err != EWOULDBLOCK && err != EAGAIN ) {
-                char ebuf[128];
-                Debug( LDAP_DEBUG_ANY, "handle_requests: "
-                        "ber_get_next on fd %d failed errno=%d (%s)\n",
-                        c->c_fd, err,
-                        sock_errstr( err, ebuf, sizeof(ebuf) ) );
+                if ( err ) {
+                    char ebuf[128];
+                    Debug( LDAP_DEBUG_ANY, "handle_requests: "
+                            "ber_get_next on fd %d failed errno=%d (%s)\n",
+                            c->c_fd, err,
+                            sock_errstr( err, ebuf, sizeof(ebuf) ) );
+                }
 
                 c->c_currentber = NULL;
                 ber_free( ber, 1 );
@@ -222,8 +227,13 @@ client_write_cb( evutil_socket_t s, short what, void *arg )
     /* We might have been beaten to flushing the data by another thread */
     if ( c->c_pendingber && ber_flush( c->c_sb, c->c_pendingber, 1 ) ) {
         int err = sock_errno();
+
         if ( err != EWOULDBLOCK && err != EAGAIN ) {
+            char ebuf[128];
             ldap_pvt_thread_mutex_unlock( &c->c_io_mutex );
+            Debug( LDAP_DEBUG_ANY, "client_write_cb: "
+                    "ber_flush on fd %d failed errno=%d (%s)\n",
+                    c->c_fd, err, sock_errstr( err, ebuf, sizeof(ebuf) ) );
             CLIENT_LOCK_DESTROY(c);
             return;
         }

diff --git a/servers/lloadd/upstream.c b/servers/lloadd/upstream.c
index 6c82c1fe2dfcae4aa27c432fd563cc5bd6f92549..6ffbaa078b478eb4488e56d73af7656e5c92af21 100644 (file)
--- a/servers/lloadd/upstream.c
+++ b/servers/lloadd/upstream.c
@@ -526,10 +526,13 @@ upstream_read_cb( evutil_socket_t s, short what, void *arg )
         int err = sock_errno();
 
         if ( err != EWOULDBLOCK && err != EAGAIN ) {
-            char ebuf[128];
-            Debug( LDAP_DEBUG_ANY, "upstream_read_cb: "
-                    "ber_get_next on fd %d failed errno=%d (%s)\n",
-                    c->c_fd, err, sock_errstr( err, ebuf, sizeof(ebuf) ) );
+            if ( err ) {
+                char ebuf[128];
+                Debug( LDAP_DEBUG_ANY, "upstream_read_cb: "
+                        "ber_get_next on fd %d failed errno=%d (%s)\n",
+                        c->c_fd, err,
+                        sock_errstr( err, ebuf, sizeof(ebuf) ) );
+            }
 
             c->c_currentber = NULL;
             ber_free( ber, 1 );
@@ -706,10 +709,12 @@ upstream_write_cb( evutil_socket_t s, short what, void *arg )
     /* We might have been beaten to flushing the data by another thread */
     if ( c->c_pendingber && ber_flush( c->c_sb, c->c_pendingber, 1 ) ) {
         int err = sock_errno();
+
         if ( err != EWOULDBLOCK && err != EAGAIN ) {
+            char ebuf[128];
             Debug( LDAP_DEBUG_ANY, "upstream_write_cb: "
-                    "error writing to connection %ld\n",
-                    c->c_connid );
+                    "ber_flush on fd %d failed errno=%d (%s)\n",
+                    c->c_fd, err, sock_errstr( err, ebuf, sizeof(ebuf) ) );
             ldap_pvt_thread_mutex_unlock( &c->c_io_mutex );
             UPSTREAM_LOCK_DESTROY(c);
             return;