Fix some duplicated IP address parsing code.

[cubemap] / config.cpp

diff --git a/config.cpp b/config.cpp
index f5182aac1512e9b8307ac57d16ada0b639260257..3122cc568de0514f9386ad7844b3ffc478cee052 100644 (file)
--- a/config.cpp
+++ b/config.cpp
@@ -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");
@@ -119,13 +177,30 @@ bool parse_port(const ConfigLine &line, Config *config)
                return false;
        }
 
+       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.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);
+       acceptor.addr = CreateAnyAddress(port);
+
+       config->acceptors.push_back(acceptor);
+       return true;
+}
+
+bool parse_listen(const ConfigLine &line, Config *config)
+{
+       if (line.arguments.size() != 1) {
+               log(ERROR, "'listen' takes exactly one argument");
                return false;
        }
 
+       AcceptorConfig acceptor;
+       if (!parse_hostport(line.arguments[0], &acceptor.addr)) {
+               return false;
+       }
        config->acceptors.push_back(acceptor);
        return true;
 }
@@ -256,50 +331,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(line.arguments[0], &udpstream.dst)) {
+               return false;
        }
 
        map<string, string>::const_iterator src_it = line.parameters.find("src");
@@ -375,6 +408,8 @@ bool parse_error_log(const ConfigLine &line, Config *config)
        return true;
 }
 
+}  // namespace
+
 bool parse_config(const string &filename, Config *config)
 {
        vector<ConfigLine> lines;
@@ -423,6 +458,10 @@ bool parse_config(const string &filename, Config *config)
                        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;