syslogd: close/re-resolve/reopen remote sockets on some sendto() errors

[oweals/busybox.git] / sysklogd / syslogd.c

diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c
index 33203697d37373b90553de27f6145cb7d207e8a8..37a119b3d7b2cd66e325cdb2195dcb9204373a3b 100644 (file)
--- a/sysklogd/syslogd.c
+++ b/sysklogd/syslogd.c
@@ -46,7 +46,7 @@
 #undef SYSLOGD_WRLOCK
 
 enum {
-       MAX_READ = 256,
+       MAX_READ = CONFIG_FEATURE_SYSLOGD_READ_BUFFER_SIZE,
        DNS_WAIT_SEC = 2 * 60,
 };
 
@@ -546,7 +546,7 @@ static int try_to_resolve_remote(remoteHost_t *rh)
                if (!rh->remoteAddr)
                        return -1;
        }
-       return socket(rh->remoteAddr->u.sa.sa_family, SOCK_DGRAM, 0);
+       return xsocket(rh->remoteAddr->u.sa.sa_family, SOCK_DGRAM, 0);
 }
 #endif
 
@@ -636,11 +636,25 @@ static void do_syslogd(void)
                                if (rh->remoteFD == -1)
                                        continue;
                        }
-                       /* Send message to remote logger, ignore possible error */
-                       /* TODO: on some errors, close and set G.remoteFD to -1
-                        * so that DNS resolution and connect is retried? */
-                       sendto(rh->remoteFD, recvbuf, sz+1, MSG_DONTWAIT,
-                               &(rh->remoteAddr->u.sa), rh->remoteAddr->len);
+
+                       /* Send message to remote logger.
+                        * On some errors, close and set remoteFD to -1
+                        * so that DNS resolution is retried.
+                        */
+                       if (sendto(rh->remoteFD, recvbuf, sz+1,
+                                       MSG_DONTWAIT | MSG_NOSIGNAL,
+                                       &(rh->remoteAddr->u.sa), rh->remoteAddr->len) == -1
+                       ) {
+                               switch (errno) {
+                               case ECONNRESET:
+                               case ENOTCONN: /* paranoia */
+                               case EPIPE:
+                                       close(rh->remoteFD);
+                                       rh->remoteFD = -1;
+                                       free(rh->remoteAddr);
+                                       rh->remoteAddr = NULL;
+                               }
+                       }
                }
 #endif
                if (!ENABLE_FEATURE_REMOTE_LOG || (option_mask32 & OPT_locallog)) {