We can safely delete a connection_t in terminate_connection() now.
authorGuus Sliepen <guus@tinc-vpn.org>
Sat, 19 May 2007 12:07:30 +0000 (12:07 +0000)
committerGuus Sliepen <guus@tinc-vpn.org>
Sat, 19 May 2007 12:07:30 +0000 (12:07 +0000)
src/connection.h
src/net.c
src/net_socket.c
src/process.c
src/protocol_misc.c
src/tincd.c

index 9906f0d7a70b03ea50227c3d5f37d2c1fb388ceb..6537e519a0b511f1d5017c93df81abe292639aea 100644 (file)
@@ -40,7 +40,7 @@ typedef union connection_status_t {
                int active:1;                           /* 1 if active.. */
                int connecting:1;                       /* 1 if we are waiting for a non-blocking connect() to finish */
                int termreq:1;                          /* the termination of this connection was requested */
-               int remove:1;                           /* Set to 1 if you want this connection removed */
+               int remove_unused:1;                            /* Set to 1 if you want this connection removed */
                int timeout:1;                          /* 1 if gotten timeout */
                int encryptout:1;                       /* 1 if we can encrypt outgoing traffic */
                int decryptin:1;                        /* 1 if we have to decrypt incoming traffic */
index 469f10fce94ae69827d640ec23365268a152e882..d78770e12c7da38c38339eba8d88d92a714bed96 100644 (file)
--- a/src/net.c
+++ b/src/net.c
@@ -100,31 +100,6 @@ static void purge(void) {
        }
 }
 
-/*
-  put all file descriptors into events
-  While we're at it, purge stuf that needs to be removed.
-*/
-static int build_fdset(void) {
-       splay_node_t *node, *next;
-       connection_t *c;
-       int i, max = 0;
-
-       cp();
-
-       for(node = connection_tree->head; node; node = next) {
-               next = node->next;
-               c = node->data;
-
-               if(c->status.remove) {
-                       connection_del(c);
-                       if(!connection_tree->head)
-                               purge();
-               }
-       }
-
-       return 0;
-}
-
 /*
   Terminate a connection:
   - Close the socket
@@ -135,13 +110,9 @@ static int build_fdset(void) {
 void terminate_connection(connection_t *c, bool report) {
        cp();
 
-       if(c->status.remove)
-               return;
-
        ifdebug(CONNECTIONS) logger(LOG_NOTICE, _("Closing connection with %s (%s)"),
                           c->name, c->hostname);
 
-       c->status.remove = true;
        c->status.active = false;
 
        if(c->node)
@@ -175,18 +146,20 @@ void terminate_connection(connection_t *c, bool report) {
                }
        }
 
+       free(c->outbuf);
+       c->outbuf = NULL;
+       c->outbuflen = 0;
+       c->outbufsize = 0;
+       c->outbufstart = 0;
+
        /* Check if this was our outgoing connection */
 
        if(c->outgoing) {
                retry_outgoing(c->outgoing);
                c->outgoing = NULL;
+       } else {
+               connection_del(c);
        }
-
-       free(c->outbuf);
-       c->outbuf = NULL;
-       c->outbuflen = 0;
-       c->outbufsize = 0;
-       c->outbufstart = 0;
 }
 
 /*
@@ -215,16 +188,11 @@ static void timeout_handler(int fd, short events, void *event) {
                                                           c->name, c->hostname, now - c->last_ping_time);
                                        c->status.timeout = true;
                                        terminate_connection(c, true);
+                                       continue;
                                } else if(c->last_ping_time + pinginterval < now) {
                                        send_ping(c);
                                }
                        } else {
-                               if(c->status.remove) {
-                                       logger(LOG_WARNING, _("Old connection_t for %s (%s) status %04x still lingering, deleting..."),
-                                                  c->name, c->hostname, c->status.value);
-                                       connection_del(c);
-                                       continue;
-                               }
                                ifdebug(CONNECTIONS) logger(LOG_WARNING, _("Timeout from %s (%s) during authentication"),
                                                   c->name, c->hostname);
                                if(c->status.connecting) {
@@ -233,6 +201,7 @@ static void timeout_handler(int fd, short events, void *event) {
                                        do_outgoing_connection(c);
                                } else {
                                        terminate_connection(c, false);
+                                       continue;
                                }
                        }
                }
@@ -246,9 +215,6 @@ void handle_meta_connection_data(int fd, short events, void *data) {
        int result;
        socklen_t len = sizeof(result);
 
-       if (c->status.remove)
-               return;
-
        if(c->status.connecting) {
                getsockopt(c->socket, SOL_SOCKET, SO_ERROR, &result, &len);
 
@@ -383,8 +349,6 @@ static void sigalrm_handler(int signal, short events, void *data) {
   this is where it all happens...
 */
 int main_loop(void) {
-       struct timeval tv;
-       int r;
        struct event timeout_event;
        struct event sighup_event;
        struct event sigint_event;
index ec774f6a73e7bc6b5e04ecd4aec91d9d835025d8..3fea33e18ed2896bd81ec00bc8f3a7ba26596cc9 100644 (file)
@@ -274,7 +274,6 @@ begin:
                if(!c->outgoing->cfg) {
                        ifdebug(CONNECTIONS) logger(LOG_ERR, _("Could not set up a meta connection to %s"),
                                           c->name);
-                       c->status.remove = true;
                        retry_outgoing(c->outgoing);
                        return;
                }
index dcc6cb334a5b764fadb4fc17e97cb45a74c51fe7..1b5f00db096dde052b1e19b8f8dab6cbaa59b6b4 100644 (file)
@@ -45,8 +45,6 @@ extern bool use_logfile;
 
 sigset_t emptysigset;
 
-static int saved_debug_level = -1;
-
 static void memory_full(int size) {
        logger(LOG_ERR, _("Memory exhausted (couldn't allocate %d bytes), exitting."), size);
        cp_trace();
index ca1dc315993884a0aa30a4138c2ea805c961c288..1e0bc50d8f9749d47cbd0ab7f32cedbd38d09c7f 100644 (file)
@@ -90,9 +90,7 @@ bool error_h(connection_t *c)
        ifdebug(ERROR) logger(LOG_NOTICE, _("Error message from %s (%s): %d: %s"),
                           c->name, c->hostname, err, errorstring);
 
-       terminate_connection(c, c->status.active);
-
-       return true;
+       return false;
 }
 
 bool send_termreq(connection_t *c)
@@ -106,9 +104,7 @@ bool termreq_h(connection_t *c)
 {
        cp();
 
-       terminate_connection(c, c->status.active);
-
-       return true;
+       return false;
 }
 
 bool send_ping(connection_t *c)
index bde1b9bc538e4b05a6a922dbb2bb1ed8b9926ce8..9730e21112ba3804e9794cf61fa012faefdd050a 100644 (file)
@@ -466,7 +466,7 @@ int main(int argc, char **argv)
        if(!read_server_config())
                return 1;
 
-       if(event_init() < 0) {
+       if(!event_init()) {
                logger(LOG_ERR, _("Error initializing libevent!"));
                return 1;
        }