X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=http_send.c;h=7035f27968f8f90a7d2ce89a3cb77e98130ac632;hp=97b4d919e4d3c6047bcb4283002dccf1e932150c;hb=518a2727540cf5e105d28aa8fd31bb6afad05bf6;hpb=ae0e4594c6a0312c5b4b4c0bde86f9c12253d11b diff --git a/http_send.c b/http_send.c index 97b4d919..7035f279 100644 --- a/http_send.c +++ b/http_send.c @@ -1,35 +1,29 @@ /* * Copyright (C) 2005-2007 Andre Noll * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2 of the License, or - * (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111, USA. + * Licensed under the GPL v2. For licencing details see COPYING. */ /** \file http_send.c paraslash's http sender */ +#include +#include +#include "para.h" +#include "error.h" +#include "string.h" #include "server.cmdline.h" +#include "afh.h" +#include "afs.h" #include "server.h" #include "http.h" #include "vss.h" #include "send.h" #include "list.h" #include "close_on_fork.h" -#include "error.h" #include "net.h" -#include "string.h" #include "fd.h" +#include "chunk_queue.h" /** \cond convert sock_addr_in to ascii */ #define CLIENT_ADDR(hc) inet_ntoa((hc)->addr.sin_addr) @@ -38,68 +32,56 @@ #define HTTP_ERR_MSG "HTTP/1.0 400 Bad Request\n" /** \endcond */ -/** the possible states of a client from the server's POV */ +/** The possible states of a client from the server's POV. */ enum http_status { + /** We accepted the connection on the tcp socket. */ HTTP_CONNECTED, + /** Successfully received the get request. */ HTTP_GOT_GET_REQUEST, + /** We sent the OK message back to the client. */ HTTP_SENT_OK_MSG, + /** Connection established, we might need to send the audio file header. */ HTTP_READY_TO_STREAM, + /** Connection is ready for sending audio data. */ HTTP_STREAMING, + /** We didn't receive a valid get request. */ HTTP_INVALID_GET_REQUEST }; -/** clients will be kicked if there are more than that many bytes pending */ -#define MAX_BACKLOG 40000 -/** the list of connected clients **/ +/** Clients will be kicked if there are more than that many bytes pending. */ +#define MAX_BACKLOG 400000 +/** The list of connected clients. */ static struct list_head clients; -/** the whitelist/blacklist */ +/** The whitelist/blacklist. */ static struct list_head access_perm_list; -/** describes one client that connected the tcp port of the http sender */ +/** Describes one client that connected the tcp port of the http sender. */ struct http_client { -/** the file descriptor of the client */ + /** The file descriptor of the client. */ int fd; -/** address information about the client */ + /** Address information about the client. */ struct sockaddr_in addr; -/** the client's current status */ + /** The client's current status. */ enum http_status status; -/** non-zero if we included \a fd in the read set */ + /** Non-zero if we included \a fd in the read set.*/ int check_r; -/** non-zero if we included \a fd in the write set */ + /** Non-zero if we included \a fd in the write set. */ int check_w; -/** the position of this client in the client list */ - struct list_head node; -/** the list of pending packets for this client */ - struct list_head packet_queue; -/** the number of pending bytes for this client */ - unsigned long pq_bytes; -}; - -/** - * describes one queued data packet for a client - * - * The send function of the http sender checks each client fd for writing. If a - * client fd is not ready, it tries to queue that packet for this client until - * the number of queued bytes exceeds \p MAX_BACKLOG. - */ -struct queued_packet { -/** the length of the packet in bytes */ - unsigned int len; -/** pointer to the packet data */ - char *packet; -/** position of the packet in the packet list */ + /** The position of this client in the client list. */ struct list_head node; + /** The list of pending chunks for this client. */ + struct chunk_queue *cq; }; /** - * describes one entry in the blacklist/whitelist of the http sender + * Describes one entry in the blacklist/whitelist of the http sender. */ struct access_info { - /** the address to be black/whitelisted */ + /** The address to be black/whitelisted. */ struct in_addr addr; - /** the netmask for this entry */ + /** The netmask for this entry. */ unsigned netmask; - /** the position of this entry in the access_perm_list */ + /** The position of this entry in the access_perm_list. */ struct list_head node; }; @@ -109,17 +91,12 @@ static struct sender *self; static void http_shutdown_client(struct http_client *hc, const char *msg) { - struct queued_packet *qp, *tmp; PARA_INFO_LOG("shutting down %s on fd %d (%s)\n", CLIENT_ADDR(hc), hc->fd, msg); numclients--; close(hc->fd); del_close_on_fork_list(hc->fd); - list_for_each_entry_safe(qp, tmp, &hc->packet_queue, node) { - free(qp->packet); - list_del(&qp->node); - free(qp); - } + cq_destroy(hc->cq); list_del(&hc->node); free(hc); } @@ -152,50 +129,36 @@ static int http_send_err_msg(struct http_client *hc) return http_send_msg(hc, HTTP_ERR_MSG); } -static int queue_packet(struct http_client *hc, const char *buf, size_t len) +static int send_queued_chunks(struct http_client *hc) { - struct queued_packet *qp; - if (hc->pq_bytes + len > MAX_BACKLOG) { - http_shutdown_client(hc, "packet queue overrun"); - return -E_QUEUE; - } - qp = para_malloc(sizeof(struct queued_packet)); - hc->pq_bytes += len; - qp->packet = para_malloc(len); - memcpy(qp->packet, buf, len); - qp->len = len; - list_add_tail(&qp->node, &hc->packet_queue); - PARA_INFO_LOG("%lu bytes queued for fd %d\n", hc->pq_bytes, hc->fd); - return 1; -} - -static int send_queued_packets(struct http_client *hc) -{ - int ret; - struct queued_packet *qp, *tmp; - - if (list_empty(&hc->packet_queue)) - return 1; - list_for_each_entry_safe(qp, tmp, &hc->packet_queue, node) { - ret = write_ok(hc->fd); + struct queued_chunk *qc; + while ((qc = cq_peek(hc->cq))) { + char *buf; + size_t len; + int ret = write_ok(hc->fd); if (ret <= 0) return ret? -E_WRITE_OK : 0; - ret = write(hc->fd, qp->packet, qp->len); + cq_get(qc, &buf, &len); + ret = write(hc->fd, buf, len); if (ret < 0) - return ret; - if (ret != qp->len) { - qp->len -= ret; - memmove(qp->packet, qp->packet + ret, qp->len); - return 0; - } - hc->pq_bytes -= qp->len; - free(qp->packet); - list_del(&qp->node); - free(qp); + return -E_SEND_QUEUED_CHUNK; + cq_update(hc->cq, ret); + if (ret != len) + return 1; + cq_dequeue(hc->cq); } return 1; } +static int queue_chunk_or_shutdown(struct http_client *hc, long unsigned chunk_num, + size_t sent) +{ + int ret = cq_enqueue(hc->cq, chunk_num, sent); + if (ret < 0) + http_shutdown_client(hc, "queue error"); + return ret; +} + static void http_send( long unsigned current_chunk, __a_unused long unsigned chunks_sent, const char *buf, size_t len) { @@ -207,39 +170,37 @@ static void http_send( long unsigned current_chunk, hc->status != HTTP_READY_TO_STREAM) continue; if (hc->status == HTTP_READY_TO_STREAM) { - unsigned hlen; + size_t hlen; char *hbuf = vss_get_header(&hlen); if (hbuf && hlen > 0 && current_chunk) { /* need to send header */ - PARA_INFO_LOG("queueing header: %d\n", hlen); - if (queue_packet(hc, hbuf, hlen) < 0) + PARA_INFO_LOG("queueing header: %zu\n", hlen); + if (queue_chunk_or_shutdown(hc, -1U, 0) < 0) continue; } else - PARA_INFO_LOG("%s", "no need to queue header\n"); + PARA_INFO_LOG("no need to queue header\n"); hc->status = HTTP_STREAMING; } - ret = send_queued_packets(hc); + ret = send_queued_chunks(hc); if (ret < 0) { - http_shutdown_client(hc, "send error"); + http_shutdown_client(hc, "queue send error"); continue; } if (!len) continue; if (!ret || write_ok(hc->fd) <= 0) { - PARA_INFO_LOG("fd %d not ready (%lu bytes queued)," - " trying to queue packet\n", hc->fd, - hc->pq_bytes); - queue_packet(hc, buf, len); + queue_chunk_or_shutdown(hc, current_chunk, 0); continue; } // PARA_DEBUG_LOG("sending %d -> %s\n", len, CLIENT_ADDR(hc)); ret = write(hc->fd, buf, len); +// PARA_DEBUG_LOG("ret: %d\n", ret); if (ret < 0) { http_shutdown_client(hc, "send error"); continue; } if (ret != len) - queue_packet(hc, buf + ret, len - ret); + queue_chunk_or_shutdown(hc, current_chunk, ret); } } @@ -319,7 +280,7 @@ static void http_post_select(fd_set *rfds, fd_set *wfds) goto err_out; } hc->status = HTTP_CONNECTED; - INIT_LIST_HEAD(&hc->packet_queue); + hc->cq = cq_new(MAX_BACKLOG); PARA_INFO_LOG("accepted client #%d: %s (fd %d)\n", numclients, CLIENT_ADDR(hc), hc->fd); numclients++; @@ -504,7 +465,7 @@ static void init_access_control_list(void) if (!p) goto err; *p = '\0'; - if (!inet_aton(arg, &scd.addr)) + if (!inet_pton(AF_INET, arg, &scd.addr)) goto err; scd.netmask = atoi(++p); if (scd.netmask < 0 || scd.netmask > 32) @@ -530,12 +491,12 @@ static char *http_help(void) } /** - * the init function of the http sender + * The init function of the http sender. * - * \param s pointer to the http sender struct + * \param s Pointer to the http sender struct. * - * It initializes all function pointers of \a s, init the client list and the - * acess control list as well. If autostart is wanted, open the tcp port. + * It initializes all function pointers of \a s, the client list and the access + * control list. If the autostart option was given, the tcp port is opened. */ void http_send_init(struct sender *s) {