upd sender: Don't send FEC EOF from command handler context.

Without this, the EOF packet might be sent twice: once by the command
handler which stopped the stream and once by the server process. This
does not hurt, but results in additional unnecessary network traffic,
so return early from udp_close_target() when we're running in command
handler context.

diff --git a/udp_send.c b/udp_send.c
index 04e2982f86bf125c5b4ce8ab2d6daa9280ea0bce..96a25d370a99d3fdcb6782ef9451ae86b1cfc3a4 100644 (file)
--- a/udp_send.c
+++ b/udp_send.c
@@ -56,6 +56,8 @@ static void udp_close_target(struct sender_client *sc)
        size_t len;
        struct udp_target *ut = sc->private_data;
 
+       if (process_is_command_handler())
+               return;
        if (ut->sent_fec_eof)
                return;
        PARA_NOTICE_LOG("sending FEC EOF\n");