From: Guus Sliepen Date: Fri, 20 Jul 2001 13:54:19 +0000 (+0000) Subject: Remove all unnecessary status.meta and status.active checks. X-Git-Tag: release-1.0pre5~94 X-Git-Url: https://www.tinc-vpn.org/git/browse?p=tinc;a=commitdiff_plain;h=37ed4265fa73d4c06c74362514d78c92029b2f05 Remove all unnecessary status.meta and status.active checks. --- diff --git a/src/meta.c b/src/meta.c index 4f51e2a5..e4ebb426 100644 --- a/src/meta.c +++ b/src/meta.c @@ -17,7 +17,7 @@ along with this program; if not, write to the Free Software Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. - $Id: meta.c,v 1.1.2.19 2001/07/04 08:41:36 guus Exp $ + $Id: meta.c,v 1.1.2.20 2001/07/20 13:54:19 guus Exp $ */ #include "config.h" @@ -75,7 +75,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p != cl && p->status.meta && p->status.active) + if(p != cl && p->status.active) send_meta(p, buffer, length); } cp diff --git a/src/net.c b/src/net.c index 9048d507..dc2d65d0 100644 --- a/src/net.c +++ b/src/net.c @@ -17,7 +17,7 @@ along with this program; if not, write to the Free Software Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. - $Id: net.c,v 1.35.4.121 2001/07/19 12:29:40 guus Exp $ + $Id: net.c,v 1.35.4.122 2001/07/20 13:54:19 guus Exp $ */ #include "config.h" @@ -253,7 +253,7 @@ cp send_udppacket(cl, packet); } -/* Broadcast a packet to all active connections */ +/* Broadcast a packet to all active direct connections */ void broadcast_packet(connection_t *from, vpn_packet_t *packet) { @@ -267,7 +267,7 @@ cp for(node = connection_tree->head; node; node = node->next) { cl = (connection_t *)node->data; - if(cl->status.meta && cl != from) + if(cl->status.active && cl != from) send_packet(cl, packet); } cp @@ -1083,8 +1083,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p->status.meta) - FD_SET(p->meta_socket, fs); + FD_SET(p->meta_socket, fs); } FD_SET(myself->meta_socket, fs); @@ -1163,11 +1162,10 @@ cp if(cl->status.meta) { - /* Find all connections that were lost because they were behind cl (the connection that was dropped). */ - for(node = connection_tree->head; node; node = node->next) + for(node = active_tree->head; node; node = node->next) { p = (connection_t *)node->data; if(p->nexthop == cl && p != cl) @@ -1180,7 +1178,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p->status.meta && p->status.active && p != cl) + if(p->status.active && p != cl) send_del_host(p, cl); /* Sounds like recursion, but p does not have a meta connection :) */ } } @@ -1229,7 +1227,7 @@ cp for(node = connection_tree->head; node; node = node->next) { cl = (connection_t *)node->data; - if(cl->status.active && cl->status.meta) + if(cl->status.active) { if(cl->last_ping_time + timeout < now) { @@ -1301,13 +1299,12 @@ cp if(p->status.remove) return; - if(p->status.meta) - if(FD_ISSET(p->meta_socket, f)) - if(receive_meta(p) < 0) - { - terminate_connection(p); - return; - } + if(FD_ISSET(p->meta_socket, f)) + if(receive_meta(p) < 0) + { + terminate_connection(p); + return; + } } if(FD_ISSET(myself->meta_socket, f)) diff --git a/src/protocol.c b/src/protocol.c index f0bf1802..29282a1c 100644 --- a/src/protocol.c +++ b/src/protocol.c @@ -17,7 +17,7 @@ along with this program; if not, write to the Free Software Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. - $Id: protocol.c,v 1.28.4.100 2001/07/19 12:29:40 guus Exp $ + $Id: protocol.c,v 1.28.4.101 2001/07/20 13:54:19 guus Exp $ */ #include "config.h" @@ -307,7 +307,7 @@ cp { p = (connection_t *)node->data; - if(p != cl && p->status.active) + if(p != cl) { /* Notify others of this connection */ @@ -690,7 +690,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p->status.meta && p->status.active && p!= cl) + if(p->status.active && p!= cl) send_add_subnet(p, subnet); } cp @@ -774,7 +774,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p->status.meta && p->status.active && p!= cl) + if(p->status.active && p!= cl) send_del_subnet(p, subnet); } cp @@ -863,7 +863,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p->status.meta && p->status.active && p!=cl) + if(p->status.active && p!=cl) send_add_host(p, new); } @@ -948,7 +948,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p->status.meta && p->status.active && p!=cl) + if(p->status.active && p!=cl) send_del_host(p, old); } cp @@ -1088,7 +1088,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p != cl && p->status.meta && p->status.active) + if(p != cl && p->status.active) if(!(p->options & OPTION_INDIRECT) || from == myself) send_request(p, "%d %s", KEY_CHANGED, from->name); } diff --git a/src/route.c b/src/route.c index 9a555de4..5bf0b55b 100644 --- a/src/route.c +++ b/src/route.c @@ -17,7 +17,7 @@ along with this program; if not, write to the Free Software Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. - $Id: route.c,v 1.1.2.15 2001/07/01 09:21:14 guus Exp $ + $Id: route.c,v 1.1.2.16 2001/07/20 13:54:19 guus Exp $ */ #include "config.h" @@ -78,7 +78,7 @@ cp for(node = connection_tree->head; node; node = node->next) { p = (connection_t *)node->data; - if(p->status.meta && p->status.active && p!= myself) + if(p->status.active && p!= myself) send_add_subnet(p, subnet); } } diff --git a/src/tincd.c b/src/tincd.c index f9cd82ff..75788d6f 100644 --- a/src/tincd.c +++ b/src/tincd.c @@ -17,7 +17,7 @@ along with this program; if not, write to the Free Software Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. - $Id: tincd.c,v 1.10.4.49 2001/07/04 08:43:32 guus Exp $ + $Id: tincd.c,v 1.10.4.50 2001/07/20 13:54:19 guus Exp $ */ #include "config.h" @@ -212,7 +212,7 @@ int keygen(int bits) if(!rsa_key) { - fprintf(stderr, _("Error during key generation!")); + fprintf(stderr, _("Error during key generation!\n")); return -1; } else