]> git.sesse.net Git - cubemap/blobdiff - config.cpp
Check the return value of fclose() in config.cpp.
[cubemap] / config.cpp
index f17302a4e92c4b136a2d1c182d8aef4ff313c8dd..f1b05c1821035819debc55d364c05189a774fed9 100644 (file)
@@ -11,6 +11,7 @@
 #include <utility>
 #include <vector>
 
+#include "acceptor.h"
 #include "config.h"
 #include "log.h"
 #include "parse.h"
@@ -25,6 +26,63 @@ struct ConfigLine {
        map<string, string> parameters;
 };
 
+namespace {
+
+bool parse_hostport(const string &hostport, sockaddr_in6 *addr)
+{
+       memset(addr, 0, sizeof(*addr));
+       addr->sin6_family = AF_INET6;
+
+       string port_string;
+
+       // See if the argument if on the type [ipv6addr]:port.
+       if (!hostport.empty() && hostport[0] == '[') {
+               size_t split = hostport.find("]:");
+               if (split == string::npos) {
+                       log(ERROR, "address '%s' is malformed; must be either [ipv6addr]:port or ipv4addr:port");
+                       return false;
+               }
+
+               string host(hostport.begin() + 1, hostport.begin() + split);
+               port_string = hostport.substr(split + 2);
+
+               if (inet_pton(AF_INET6, host.c_str(), &addr->sin6_addr) != 1) {
+                       log(ERROR, "'%s' is not a valid IPv6 address");
+                       return false;
+               }
+       } else {
+               // OK, then it must be ipv4addr:port.
+               size_t split = hostport.find(":");
+               if (split == string::npos) {
+                       log(ERROR, "address '%s' is malformed; must be either [ipv6addr]:port or ipv4addr:port");
+                       return false;
+               }
+
+               string host(hostport.begin(), hostport.begin() + split);
+               port_string = hostport.substr(split + 1);
+
+               // Parse to an IPv4 address, then construct a mapped-v4 address from that.
+               in_addr addr4;
+
+               if (inet_pton(AF_INET, host.c_str(), &addr4) != 1) {
+                       log(ERROR, "'%s' is not a valid IPv4 address");
+                       return false;
+               }
+
+               addr->sin6_addr.s6_addr32[2] = htonl(0xffff);
+               addr->sin6_addr.s6_addr32[3] = addr4.s_addr;
+       }
+
+       int port = atoi(port_string.c_str());
+       if (port < 1 || port >= 65536) {
+               log(ERROR, "port %d is out of range (must be [1,65536>).", port);
+               return false;
+       }
+       addr->sin6_port = ntohs(port);
+
+       return true;
+}
+
 bool read_config(const string &filename, vector<ConfigLine> *lines)
 {
        FILE *fp = fopen(filename.c_str(), "r");
@@ -74,7 +132,10 @@ bool read_config(const string &filename, vector<ConfigLine> *lines)
                lines->push_back(line);
        }
 
-       fclose(fp);
+       if (fclose(fp) == EOF) {
+               log_perror(filename.c_str());
+               return false;
+       }
        return true;
 }
 
@@ -119,69 +180,32 @@ bool parse_port(const ConfigLine &line, Config *config)
                return false;
        }
 
-       AcceptorConfig acceptor;
-       acceptor.port = atoi(line.arguments[0].c_str());
-       if (acceptor.port < 1 || acceptor.port >= 65536) {
-               log(ERROR, "port %d is out of range (must be [1,65536>).", acceptor.port);
+       int port = atoi(line.arguments[0].c_str());
+       if (port < 1 || port >= 65536) {
+               log(ERROR, "port %d is out of range (must be [1,65536>).", port);
                return false;
        }
 
+       AcceptorConfig acceptor;
+       acceptor.addr = CreateAnyAddress(port);
+
        config->acceptors.push_back(acceptor);
        return true;
 }
 
-int allocate_mark_pool(int from, int to, Config *config)
+bool parse_listen(const ConfigLine &line, Config *config)
 {
-       int pool_index = -1;    
-
-       // Reuse mark pools if an identical one exists.
-       // Otherwise, check if we're overlapping some other mark pool.
-       for (size_t i = 0; i < config->mark_pools.size(); ++i) {
-               const MarkPoolConfig &pool = config->mark_pools[i];
-               if (from == pool.from && to == pool.to) {
-                       pool_index = i;
-               } else if ((from >= pool.from && from < pool.to) ||
-                          (to >= pool.from && to < pool.to)) {
-                       log(WARNING, "Mark pool %d-%d partially overlaps with %d-%d, you may get duplicate marks."
-                                    "Mark pools must either be completely disjunct, or completely overlapping.",
-                                    from, to, pool.from, pool.to);
-               }
+       if (line.arguments.size() != 1) {
+               log(ERROR, "'listen' takes exactly one argument");
+               return false;
        }
 
-       if (pool_index != -1) {
-               return pool_index;
+       AcceptorConfig acceptor;
+       if (!parse_hostport(line.arguments[0], &acceptor.addr)) {
+               return false;
        }
-
-       // No match to existing pools.
-       MarkPoolConfig pool;
-       pool.from = from;
-       pool.to = to;
-       config->mark_pools.push_back(pool);
-
-       return config->mark_pools.size() - 1;
-}
-
-bool parse_mark_pool(const string &mark_str, int *from, int *to)
-{
-        size_t split = mark_str.find_first_of('-');
-        if (split == string::npos) {
-                log(ERROR, "Invalid mark specification '%s' (expected 'X-Y').",
-                        mark_str.c_str());
-                return false;
-        }
-
-        string from_str(mark_str.begin(), mark_str.begin() + split);
-        string to_str(mark_str.begin() + split + 1, mark_str.end());
-        *from = atoi(from_str.c_str());
-        *to = atoi(to_str.c_str());
-
-        if (*from <= 0 || *from >= 65536 || *to <= 0 || *to >= 65536) {
-                log(ERROR, "Mark pool range %d-%d is outside legal range [1,65536>.",
-                        *from, *to);
-                return false;
-        }
-
-        return true;
+       config->acceptors.push_back(acceptor);
+       return true;
 }
 
 bool parse_stream(const ConfigLine &line, Config *config)
@@ -222,16 +246,12 @@ bool parse_stream(const ConfigLine &line, Config *config)
                return false;
        }
 
-       // Parse marks, if so desired.
-       map<string, string>::const_iterator mark_parm_it = line.parameters.find("mark");
-       if (mark_parm_it == line.parameters.end()) {
-               stream.mark_pool = -1;
+       // Parse the pacing rate, converting from kilobits to bytes as needed.
+       map<string, string>::const_iterator pacing_rate_it = line.parameters.find("pacing_rate_kbit");
+       if (pacing_rate_it == line.parameters.end()) {
+               stream.pacing_rate = ~0U;
        } else {
-               int from, to;
-               if (!parse_mark_pool(mark_parm_it->second, &from, &to)) {
-                       return false;
-               }
-               stream.mark_pool = allocate_mark_pool(from, to, config);
+               stream.pacing_rate = atoi(pacing_rate_it->second.c_str()) * 1024 / 8;
        }
 
        config->streams.push_back(stream);
@@ -248,50 +268,8 @@ bool parse_udpstream(const ConfigLine &line, Config *config)
        UDPStreamConfig udpstream;
 
        string hostport = line.arguments[0];
-
-       // See if the argument if on the type [ipv6addr]:port.
-       if (!hostport.empty() && hostport[0] == '[') {
-               size_t split = hostport.find("]:");
-               if (split == string::npos) {
-                       log(ERROR, "udpstream destination '%s' is malformed; must be either [ipv6addr]:port or ipv4addr:port");
-                       return false;
-               }
-
-               string host(hostport.begin() + 1, hostport.begin() + split);
-               string port = hostport.substr(split + 2);
-
-               udpstream.dst.sin6_family = AF_INET6;
-               if (inet_pton(AF_INET6, host.c_str(), &udpstream.dst.sin6_addr) != 1) {
-                       log(ERROR, "udpstream destination host '%s' is not a valid IPv6 address");
-                       return false;
-               }
-
-               udpstream.dst.sin6_port = htons(atoi(port.c_str()));  // TODO: Verify validity.
-       } else {
-               // OK, then it must be ipv4addr:port.
-               size_t split = hostport.find(":");
-               if (split == string::npos) {
-                       log(ERROR, "udpstream destination '%s' is malformed; must be either [ipv6addr]:port or ipv4addr:port");
-                       return false;
-               }
-
-               string host(hostport.begin(), hostport.begin() + split);
-               string port = hostport.substr(split + 1);
-
-               // Parse to an IPv4 address, then construct a mapped-v4 address from that.
-               in_addr addr4;
-
-               if (inet_pton(AF_INET, host.c_str(), &addr4) != 1) {
-                       log(ERROR, "udpstream destination host '%s' is not a valid IPv4 address");
-                       return false;
-               }
-
-               udpstream.dst.sin6_family = AF_INET6;
-               udpstream.dst.sin6_addr.s6_addr32[0] = 0;
-               udpstream.dst.sin6_addr.s6_addr32[1] = 0;
-               udpstream.dst.sin6_addr.s6_addr32[2] = htonl(0xffff);
-               udpstream.dst.sin6_addr.s6_addr32[3] = addr4.s_addr;
-               udpstream.dst.sin6_port = htons(atoi(port.c_str()));  // TODO: Verify validity.
+       if (!parse_hostport(hostport, &udpstream.dst)) {
+               return false;
        }
 
        map<string, string>::const_iterator src_it = line.parameters.find("src");
@@ -304,21 +282,18 @@ bool parse_udpstream(const ConfigLine &line, Config *config)
                // TODO: Verify that the URL is parseable?
        }
 
-       // Parse marks, if so desired.
-       map<string, string>::const_iterator mark_parm_it = line.parameters.find("mark");
-       if (mark_parm_it == line.parameters.end()) {
-               udpstream.mark_pool = -1;
+       // Parse the pacing rate, converting from kilobits to bytes as needed.
+       map<string, string>::const_iterator pacing_rate_it = line.parameters.find("pacing_rate_kbit");
+       if (pacing_rate_it == line.parameters.end()) {
+               udpstream.pacing_rate = ~0U;
        } else {
-               int from, to;
-               if (!parse_mark_pool(mark_parm_it->second, &from, &to)) {
-                       return false;
-               }
-               udpstream.mark_pool = allocate_mark_pool(from, to, config);
+               udpstream.pacing_rate = atoi(pacing_rate_it->second.c_str()) * 1024 / 8;
        }
 
        config->udpstreams.push_back(udpstream);
        return true;
 }
+
 bool parse_error_log(const ConfigLine &line, Config *config)
 {
        if (line.arguments.size() != 0) {
@@ -358,6 +333,8 @@ bool parse_error_log(const ConfigLine &line, Config *config)
        return true;
 }
 
+}  // namespace
+
 bool parse_config(const string &filename, Config *config)
 {
        vector<ConfigLine> lines;
@@ -381,7 +358,14 @@ bool parse_config(const string &filename, Config *config)
        bool has_stats_file = fetch_config_string(lines, "stats_file", &config->stats_file);
        bool has_stats_interval = fetch_config_int(lines, "stats_interval", &config->stats_interval);
        if (has_stats_interval && !has_stats_file) {
-               log(WARNING, "'stats_interval' given, but no 'stats_file'. No statistics will be written.");
+               log(WARNING, "'stats_interval' given, but no 'stats_file'. No client statistics will be written.");
+       }
+
+       config->input_stats_interval = 60;
+       bool has_input_stats_file = fetch_config_string(lines, "input_stats_file", &config->input_stats_file);
+       bool has_input_stats_interval = fetch_config_int(lines, "input_stats_interval", &config->input_stats_interval);
+       if (has_input_stats_interval && !has_input_stats_file) {
+               log(WARNING, "'input_stats_interval' given, but no 'input_stats_file'. No input statistics will be written.");
        }
        
        fetch_config_string(lines, "access_log", &config->access_log_file);
@@ -391,12 +375,18 @@ bool parse_config(const string &filename, Config *config)
                if (line.keyword == "num_servers" ||
                    line.keyword == "stats_file" ||
                    line.keyword == "stats_interval" ||
+                   line.keyword == "input_stats_file" ||
+                   line.keyword == "input_stats_interval" ||
                    line.keyword == "access_log") {
                        // Already taken care of, above.
                } else if (line.keyword == "port") {
                        if (!parse_port(line, config)) {
                                return false;
                        }
+               } else if (line.keyword == "listen") {
+                       if (!parse_listen(line, config)) {
+                               return false;
+                       }
                } else if (line.keyword == "stream") {
                        if (!parse_stream(line, config)) {
                                return false;