Fix a bug in udp_send_buf().

In case multiple targets are defined and a write for one of them
fails, the old code would return early rather than continue to send
the chunk to the other targets.

diff --git a/udp_send.c b/udp_send.c
index f2a430b6b774e9650cad9cb1aba864b4851fa3a9..7704b0a1a8f01b1580bae58c59a2817ca324371e 100644 (file)
--- a/udp_send.c
+++ b/udp_send.c
@@ -77,8 +77,10 @@ static void udp_send_buf(char *buf, size_t len)
                if (ut->fd < 0)
                        continue;
                ret = write_nonblock(ut->fd, buf, len, len);
-               if (ret < 0) /* TODO: Use chunk queueing */
-                       return udp_delete_target(ut, "send error");
+               if (ret < 0) {
+                       udp_delete_target(ut, para_strerror(-ret));
+                       continue;
+               }
                if (ret != len)
                        PARA_WARNING_LOG("short write %zu/%zu\n", ret, len);
        }