Reformat all code using astyle.
[tinc] / src / net_socket.c
index 46dbdf6..c020e8f 100644 (file)
@@ -60,12 +60,14 @@ static void configure_tcp(connection_t *c) {
        if(fcntl(c->socket, F_SETFL, flags | O_NONBLOCK) < 0) {
                logger(DEBUG_ALWAYS, LOG_ERR, "fcntl for %s: %s", c->hostname, strerror(errno));
        }
+
 #elif defined(WIN32)
        unsigned long arg = 1;
 
        if(ioctlsocket(c->socket, FIONBIO, &arg) != 0) {
                logger(DEBUG_ALWAYS, LOG_ERR, "ioctlsocket for %s: %s", c->hostname, sockstrerror(sockerrno));
        }
+
 #endif
 
 #if defined(IPPROTO_TCP) && defined(TCP_NODELAY)
@@ -92,8 +94,9 @@ static bool bind_to_interface(int sd) {
        int status;
 #endif /* defined(SOL_SOCKET) && defined(SO_BINDTODEVICE) */
 
-       if(!get_config_string (lookup_config (config_tree, "BindToInterface"), &iface))
+       if(!get_config_string(lookup_config(config_tree, "BindToInterface"), &iface)) {
                return true;
+       }
 
 #if defined(SOL_SOCKET) && defined(SO_BINDTODEVICE)
        memset(&ifr, 0, sizeof(ifr));
@@ -101,11 +104,13 @@ static bool bind_to_interface(int sd) {
        ifr.ifr_ifrn.ifrn_name[IFNAMSIZ - 1] = 0;
 
        status = setsockopt(sd, SOL_SOCKET, SO_BINDTODEVICE, (void *)&ifr, sizeof(ifr));
+
        if(status) {
                logger(DEBUG_ALWAYS, LOG_ERR, "Can't bind to interface %s: %s", iface,
-                               sockstrerror(sockerrno));
+                      sockstrerror(sockerrno));
                return false;
        }
+
 #else /* if !defined(SOL_SOCKET) || !defined(SO_BINDTODEVICE) */
        logger(DEBUG_ALWAYS, LOG_WARNING, "%s not supported on this platform", "BindToInterface");
 #endif
@@ -117,21 +122,28 @@ static bool bind_to_address(connection_t *c) {
        int s = -1;
 
        for(int i = 0; i < listen_sockets && listen_socket[i].bindto; i++) {
-               if(listen_socket[i].sa.sa.sa_family != c->address.sa.sa_family)
+               if(listen_socket[i].sa.sa.sa_family != c->address.sa.sa_family) {
                        continue;
-               if(s >= 0)
+               }
+
+               if(s >= 0) {
                        return false;
+               }
+
                s = i;
        }
 
-       if(s < 0)
+       if(s < 0) {
                return false;
+       }
 
        sockaddr_t sa = listen_socket[s].sa;
-       if(sa.sa.sa_family == AF_INET)
+
+       if(sa.sa.sa_family == AF_INET) {
                sa.in.sin_port = 0;
-       else if(sa.sa.sa_family == AF_INET6)
+       } else if(sa.sa.sa_family == AF_INET6) {
                sa.in6.sin6_port = 0;
+       }
 
        if(bind(c->socket, &sa.sa, SALEN(sa.sa))) {
                logger(DEBUG_CONNECTIONS, LOG_WARNING, "Can't bind outgoing socket: %s", sockstrerror(sockerrno));
@@ -164,12 +176,15 @@ int setup_listen_socket(const sockaddr_t *sa) {
        setsockopt(nfd, SOL_SOCKET, SO_REUSEADDR, (void *)&option, sizeof(option));
 
 #if defined(IPPROTO_IPV6) && defined(IPV6_V6ONLY)
-       if(sa->sa.sa_family == AF_INET6)
+
+       if(sa->sa.sa_family == AF_INET6) {
                setsockopt(nfd, IPPROTO_IPV6, IPV6_V6ONLY, (void *)&option, sizeof(option));
+       }
+
 #endif
 
        if(get_config_string
-          (lookup_config(config_tree, "BindToInterface"), &iface)) {
+                       (lookup_config(config_tree, "BindToInterface"), &iface)) {
 #if defined(SOL_SOCKET) && defined(SO_BINDTODEVICE)
                struct ifreq ifr;
 
@@ -179,9 +194,10 @@ int setup_listen_socket(const sockaddr_t *sa) {
                if(setsockopt(nfd, SOL_SOCKET, SO_BINDTODEVICE, (void *)&ifr, sizeof(ifr))) {
                        closesocket(nfd);
                        logger(DEBUG_ALWAYS, LOG_ERR, "Can't bind to interface %s: %s", iface,
-                                  sockstrerror(sockerrno));
+                              sockstrerror(sockerrno));
                        return -1;
                }
+
 #else
                logger(DEBUG_ALWAYS, LOG_WARNING, "%s not supported on this platform", "BindToInterface");
 #endif
@@ -227,13 +243,14 @@ int setup_vpn_in_socket(const sockaddr_t *sa) {
                if(fcntl(nfd, F_SETFL, flags | O_NONBLOCK) < 0) {
                        closesocket(nfd);
                        logger(DEBUG_ALWAYS, LOG_ERR, "System call `%s' failed: %s", "fcntl",
-                                  strerror(errno));
+                              strerror(errno));
                        return -1;
                }
        }
 #elif defined(WIN32)
        {
                unsigned long arg = 1;
+
                if(ioctlsocket(nfd, FIONBIO, &arg) != 0) {
                        closesocket(nfd);
                        logger(DEBUG_ALWAYS, LOG_ERR, "Call to `%s' failed: %s", "ioctlsocket", sockstrerror(sockerrno));
@@ -246,15 +263,20 @@ int setup_vpn_in_socket(const sockaddr_t *sa) {
        setsockopt(nfd, SOL_SOCKET, SO_REUSEADDR, (void *)&option, sizeof(option));
        setsockopt(nfd, SOL_SOCKET, SO_BROADCAST, (void *)&option, sizeof(option));
 
-       if(udp_rcvbuf && setsockopt(nfd, SOL_SOCKET, SO_RCVBUF, (void *)&udp_rcvbuf, sizeof(udp_rcvbuf)))
+       if(udp_rcvbuf && setsockopt(nfd, SOL_SOCKET, SO_RCVBUF, (void *)&udp_rcvbuf, sizeof(udp_rcvbuf))) {
                logger(DEBUG_ALWAYS, LOG_WARNING, "Can't set UDP SO_RCVBUF to %i: %s", udp_rcvbuf, sockstrerror(sockerrno));
+       }
 
-       if(udp_sndbuf && setsockopt(nfd, SOL_SOCKET, SO_SNDBUF, (void *)&udp_sndbuf, sizeof(udp_sndbuf)))
+       if(udp_sndbuf && setsockopt(nfd, SOL_SOCKET, SO_SNDBUF, (void *)&udp_sndbuf, sizeof(udp_sndbuf))) {
                logger(DEBUG_ALWAYS, LOG_WARNING, "Can't set UDP SO_SNDBUF to %i: %s", udp_sndbuf, sockstrerror(sockerrno));
+       }
 
 #if defined(IPPROTO_IPV6) && defined(IPV6_V6ONLY)
-       if(sa->sa.sa_family == AF_INET6)
+
+       if(sa->sa.sa_family == AF_INET6) {
                setsockopt(nfd, IPPROTO_IPV6, IPV6_V6ONLY, (void *)&option, sizeof(option));
+       }
+
 #endif
 
 #if defined(IP_DONTFRAG) && !defined(IP_DONTFRAGMENT)
@@ -262,30 +284,38 @@ int setup_vpn_in_socket(const sockaddr_t *sa) {
 #endif
 
 #if defined(IPPROTO_IP) && defined(IP_MTU_DISCOVER) && defined(IP_PMTUDISC_DO)
+
        if(myself->options & OPTION_PMTU_DISCOVERY) {
                option = IP_PMTUDISC_DO;
                setsockopt(nfd, IPPROTO_IP, IP_MTU_DISCOVER, (void *)&option, sizeof(option));
        }
+
 #elif defined(IPPROTO_IP) && defined(IP_DONTFRAGMENT)
+
        if(myself->options & OPTION_PMTU_DISCOVERY) {
                option = 1;
                setsockopt(nfd, IPPROTO_IP, IP_DONTFRAGMENT, (void *)&option, sizeof(option));
        }
+
 #endif
 
 #if defined(IPPROTO_IPV6) && defined(IPV6_MTU_DISCOVER) && defined(IPV6_PMTUDISC_DO)
+
        if(myself->options & OPTION_PMTU_DISCOVERY) {
                option = IPV6_PMTUDISC_DO;
                setsockopt(nfd, IPPROTO_IPV6, IPV6_MTU_DISCOVER, (void *)&option, sizeof(option));
        }
+
 #elif defined(IPPROTO_IPV6) && defined(IPV6_DONTFRAG)
+
        if(myself->options & OPTION_PMTU_DISCOVERY) {
                option = 1;
                setsockopt(nfd, IPPROTO_IPV6, IPV6_DONTFRAG, (void *)&option, sizeof(option));
        }
+
 #endif
 
-       if (!bind_to_interface(nfd)) {
+       if(!bind_to_interface(nfd)) {
                closesocket(nfd);
                return -1;
        }
@@ -308,10 +338,13 @@ static void retry_outgoing_handler(void *data) {
 void retry_outgoing(outgoing_t *outgoing) {
        outgoing->timeout += 5;
 
-       if(outgoing->timeout > maxtimeout)
+       if(outgoing->timeout > maxtimeout) {
                outgoing->timeout = maxtimeout;
+       }
 
-       timeout_add(&outgoing->ev, retry_outgoing_handler, outgoing, &(struct timeval){outgoing->timeout, rand() % 100000});
+       timeout_add(&outgoing->ev, retry_outgoing_handler, outgoing, &(struct timeval) {
+               outgoing->timeout, rand() % 100000
+       });
 
        logger(DEBUG_CONNECTIONS, LOG_NOTICE, "Trying to re-establish outgoing connection in %d seconds", outgoing->timeout);
 }
@@ -358,14 +391,19 @@ static void do_outgoing_pipe(connection_t *c, char *command) {
        setenv("REMOTEPORT", port, true);
        setenv("NODE", c->name, true);
        setenv("NAME", myself->name, true);
-       if(netname)
+
+       if(netname) {
                setenv("NETNAME", netname, true);
+       }
 
        int result = system(command);
-       if(result < 0)
+
+       if(result < 0) {
                logger(DEBUG_ALWAYS, LOG_ERR, "Could not execute %s: %s", command, strerror(errno));
-       else if(result)
+       } else if(result) {
                logger(DEBUG_ALWAYS, LOG_ERR, "%s exited with non-zero status %d", command, result);
+       }
+
        exit(result);
 #else
        logger(DEBUG_ALWAYS, LOG_ERR, "Proxy type exec not supported on this platform!");
@@ -374,10 +412,12 @@ static void do_outgoing_pipe(connection_t *c, char *command) {
 }
 
 static void handle_meta_write(connection_t *c) {
-       if(c->outbuf.len <= c->outbuf.offset)
+       if(c->outbuf.len <= c->outbuf.offset) {
                return;
+       }
 
        ssize_t outlen = send(c->socket, c->outbuf.data + c->outbuf.offset, c->outbuf.len - c->outbuf.offset, 0);
+
        if(outlen <= 0) {
                if(!sockerrno || sockerrno == EPIPE) {
                        logger(DEBUG_CONNECTIONS, LOG_NOTICE, "Connection closed by %s (%s)", c->name, c->hostname);
@@ -393,8 +433,10 @@ static void handle_meta_write(connection_t *c) {
        }
 
        buffer_read(&c->outbuf, outlen);
-       if(!c->outbuf.len)
+
+       if(!c->outbuf.len) {
                io_set(&c->io, IO_READ);
+       }
 }
 
 static void handle_meta_io(void *data, int flags) {
@@ -414,20 +456,25 @@ static void handle_meta_io(void *data, int flags) {
                   | Successful | (success) | (success)   | (success) |
                   +------------+-----------+-------------+-----------+
                */
-               if (send(c->socket, NULL, 0, 0) != 0) {
-                       if (sockwouldblock(sockerrno))
+               if(send(c->socket, NULL, 0, 0) != 0) {
+                       if(sockwouldblock(sockerrno)) {
                                return;
+                       }
+
                        int socket_error;
-                       if (!socknotconn(sockerrno))
+
+                       if(!socknotconn(sockerrno)) {
                                socket_error = sockerrno;
-                       else {
+                       else {
                                socklen_t len = sizeof(socket_error);
                                getsockopt(c->socket, SOL_SOCKET, SO_ERROR, (void *)&socket_error, &len);
                        }
-                       if (socket_error) {
+
+                       if(socket_error) {
                                logger(DEBUG_CONNECTIONS, LOG_DEBUG, "Error while connecting to %s (%s): %s", c->name, c->hostname, sockstrerror(socket_error));
                                terminate_connection(c, false);
                        }
+
                        return;
                }
 
@@ -435,10 +482,11 @@ static void handle_meta_io(void *data, int flags) {
                finish_connecting(c);
        }
 
-       if(flags & IO_WRITE)
+       if(flags & IO_WRITE) {
                handle_meta_write(c);
-       else
+       } else {
                handle_meta_connection_data(c);
+       }
 }
 
 static void free_known_addresses(struct addrinfo *ai) {
@@ -454,6 +502,7 @@ bool do_outgoing_connection(outgoing_t *outgoing) {
        int result;
 
 begin:
+
        if(!outgoing->ai && !outgoing->kai) {
                if(!outgoing->cfg) {
                        logger(DEBUG_CONNECTIONS, LOG_ERR, "Could not set up a meta connection to %s", outgoing->name);
@@ -464,12 +513,14 @@ begin:
                get_config_string(outgoing->cfg, &address);
 
                space = strchr(address, ' ');
+
                if(space) {
                        port = xstrdup(space + 1);
                        *space = 0;
                } else {
-                       if(!get_config_string(lookup_config(outgoing->config_tree, "Port"), &port))
+                       if(!get_config_string(lookup_config(outgoing->config_tree, "Port"), &port)) {
                                port = xstrdup("655");
+                       }
                }
 
                outgoing->ai = str2addrinfo(address, port, SOCK_STREAM);
@@ -481,12 +532,16 @@ begin:
        }
 
        if(!outgoing->aip) {
-               if(outgoing->ai)
+               if(outgoing->ai) {
                        freeaddrinfo(outgoing->ai);
+               }
+
                outgoing->ai = NULL;
 
-               if(outgoing->kai)
+               if(outgoing->kai) {
                        free_known_addresses(outgoing->kai);
+               }
+
                outgoing->kai = NULL;
 
                goto begin;
@@ -509,10 +564,12 @@ begin:
                do_outgoing_pipe(c, proxyhost);
        } else {
                proxyai = str2addrinfo(proxyhost, proxyport, SOCK_STREAM);
+
                if(!proxyai) {
                        free_connection(c);
                        goto begin;
                }
+
                logger(DEBUG_CONNECTIONS, LOG_INFO, "Using proxy at %s port %s", proxyhost, proxyport);
                c->socket = socket(proxyai->ai_family, SOCK_STREAM, IPPROTO_TCP);
                configure_tcp(c);
@@ -531,8 +588,11 @@ begin:
        if(proxytype != PROXY_EXEC) {
 #if defined(IPPROTO_IPV6) && defined(IPV6_V6ONLY)
                int option = 1;
-               if(c->address.sa.sa_family == AF_INET6)
+
+               if(c->address.sa.sa_family == AF_INET6) {
                        setsockopt(c->socket, IPPROTO_IPV6, IPV6_V6ONLY, (void *)&option, sizeof(option));
+               }
+
 #endif
 
                bind_to_interface(c->socket);
@@ -546,8 +606,10 @@ begin:
        } else if(proxytype == PROXY_EXEC) {
                result = 0;
        } else {
-               if(!proxyai)
+               if(!proxyai) {
                        abort();
+               }
+
                result = connect(c->socket, proxyai->ai_addr, proxyai->ai_addrlen);
                freeaddrinfo(proxyai);
        }
@@ -574,7 +636,7 @@ begin:
 
        connection_add(c);
 
-       io_add(&c->io, handle_meta_io, c, c->socket, IO_READ|IO_WRITE);
+       io_add(&c->io, handle_meta_io, c, c->socket, IO_READ | IO_WRITE);
 
        return true;
 }
@@ -585,18 +647,22 @@ static struct addrinfo *get_known_addresses(node_t *n) {
        struct addrinfo *oai = NULL;
 
        for splay_each(edge_t, e, n->edge_tree) {
-               if(!e->reverse)
+               if(!e->reverse) {
                        continue;
+               }
 
                bool found = false;
+
                for(struct addrinfo *aip = ai; aip; aip = aip->ai_next) {
                        if(!sockaddrcmp(&e->reverse->address, (sockaddr_t *)aip->ai_addr)) {
                                found = true;
                                break;
                        }
                }
-               if(found)
+
+               if(found) {
                        continue;
+               }
 
                oai = ai;
                ai = xzalloc(sizeof(*ai));
@@ -619,6 +685,7 @@ void setup_outgoing_connection(outgoing_t *outgoing) {
 
        if(n && n->connection) {
                logger(DEBUG_CONNECTIONS, LOG_INFO, "Already connected to %s", outgoing->name);
+
                if(!n->connection->outgoing) {
                        n->connection->outgoing = outgoing;
                        return;
@@ -632,8 +699,10 @@ void setup_outgoing_connection(outgoing_t *outgoing) {
        outgoing->cfg = lookup_config(outgoing->config_tree, "Address");
 
        if(!outgoing->cfg) {
-               if(n)
+               if(n) {
                        outgoing->aip = outgoing->kai = get_known_addresses(n);
+               }
+
                if(!outgoing->kai) {
                        logger(DEBUG_ALWAYS, LOG_DEBUG, "No address known for %s", outgoing->name);
                        goto remove;
@@ -681,10 +750,11 @@ void handle_new_meta_connection(void *data, int flags) {
                static int samehost_burst;
                static int samehost_burst_time;
 
-               if(now.tv_sec - samehost_burst_time > samehost_burst)
+               if(now.tv_sec - samehost_burst_time > samehost_burst) {
                        samehost_burst = 0;
-               else
+               } else {
                        samehost_burst -= now.tv_sec - samehost_burst_time;
+               }
 
                samehost_burst_time = now.tv_sec;
                samehost_burst++;
@@ -702,10 +772,11 @@ void handle_new_meta_connection(void *data, int flags) {
        static int connection_burst;
        static int connection_burst_time;
 
-       if(now.tv_sec - connection_burst_time > connection_burst)
+       if(now.tv_sec - connection_burst_time > connection_burst) {
                connection_burst = 0;
-       else
+       } else {
                connection_burst -= now.tv_sec - connection_burst_time;
+       }
 
        connection_burst_time = now.tv_sec;
        connection_burst++;
@@ -786,17 +857,21 @@ void handle_new_unix_connection(void *data, int flags) {
 static void free_outgoing(outgoing_t *outgoing) {
        timeout_del(&outgoing->ev);
 
-       if(outgoing->ai)
+       if(outgoing->ai) {
                freeaddrinfo(outgoing->ai);
+       }
 
-       if(outgoing->kai)
+       if(outgoing->kai) {
                free_known_addresses(outgoing->kai);
+       }
 
-       if(outgoing->config_tree)
+       if(outgoing->config_tree) {
                exit_configuration(&outgoing->config_tree);
+       }
 
-       if(outgoing->name)
+       if(outgoing->name) {
                free(outgoing->name);
+       }
 
        free(outgoing);
 }
@@ -807,8 +882,9 @@ void try_outgoing_connections(void) {
        if(!outgoing_list) {
                outgoing_list = list_alloc((list_action_t)free_outgoing);
        } else {
-               for list_each(outgoing_t, outgoing, outgoing_list)
+               for list_each(outgoing_t, outgoing, outgoing_list) {
                        outgoing->timeout = -1;
+               }
        }
 
        /* Make sure there is one outgoing_t in the list for each ConnectTo. */
@@ -819,8 +895,8 @@ void try_outgoing_connections(void) {
 
                if(!check_id(name)) {
                        logger(DEBUG_ALWAYS, LOG_ERR,
-                                  "Invalid name for outgoing connection in %s line %d",
-                                  cfg->file, cfg->line);
+                              "Invalid name for outgoing connection in %s line %d",
+                              cfg->file, cfg->line);
                        free(name);
                        continue;
                }
@@ -861,6 +937,7 @@ void try_outgoing_connections(void) {
        /* Delete outgoing_ts for which there is no ConnectTo. */
 
        for list_each(outgoing_t, outgoing, outgoing_list)
-               if(outgoing->timeout == -1)
+               if(outgoing->timeout == -1) {
                        list_delete_node(outgoing_list, node);
+               }
 }