Fix all warnings when compiling with -Wall -W -pedantic.
[tinc] / src / protocol_subnet.c
index c6c0187..2c4d08f 100644 (file)
 bool send_add_subnet(connection_t *c, const subnet_t *subnet) {
        char netstr[MAXNETSTR];
 
-       cp();
-
-       if(!net2str(netstr, sizeof netstr, subnet))
+       if(!net2str(netstr, sizeof(netstr), subnet)) {
                return false;
+       }
 
        return send_request(c, "%d %x %s %s", ADD_SUBNET, rand(), subnet->owner->name, netstr);
 }
@@ -47,34 +46,33 @@ bool add_subnet_h(connection_t *c) {
        char subnetstr[MAX_STRING_SIZE];
        char name[MAX_STRING_SIZE];
        node_t *owner;
-       subnet_t s = {0}, *new;
-
-       cp();
+       subnet_t s = {0}, *new, *old;
 
        if(sscanf(c->buffer, "%*d %*x " MAX_STRING " " MAX_STRING, name, subnetstr) != 2) {
-               logger(LOG_ERR, _("Got bad %s from %s (%s)"), "ADD_SUBNET", c->name,
-                          c->hostname);
+               logger(LOG_ERR, "Got bad %s from %s (%s)", "ADD_SUBNET", c->name,
+                      c->hostname);
                return false;
        }
 
        /* Check if owner name is valid */
 
        if(!check_id(name)) {
-               logger(LOG_ERR, _("Got bad %s from %s (%s): %s"), "ADD_SUBNET", c->name,
-                          c->hostname, _("invalid name"));
+               logger(LOG_ERR, "Got bad %s from %s (%s): %s", "ADD_SUBNET", c->name,
+                      c->hostname, "invalid name");
                return false;
        }
 
        /* Check if subnet string is valid */
 
        if(!str2net(&s, subnetstr)) {
-               logger(LOG_ERR, _("Got bad %s from %s (%s): %s"), "ADD_SUBNET", c->name,
-                          c->hostname, _("invalid subnet string"));
+               logger(LOG_ERR, "Got bad %s from %s (%s): %s", "ADD_SUBNET", c->name,
+                      c->hostname, "invalid subnet string");
                return false;
        }
 
-       if(seen_request(c->buffer))
+       if(seen_request(c->buffer)) {
                return true;
+       }
 
        /* Check if the owner of the new subnet is in the connection list */
 
@@ -82,8 +80,8 @@ bool add_subnet_h(connection_t *c) {
 
        if(tunnelserver && owner != myself && owner != c->node) {
                /* in case of tunnelserver, ignore indirect subnet registrations */
-               ifdebug(PROTOCOL) logger(LOG_WARNING, _("Ignoring indirect %s from %s (%s) for %s"),
-                                  "ADD_SUBNET", c->name, c->hostname, subnetstr);
+               ifdebug(PROTOCOL) logger(LOG_WARNING, "Ignoring indirect %s from %s (%s) for %s",
+                                        "ADD_SUBNET", c->name, c->hostname, subnetstr);
                return true;
        }
 
@@ -95,42 +93,35 @@ bool add_subnet_h(connection_t *c) {
 
        /* Check if we already know this subnet */
 
-       if(lookup_subnet(owner, &s))
+       if(lookup_subnet(owner, &s)) {
                return true;
+       }
 
        /* If we don't know this subnet, but we are the owner, retaliate with a DEL_SUBNET */
 
        if(owner == myself) {
-               ifdebug(PROTOCOL) logger(LOG_WARNING, _("Got %s from %s (%s) for ourself"),
-                                  "ADD_SUBNET", c->name, c->hostname);
+               ifdebug(PROTOCOL) logger(LOG_WARNING, "Got %s from %s (%s) for ourself",
+                                        "ADD_SUBNET", c->name, c->hostname);
                s.owner = myself;
                send_del_subnet(c, &s);
                return true;
        }
 
-       /* In tunnel server mode, check if the subnet matches one in the config file of this node */
+       /* In tunnel server mode, we should already know all allowed subnets */
 
        if(tunnelserver) {
-               config_t *cfg;
-               subnet_t *allowed;
-
-               for(cfg = lookup_config(c->config_tree, "Subnet"); cfg; cfg = lookup_config_next(c->config_tree, cfg)) {
-                       if(!get_config_subnet(cfg, &allowed))
-                               return false;
-
-                       if(!subnet_compare(&s, allowed))
-                               break;
-
-                       free_subnet(allowed);
-               }
+               logger(LOG_WARNING, "Ignoring unauthorized %s from %s (%s): %s",
+                      "ADD_SUBNET", c->name, c->hostname, subnetstr);
+               return true;
+       }
 
-               if(!cfg) {
-                       logger(LOG_WARNING, _("Unauthorized %s from %s (%s) for %s"),
-                               "ADD_SUBNET", c->name, c->hostname, subnetstr);
-                       return false;
-               }
+       /* Ignore if strictsubnets is true, but forward it to others */
 
-               free_subnet(allowed);
+       if(strictsubnets) {
+               logger(LOG_WARNING, "Ignoring unauthorized %s from %s (%s): %s",
+                      "ADD_SUBNET", c->name, c->hostname, subnetstr);
+               forward_request(c);
+               return true;
        }
 
        /* If everything is correct, add the subnet to the list of the owner */
@@ -138,13 +129,19 @@ bool add_subnet_h(connection_t *c) {
        *(new = new_subnet()) = s;
        subnet_add(owner, new);
 
-       if(owner->status.reachable)
+       if(owner->status.reachable) {
                subnet_update(owner, new, true);
+       }
 
        /* Tell the rest */
 
-       if(!tunnelserver)
-               forward_request(c);
+       forward_request(c);
+
+       /* Fast handoff of roaming MAC addresses */
+
+       if(s.type == SUBNET_MAC && owner != myself && (old = lookup_subnet(myself, &s)) && old->expires) {
+               old->expires = now;
+       }
 
        return true;
 }
@@ -152,10 +149,9 @@ bool add_subnet_h(connection_t *c) {
 bool send_del_subnet(connection_t *c, const subnet_t *s) {
        char netstr[MAXNETSTR];
 
-       cp();
-
-       if(!net2str(netstr, sizeof netstr, s))
+       if(!net2str(netstr, sizeof(netstr), s)) {
                return false;
+       }
 
        return send_request(c, "%d %x %s %s", DEL_SUBNET, rand(), s->owner->name, netstr);
 }
@@ -166,32 +162,31 @@ bool del_subnet_h(connection_t *c) {
        node_t *owner;
        subnet_t s = {0}, *find;
 
-       cp();
-
        if(sscanf(c->buffer, "%*d %*x " MAX_STRING " " MAX_STRING, name, subnetstr) != 2) {
-               logger(LOG_ERR, _("Got bad %s from %s (%s)"), "DEL_SUBNET", c->name,
-                          c->hostname);
+               logger(LOG_ERR, "Got bad %s from %s (%s)", "DEL_SUBNET", c->name,
+                      c->hostname);
                return false;
        }
 
        /* Check if owner name is valid */
 
        if(!check_id(name)) {
-               logger(LOG_ERR, _("Got bad %s from %s (%s): %s"), "DEL_SUBNET", c->name,
-                          c->hostname, _("invalid name"));
+               logger(LOG_ERR, "Got bad %s from %s (%s): %s", "DEL_SUBNET", c->name,
+                      c->hostname, "invalid name");
                return false;
        }
 
        /* Check if subnet string is valid */
 
        if(!str2net(&s, subnetstr)) {
-               logger(LOG_ERR, _("Got bad %s from %s (%s): %s"), "DEL_SUBNET", c->name,
-                          c->hostname, _("invalid subnet string"));
+               logger(LOG_ERR, "Got bad %s from %s (%s): %s", "DEL_SUBNET", c->name,
+                      c->hostname, "invalid subnet string");
                return false;
        }
 
-       if(seen_request(c->buffer))
+       if(seen_request(c->buffer)) {
                return true;
+       }
 
        /* Check if the owner of the subnet being deleted is in the connection list */
 
@@ -199,14 +194,14 @@ bool del_subnet_h(connection_t *c) {
 
        if(tunnelserver && owner != myself && owner != c->node) {
                /* in case of tunnelserver, ignore indirect subnet deletion */
-               ifdebug(PROTOCOL) logger(LOG_WARNING, _("Ignoring indirect %s from %s (%s) for %s"),
-                                  "DEL_SUBNET", c->name, c->hostname, subnetstr);
+               ifdebug(PROTOCOL) logger(LOG_WARNING, "Ignoring indirect %s from %s (%s) for %s",
+                                        "DEL_SUBNET", c->name, c->hostname, subnetstr);
                return true;
        }
 
        if(!owner) {
-               ifdebug(PROTOCOL) logger(LOG_WARNING, _("Got %s from %s (%s) for %s which is not in our node tree"),
-                                  "DEL_SUBNET", c->name, c->hostname, name);
+               ifdebug(PROTOCOL) logger(LOG_WARNING, "Got %s from %s (%s) for %s which is not in our node tree",
+                                        "DEL_SUBNET", c->name, c->hostname, name);
                return true;
        }
 
@@ -217,29 +212,42 @@ bool del_subnet_h(connection_t *c) {
        find = lookup_subnet(owner, &s);
 
        if(!find) {
-               ifdebug(PROTOCOL) logger(LOG_WARNING, _("Got %s from %s (%s) for %s which does not appear in his subnet tree"),
-                                  "DEL_SUBNET", c->name, c->hostname, name);
+               ifdebug(PROTOCOL) logger(LOG_WARNING, "Got %s from %s (%s) for %s which does not appear in his subnet tree",
+                                        "DEL_SUBNET", c->name, c->hostname, name);
+
+               if(strictsubnets) {
+                       forward_request(c);
+               }
+
                return true;
        }
 
        /* If we are the owner of this subnet, retaliate with an ADD_SUBNET */
 
        if(owner == myself) {
-               ifdebug(PROTOCOL) logger(LOG_WARNING, _("Got %s from %s (%s) for ourself"),
-                                  "DEL_SUBNET", c->name, c->hostname);
+               ifdebug(PROTOCOL) logger(LOG_WARNING, "Got %s from %s (%s) for ourself",
+                                        "DEL_SUBNET", c->name, c->hostname);
                send_add_subnet(c, find);
                return true;
        }
 
+       if(tunnelserver) {
+               return true;
+       }
+
        /* Tell the rest */
 
-       if(!tunnelserver)
-               forward_request(c);
+       forward_request(c);
+
+       if(strictsubnets) {
+               return true;
+       }
 
        /* Finally, delete it. */
 
-       if(owner->status.reachable)
+       if(owner->status.reachable) {
                subnet_update(owner, find, false);
+       }
 
        subnet_del(owner, find);