http_send: shutdown clients even if status is not yet HTTP_STREAMING

It's better to let the client reconnect than to send a stream it did not
expect. Clean up a superfluous return statement.

diff --git a/http_send.c b/http_send.c
index 6aa82e448b558c4b4068daaed70c8930d1f1e9e3..08d20ec498299beec40de93ccb58461c69c33342 100644 (file)
--- a/http_send.c
+++ b/http_send.c
@@ -124,7 +124,6 @@ static void http_shutdown_client(struct http_client *hc, const char *msg)
        }
        list_del(&hc->node);
        free(hc);
-       return;
 }
 
 static void http_shutdown_clients_real(void)
@@ -137,8 +136,7 @@ static void http_shutdown_clients(void)
 {
        struct http_client *hc, *tmp;
        list_for_each_entry_safe(hc, tmp, &clients, node)
-               if (hc->status == HTTP_STREAMING)
-                       http_shutdown_client(hc, "afs request");
+               http_shutdown_client(hc, "afs request");
 }
 
 static int http_send_msg(struct http_client *hc, const char *msg)