]> git.meshlink.io Git - meshlink/commitdiff
Merge branch 'channels'
authorGuus Sliepen <guus@meshlink.io>
Fri, 22 Aug 2014 08:58:47 +0000 (10:58 +0200)
committerGuus Sliepen <guus@meshlink.io>
Fri, 22 Aug 2014 09:00:15 +0000 (11:00 +0200)
Conflicts:
.gitmodules
src/node.h

1  2 
.gitmodules
src/Makefile.am
src/meshlink.c
src/meshlink_internal.h
src/node.h

diff --combined .gitmodules
index 0e6f5440dc21eb7dad698f958e642815ae489de5,add1976b253c2319bc2b7113934679b1b4fb05d4..e938b09181fe4913bb679de84d167cd8917664c7
@@@ -1,4 -1,3 +1,7 @@@
 +[submodule "avahi"]
 +      path = avahi
 +      url = git@chicago.everbase.net:meshlink/avahi-noptr.git
 +      branch = noptr
+ [submodule "src/utcp"]
+       path = src/utcp
+       url = git://meshlink.io/utcp
diff --combined src/Makefile.am
index cae2ab95458666bd89d738bae7eab34890788914,c1d887e9759d2e0687bcda210aafd92c01a66bcd..a2a9305ac54e32f219be1439581d728b5fb66f13
@@@ -28,6 -28,9 +28,9 @@@ chacha_poly1305_SOURCES = 
        chacha-poly1305/chacha-poly1305.c chacha-poly1305/chacha-poly1305.h \
        chacha-poly1305/poly1305.c chacha-poly1305/poly1305.h
  
+ utcp_SOURCES = \
+       utcp/utcp.c utcp/utcp.h
  sptps_test_SOURCES = \
        crypto.c crypto.h \
        logger.c logger.h \
@@@ -56,7 -59,7 +59,7 @@@ sptps_speed_SOURCES = 
  
  lib_LTLIBRARIES = libmeshlink.la
  
 -libmeshlink_la_LDFLAGS = -export-symbols-regex '^meshlink_'
 +libmeshlink_la_LDFLAGS = -export-symbols-regex '^(meshlink_|devtool_)'
  
  libmeshlink_la_SOURCES = \
        meshlink.c meshlink.h \
@@@ -65,7 -68,6 +68,7 @@@
        conf.c conf.h \
        connection.c connection.h \
        crypto.c crypto.h \
 +      discovery.c discovery.h \
        dropin.c dropin.h \
        ecdh.h \
        ecdsa.h \
        system.h \
        utils.c utils.h \
        xalloc.h \
 +      devtools.c devtools.h \
        $(ed25519_SOURCES) \
-       $(chacha_poly1305_SOURCES)
+       $(chacha_poly1305_SOURCES) \
+       $(utcp_SOURCES)
  
 -libmeshlink_la_CFLAGS = -fPIC
 +libmeshlink_la_CFLAGS = -fPIC -I../avahi/
  
 -libmeshlink_la_LIBADD = -lpthread
 +libmeshlink_la_LIBADD = -lpthread -luuid ../avahi/avahi-core/.libs/libavahi-core.a ../avahi/avahi-common/.libs/libavahi-common.a
  
  libmeshlink_la_SOURCES += \
        ed25519/ecdh.c \
diff --combined src/meshlink.c
index ddaa740b88a4b7fd501b02e2f95e7e105260e403,f44dba47b7a7dec9ec2ed7ec394c24e934b01ae3..ef02580ea93d641059eb203c7e3a8a17b14c1ab9
@@@ -21,8 -21,6 +21,8 @@@
  #define VAR_MULTIPLE 4  /* Multiple statements allowed */
  #define VAR_OBSOLETE 8  /* Should not be used anymore */
  #define VAR_SAFE 16     /* Variable is safe when accepting invitations */
 +#define MAX_ADDRESS_LENGTH 45 /* Max length of an (IPv6) address */
 +#define MAX_PORT_LENGTH 5 /* 0-65535 */
  typedef struct {
        const char *name;
        int type;
@@@ -33,7 -31,6 +33,7 @@@
  
  #include "crypto.h"
  #include "ecdsagen.h"
 +#include "logger.h"
  #include "meshlink_internal.h"
  #include "netutl.h"
  #include "node.h"
  #include "utils.h"
  #include "xalloc.h"
  #include "ed25519/sha512.h"
 +#include "discovery.h"
  
  #ifndef MSG_NOSIGNAL
  #define MSG_NOSIGNAL 0
  #endif
  
 +__thread meshlink_errno_t meshlink_errno;
 +meshlink_log_cb_t global_log_cb;
 +meshlink_log_level_t global_log_level;
 +
  //TODO: this can go away completely
  const var_t variables[] = {
        /* Server configuration */
  static bool fcopy(FILE *out, const char *filename) {
        FILE *in = fopen(filename, "r");
        if(!in) {
 -              fprintf(stderr, "Could not open %s: %s\n", filename, strerror(errno));
 +              logger(NULL, MESHLINK_ERROR, "Could not open %s: %s\n", filename, strerror(errno));
                return false;
        }
  
@@@ -197,10 -189,10 +197,10 @@@ static char *get_my_hostname(meshlink_h
                goto done;
  
        // If that doesn't work, guess externally visible hostname
 -      fprintf(stderr, "Trying to discover externally visible hostname...\n");
 +      logger(mesh, MESHLINK_DEBUG, "Trying to discover externally visible hostname...\n");
        struct addrinfo *ai = str2addrinfo("meshlink.io", "80", SOCK_STREAM);
        struct addrinfo *aip = ai;
 -      static const char request[] = "GET http://meshlink.io/host.cgi HTTP/1.0\r\n\r\n";
 +      static const char request[] = "GET http://www.meshlink.io/host.cgi HTTP/1.0\r\n\r\n";
  
        while(aip) {
                int s = socket(aip->ai_family, aip->ai_socktype, aip->ai_protocol);
                fprintf(f, "\nAddress = %s\n", hostname);
                fclose(f);
        } else {
 -              fprintf(stderr, "Could not append Address to %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not append Address to %s: %s\n", filename, strerror(errno));
        }
  
  done:
@@@ -286,7 -278,7 +286,7 @@@ static char *get_line(const char **data
        const char *end = strchr(*data, '\n');
        size_t len = end ? end - *data : strlen(*data);
        if(len >= sizeof line) {
 -              fprintf(stderr, "Maximum line length exceeded!\n");
 +              logger(NULL, MESHLINK_ERROR, "Maximum line length exceeded!\n");
                return NULL;
        }
        if(len && !isprint(**data))
@@@ -335,54 -327,55 +335,54 @@@ static bool try_bind(int port) 
  
        while(ai) {
                int fd = socket(ai->ai_family, SOCK_STREAM, IPPROTO_TCP);
 -              if(!fd)
 +              if(!fd) {
 +                      freeaddrinfo(ai);
                        return false;
 +              }
                int result = bind(fd, ai->ai_addr, ai->ai_addrlen);
                closesocket(fd);
 -              if(result)
 +              if(result) {
 +                      freeaddrinfo(ai);
                        return false;
 +              }
                ai = ai->ai_next;
        }
  
 +      freeaddrinfo(ai);
        return true;
  }
  
  static int check_port(meshlink_handle_t *mesh) {
 -      if(try_bind(655))
 -              return 655;
 -
 -      fprintf(stderr, "Warning: could not bind to port 655.\n");
 -
 -      for(int i = 0; i < 100; i++) {
 +      for(int i = 0; i < 1000; i++) {
                int port = 0x1000 + (rand() & 0x7fff);
                if(try_bind(port)) {
                        char filename[PATH_MAX];
                        snprintf(filename, sizeof filename, "%s" SLASH "hosts" SLASH "%s", mesh->confbase, mesh->name);
                        FILE *f = fopen(filename, "a");
                        if(!f) {
 -                              fprintf(stderr, "Please change MeshLink's Port manually.\n");
 +                              logger(mesh, MESHLINK_DEBUG, "Please change MeshLink's Port manually.\n");
                                return 0;
                        }
  
                        fprintf(f, "Port = %d\n", port);
                        fclose(f);
 -                      fprintf(stderr, "MeshLink will instead listen on port %d.\n", port);
                        return port;
                }
        }
  
 -      fprintf(stderr, "Please change MeshLink's Port manually.\n");
 +      logger(mesh, MESHLINK_DEBUG, "Please change MeshLink's Port manually.\n");
        return 0;
  }
  
  static bool finalize_join(meshlink_handle_t *mesh) {
        char *name = xstrdup(get_value(mesh->data, "Name"));
        if(!name) {
 -              fprintf(stderr, "No Name found in invitation!\n");
 +              logger(mesh, MESHLINK_DEBUG, "No Name found in invitation!\n");
                return false;
        }
  
        if(!check_id(name)) {
 -              fprintf(stderr, "Invalid Name found in invitation: %s!\n", name);
 +              logger(mesh, MESHLINK_DEBUG, "Invalid Name found in invitation: %s!\n", name);
                return false;
        }
  
  
        FILE *f = fopen(filename, "w");
        if(!f) {
 -              fprintf(stderr, "Could not create file %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create file %s: %s\n", filename, strerror(errno));
                return false;
        }
  
        snprintf(filename, sizeof filename, "%s" SLASH "hosts" SLASH "%s", mesh->confbase, name);
        FILE *fh = fopen(filename, "w");
        if(!fh) {
 -              fprintf(stderr, "Could not create file %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create file %s: %s\n", filename, strerror(errno));
                fclose(f);
                return false;
        }
  
                // Ignore unknown and unsafe variables
                if(!found) {
 -                      fprintf(stderr, "Ignoring unknown variable '%s' in invitation.\n", l);
 +                      logger(mesh, MESHLINK_DEBUG, "Ignoring unknown variable '%s' in invitation.\n", l);
                        continue;
                } else if(!(variables[i].type & VAR_SAFE)) {
 -                      fprintf(stderr, "Ignoring unsafe variable '%s' in invitation.\n", l);
 +                      logger(mesh, MESHLINK_DEBUG, "Ignoring unsafe variable '%s' in invitation.\n", l);
                        continue;
                }
  
  
        while(l && !strcasecmp(l, "Name")) {
                if(!check_id(value)) {
 -                      fprintf(stderr, "Invalid Name found in invitation.\n");
 +                      logger(mesh, MESHLINK_DEBUG, "Invalid Name found in invitation.\n");
                        return false;
                }
  
                if(!strcmp(value, name)) {
 -                      fprintf(stderr, "Secondary chunk would overwrite our own host config file.\n");
 +                      logger(mesh, MESHLINK_DEBUG, "Secondary chunk would overwrite our own host config file.\n");
                        return false;
                }
  
                f = fopen(filename, "w");
  
                if(!f) {
 -                      fprintf(stderr, "Could not create file %s: %s\n", filename, strerror(errno));
 +                      logger(mesh, MESHLINK_DEBUG, "Could not create file %s: %s\n", filename, strerror(errno));
                        return false;
                }
  
        }
  
        char *b64key = ecdsa_get_base64_public_key(mesh->self->connection->ecdsa);
 -      if(!b64key)
 +      if(!b64key) {
 +              fclose(fh);
                return false;
 +              }
  
        fprintf(fh, "ECDSAPublicKey = %s\n", b64key);
        fprintf(fh, "Port = %s\n", mesh->myport);
        free(mesh->self->name);
        free(mesh->self->connection->name);
        mesh->self->name = xstrdup(name);
 -      mesh->self->connection->name = xstrdup(name);
 +      mesh->self->connection->name = name;
  
 -      fprintf(stderr, "Configuration stored in: %s\n", mesh->confbase);
 +      logger(mesh, MESHLINK_DEBUG, "Configuration stored in: %s\n", mesh->confbase);
  
        load_all_nodes(mesh);
  
@@@ -554,11 -545,10 +554,11 @@@ static bool invitation_receive(void *ha
                        break;
  
                case 1:
 +                      mesh->thedatalen = 0;
                        return finalize_join(mesh);
  
                case 2:
 -                      fprintf(stderr, "Invitation succesfully accepted.\n");
 +                      logger(mesh, MESHLINK_DEBUG, "Invitation succesfully accepted.\n");
                        shutdown(mesh->sock, SHUT_RDWR);
                        mesh->success = true;
                        break;
@@@ -628,21 -618,12 +628,21 @@@ static bool sendline(int fd, char *form
  
  static const char *errstr[] = {
        [MESHLINK_OK] = "No error",
 +      [MESHLINK_EINVAL] = "Invalid argument",
        [MESHLINK_ENOMEM] = "Out of memory",
        [MESHLINK_ENOENT] = "No such node",
 +      [MESHLINK_EEXIST] = "Node already exists",
 +      [MESHLINK_EINTERNAL] = "Internal error",
 +      [MESHLINK_ERESOLV] = "Could not resolve hostname",
 +      [MESHLINK_ESTORAGE] = "Storage error",
 +      [MESHLINK_ENETWORK] = "Network error",
 +      [MESHLINK_EPEER] = "Error communicating with peer",
  };
  
 -const char *meshlink_strerror(meshlink_errno_t errno) {
 -      return errstr[errno];
 +const char *meshlink_strerror(meshlink_errno_t err) {
 +      if(err < 0 || err >= sizeof errstr / sizeof *errstr)
 +              return "Invalid error code";
 +      return errstr[err];
  }
  
  static bool ecdsa_keygen(meshlink_handle_t *mesh) {
        FILE *f;
        char pubname[PATH_MAX], privname[PATH_MAX];
  
 -      fprintf(stderr, "Generating ECDSA keypair:\n");
 +      logger(mesh, MESHLINK_DEBUG, "Generating ECDSA keypair:\n");
  
        if(!(key = ecdsa_generate())) {
 -              fprintf(stderr, "Error during key generation!\n");
 +              logger(mesh, MESHLINK_DEBUG, "Error during key generation!\n");
 +              meshlink_errno = MESHLINK_EINTERNAL;
                return false;
        } else
 -              fprintf(stderr, "Done.\n");
 +              logger(mesh, MESHLINK_DEBUG, "Done.\n");
  
        snprintf(privname, sizeof privname, "%s" SLASH "ecdsa_key.priv", mesh->confbase);
        f = fopen(privname, "w");
  
 -      if(!f)
 +      if(!f) {
 +              meshlink_errno = MESHLINK_ESTORAGE;
                return false;
 +      }
  
  #ifdef HAVE_FCHMOD
        fchmod(fileno(f), 0600);
  #endif
  
        if(!ecdsa_write_pem_private_key(key, f)) {
 -              fprintf(stderr, "Error writing private key!\n");
 +              logger(mesh, MESHLINK_DEBUG, "Error writing private key!\n");
                ecdsa_free(key);
                fclose(f);
 +              meshlink_errno = MESHLINK_EINTERNAL;
                return false;
        }
  
        fclose(f);
  
 -
        snprintf(pubname, sizeof pubname, "%s" SLASH "hosts" SLASH "%s", mesh->confbase, mesh->name);
        f = fopen(pubname, "a");
  
 -      if(!f)
 +      if(!f) {
 +              meshlink_errno = MESHLINK_ESTORAGE;
                return false;
 +      }
  
        char *pubkey = ecdsa_get_base64_public_key(key);
        fprintf(f, "ECDSAPublicKey = %s\n", pubkey);
  
  static bool meshlink_setup(meshlink_handle_t *mesh) {
        if(mkdir(mesh->confbase, 0777) && errno != EEXIST) {
 -              fprintf(stderr, "Could not create directory %s: %s\n", mesh->confbase, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create directory %s: %s\n", mesh->confbase, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
                return false;
        }
  
        snprintf(filename, sizeof filename, "%s" SLASH "hosts", mesh->confbase);
  
        if(mkdir(filename, 0777) && errno != EEXIST) {
 -              fprintf(stderr, "Could not create directory %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create directory %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
                return false;
        }
  
        snprintf(filename, sizeof filename, "%s" SLASH "meshlink.conf", mesh->confbase);
  
        if(!access(filename, F_OK)) {
 -              fprintf(stderr, "Configuration file %s already exists!\n", filename);
 +              logger(mesh, MESHLINK_DEBUG, "Configuration file %s already exists!\n", filename);
 +              meshlink_errno = MESHLINK_EEXIST;
                return false;
        }
  
        FILE *f = fopen(filename, "w");
        if(!f) {
 -              fprintf(stderr, "Could not create file %s: %s\n", filename, strerror(errno));
 -              return 1;
 +              logger(mesh, MESHLINK_DEBUG, "Could not create file %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              return false;
        }
  
        fprintf(f, "Name = %s\n", mesh->name);
        fclose(f);
  
 -      if(!ecdsa_keygen(mesh))
 +      if(!ecdsa_keygen(mesh)) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
                return false;
 +      }
  
        check_port(mesh);
  
        return true;
  }
  
 -meshlink_handle_t *meshlink_open(const char *confbase, const char *name) {
 +meshlink_handle_t *meshlink_open(const char *confbase, const char *name, const char* appname, dev_class_t devclass) {
 +      return meshlink_open_with_size(confbase, name, appname, devclass, sizeof(meshlink_handle_t));
 +}
 +
 +meshlink_handle_t *meshlink_open_with_size(const char *confbase, const char *name, const char* appname, dev_class_t devclass, size_t size) {
 +
        // Validate arguments provided by the application
        bool usingname = false;
 +      
 +      logger(NULL, MESHLINK_DEBUG, "meshlink_open called\n");
  
        if(!confbase || !*confbase) {
 -              fprintf(stderr, "No confbase given!\n");
 +              logger(NULL, MESHLINK_ERROR, "No confbase given!\n");
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +
 +      if(!appname || !*appname) {
 +              logger(NULL, MESHLINK_ERROR, "No appname given!\n");
 +              meshlink_errno = MESHLINK_EINVAL;
                return NULL;
        }
  
        if(!name || !*name) {
 -              fprintf(stderr, "No name given!\n");
 +              logger(NULL, MESHLINK_ERROR, "No name given!\n");
                //return NULL;
        }
        else { //check name only if there is a name != NULL
  
                if(!check_id(name)) {
 -                      fprintf(stderr, "Invalid name given!\n");
 +                      logger(NULL, MESHLINK_ERROR, "Invalid name given!\n");
 +                      meshlink_errno = MESHLINK_EINVAL;
                        return NULL;
                } else { usingname = true;}
        }
  
 -      meshlink_handle_t *mesh = xzalloc(sizeof *mesh);
 +      if(devclass < 0 || devclass > _DEV_CLASS_MAX) {
 +              logger(NULL, MESHLINK_ERROR, "Invalid devclass given!\n");
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +
 +      meshlink_handle_t *mesh = xzalloc(size);
        mesh->confbase = xstrdup(confbase);
 +      mesh->appname = xstrdup(appname);
 +      mesh->devclass = devclass;
        if (usingname) mesh->name = xstrdup(name);
 -      pthread_mutex_init ( &(mesh->outpacketqueue_mutex), NULL);
 -      pthread_mutex_init ( &(mesh->nodes_mutex), NULL);
 +
 +      // initialize mutex
 +      pthread_mutexattr_t attr;
 +      pthread_mutexattr_init(&attr);
 +      pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE);
 +      pthread_mutex_init(&(mesh->mesh_mutex), &attr);
 +      
        mesh->threadstarted = false;
        event_loop_init(&mesh->loop);
        mesh->loop.data = mesh;
  
 -      // TODO: should be set by a function.
 -      mesh->debug_level = 5;
 -
        // Check whether meshlink.conf already exists
  
        char filename[PATH_MAX];
        if(access(filename, R_OK)) {
                if(errno == ENOENT) {
                        // If not, create it
 -                      meshlink_setup(mesh);
 +                      if(!meshlink_setup(mesh)) {
 +                              // meshlink_errno is set by meshlink_setup()
 +                              return NULL;
 +                      }
                } else {
 -                      fprintf(stderr, "Cannot not read from %s: %s\n", filename, strerror(errno));
 -                      return meshlink_close(mesh), NULL;
 +                      logger(NULL, MESHLINK_ERROR, "Cannot not read from %s: %s\n", filename, strerror(errno));
 +                      meshlink_close(mesh);
 +                      meshlink_errno = MESHLINK_ESTORAGE;
 +                      return NULL;
                }
        }
  
  
        init_configuration(&mesh->config);
  
 -      if(!read_server_config(mesh))
 -              return meshlink_close(mesh), NULL;
 +      if(!read_server_config(mesh)) {
 +              meshlink_close(mesh);
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              return NULL;
 +      };
  
  #ifdef HAVE_MINGW
        struct WSAData wsa_state;
        // Setup up everything
        // TODO: we should not open listening sockets yet
  
 -      if(!setup_network(mesh))
 -              return meshlink_close(mesh), NULL;
 +      if(!setup_network(mesh)) {
 +              meshlink_close(mesh);
 +              meshlink_errno = MESHLINK_ENETWORK;
 +              return NULL;
 +      }
  
 +      logger(NULL, MESHLINK_DEBUG, "meshlink_open returning\n");
        return mesh;
  }
  
 -void *meshlink_main_loop(void *arg) {
 +static void *meshlink_main_loop(void *arg) {
        meshlink_handle_t *mesh = arg;
  
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
        try_outgoing_connections(mesh);
  
 +      logger(mesh, MESHLINK_DEBUG, "Starting main_loop...\n");
        main_loop(mesh);
 +      logger(mesh, MESHLINK_DEBUG, "main_loop returned.\n");
  
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return NULL;
  }
  
  bool meshlink_start(meshlink_handle_t *mesh) {
 +      if(!mesh) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      
 +      logger(mesh, MESHLINK_DEBUG, "meshlink_start called\n");
 +
 +      mesh->thedatalen = 0;
 +
        // TODO: open listening sockets first
  
        //Check that a valid name is set
        if(!mesh->name ) {
 -              fprintf(stderr, "No name given!\n");
 +              logger(mesh, MESHLINK_DEBUG, "No name given!\n");
 +              meshlink_errno = MESHLINK_EINVAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        // Start the main thread
  
        if(pthread_create(&mesh->thread, NULL, meshlink_main_loop, mesh) != 0) {
 -              fprintf(stderr, "Could not start thread: %s\n", strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not start thread: %s\n", strerror(errno));
                memset(&mesh->thread, 0, sizeof mesh->thread);
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        mesh->threadstarted=true;
  
 +      discovery_start(mesh);
 +
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return true;
  }
  
  void meshlink_stop(meshlink_handle_t *mesh) {
 -      // Shut down the listening sockets to signal the main thread to shut down
 -
 -      for(int i = 0; i < mesh->listen_sockets; i++) {
 -              shutdown(mesh->listen_socket[i].tcp.fd, SHUT_RDWR);
 -              shutdown(mesh->listen_socket[i].udp.fd, SHUT_RDWR);
 +      if(!mesh) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return;
        }
  
 -      // Wait for the main thread to finish
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      logger(mesh, MESHLINK_DEBUG, "meshlink_stop called\n");
 +
 +      // Stop discovery
 +      discovery_stop(mesh);
  
 +      // Shut down a listening socket to signal the main thread to shut down
 +
 +      listen_socket_t *s = &mesh->listen_socket[0];
 +      shutdown(s->tcp.fd, SHUT_RDWR);
 +
 +      // Wait for the main thread to finish
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        pthread_join(mesh->thread, NULL);
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
 +      mesh->threadstarted = false;
 +
 +      // Fix the socket
 +      
 +      closesocket(s->tcp.fd);
 +      io_del(&mesh->loop, &s->tcp);
 +      s->tcp.fd = setup_listen_socket(&s->sa);
 +      if(s->tcp.fd < 0)
 +              logger(mesh, MESHLINK_ERROR, "Could not repair listenen socket!");
 +      else
 +              io_add(&mesh->loop, &s->tcp, handle_new_meta_connection, s, s->tcp.fd, IO_READ);
 +      
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
  }
  
  void meshlink_close(meshlink_handle_t *mesh) {
 +      if(!mesh || !mesh->confbase) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return;
 +      }
 +
 +      // lock is not released after this
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
        // Close and free all resources used.
  
        close_network_connections(mesh);
  
 -      logger(DEBUG_ALWAYS, LOG_NOTICE, "Terminating");
 +      logger(mesh, MESHLINK_INFO, "Terminating");
  
        exit_configuration(&mesh->config);
        event_loop_exit(&mesh->loop);
  
 -      free(mesh);
 -
  #ifdef HAVE_MINGW
 -      WSACleanup();
 +      if(mesh->confbase)
 +              WSACleanup();
  #endif
 +
 +      ecdsa_free(mesh->invitation_key);
 +
 +      free(mesh->name);
 +      free(mesh->appname);
 +      free(mesh->confbase);
 +      pthread_mutex_destroy(&(mesh->mesh_mutex));
 +
 +      memset(mesh, 0, sizeof *mesh);
 +
 +      free(mesh);
  }
  
  void meshlink_set_receive_cb(meshlink_handle_t *mesh, meshlink_receive_cb_t cb) {
 +      if(!mesh) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
        mesh->receive_cb = cb;
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
  }
  
  void meshlink_set_node_status_cb(meshlink_handle_t *mesh, meshlink_node_status_cb_t cb) {
 +      if(!mesh) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
        mesh->node_status_cb = cb;
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
  }
  
  void meshlink_set_log_cb(meshlink_handle_t *mesh, meshlink_log_level_t level, meshlink_log_cb_t cb) {
 -      mesh->log_cb = cb;
 -      mesh->log_level = level;
 +      if(mesh) {
 +              pthread_mutex_lock(&(mesh->mesh_mutex));
 +              mesh->log_cb = cb;
 +              mesh->log_level = cb ? level : 0;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      } else {
 +              global_log_cb = cb;
 +              global_log_level = cb ? level : 0;
 +      }
  }
  
 -bool meshlink_send(meshlink_handle_t *mesh, meshlink_node_t *destination, const void *data, unsigned int len) {
 +bool meshlink_send(meshlink_handle_t *mesh, meshlink_node_t *destination, const void *data, size_t len) {
 +      if(!mesh || !destination) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
  
 -      /* If there is no outgoing list yet, create one. */
 +      if(!len)
 +              return true;
  
 -      if(!mesh->outpacketqueue)
 -              mesh->outpacketqueue = list_alloc(NULL);
 +      if(!data) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
  
        //add packet to the queue
        outpacketqueue_t *packet_in_queue = xzalloc(sizeof *packet_in_queue);
        packet_in_queue->destination=destination;
        packet_in_queue->data=data;
        packet_in_queue->len=len;
 -      pthread_mutex_lock(&(mesh->outpacketqueue_mutex));
 -      list_insert_head(mesh->outpacketqueue,packet_in_queue);
 -      pthread_mutex_unlock(&(mesh->outpacketqueue_mutex));
 +      if(!meshlink_queue_push(&mesh->outpacketqueue, packet_in_queue)) {
 +              free(packet_in_queue);
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
 +              return false;
 +      }
  
        //notify event loop
        signal_trigger(&(mesh->loop),&(mesh->datafromapp));
 +      
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return true;
  }
  
  void meshlink_send_from_queue(event_loop_t* el,meshlink_handle_t *mesh) {
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      
        vpn_packet_t packet;
        meshlink_packethdr_t *hdr = (meshlink_packethdr_t *)packet.data;
  
 -      outpacketqueue_t* p = list_get_tail(mesh->outpacketqueue);
 -      if (p)
 -      list_delete_tail(mesh->outpacketqueue);
 -      else return ;
 +      outpacketqueue_t* p = meshlink_queue_pop(&mesh->outpacketqueue);
 +      if(!p)
 +      {
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
 +              return;
 +      }
  
        if (sizeof(meshlink_packethdr_t) + p->len > MAXSIZE) {
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                //log something
 -              return ;
 +              return;
        }
  
        packet.probe = false;
          mesh->self->in_packets++;
          mesh->self->in_bytes += packet.len;
          route(mesh, mesh->self, &packet);
 +      
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return ;
  }
  
 +ssize_t meshlink_get_pmtu(meshlink_handle_t *mesh, meshlink_node_t *destination) {
 +      if(!mesh || !destination) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return -1;
 +      }
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
 +      node_t *n = (node_t *)destination;
 +      if(!n->status.reachable) {
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
 +              return 0;
 +      
 +      }
 +      else if(n->mtuprobes > 30 && n->minmtu) {
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
 +              return n->minmtu;
 +      }
 +      else {
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
 +              return MTU;
 +      }
 +}
 +
 +char *meshlink_get_fingerprint(meshlink_handle_t *mesh, meshlink_node_t *node) {
 +      if(!mesh || !node) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
 +      node_t *n = (node_t *)node;
 +
 +      if(!node_read_ecdsa_public_key(mesh, n) || !n->ecdsa) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
 +              return false;
 +      }
 +
 +      char *fingerprint = ecdsa_get_base64_public_key(n->ecdsa);
 +
 +      if(!fingerprint)
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      return fingerprint;
 +}
 +
  meshlink_node_t *meshlink_get_node(meshlink_handle_t *mesh, const char *name) {
 -      return (meshlink_node_t *)lookup_node(mesh, (char *)name); // TODO: make lookup_node() use const
 +      if(!mesh || !name) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +
 +      meshlink_node_t *node = NULL;
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      node = (meshlink_node_t *)lookup_node(mesh, (char *)name); // TODO: make lookup_node() use const
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      return node;
  }
  
 -size_t meshlink_get_all_nodes(meshlink_handle_t *mesh, meshlink_node_t **nodes, size_t nmemb) {
 -      size_t i = 0;
 +meshlink_node_t **meshlink_get_all_nodes(meshlink_handle_t *mesh, meshlink_node_t **nodes, size_t *nmemb) {
 +      if(!mesh || !nmemb || (*nmemb && !nodes)) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +
 +      meshlink_node_t **result;
  
        //lock mesh->nodes
 -      pthread_mutex_lock(&(mesh->nodes_mutex));
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
 +      *nmemb = mesh->nodes->count;
 +      result = realloc(nodes, *nmemb * sizeof *nodes);
  
 -      for splay_each(node_t, n, mesh->nodes) {
 -              if(i < nmemb)
 -                      nodes[i] = (meshlink_node_t *)n;
 -              i++;
 +      if(result) {
 +              meshlink_node_t **p = result;
 +              for splay_each(node_t, n, mesh->nodes)
 +                      *p++ = (meshlink_node_t *)n;
 +      } else {
 +              *nmemb = 0;
 +              free(nodes);
 +              meshlink_errno = MESHLINK_ENOMEM;
        }
  
 -      pthread_mutex_unlock(&(mesh->nodes_mutex));
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
  
 -      return i;
 +      return result;
  }
  
  bool meshlink_sign(meshlink_handle_t *mesh, const void *data, size_t len, void *signature, size_t *siglen) {
 -      if(*siglen < MESHLINK_SIGLEN)
 +      if(!mesh || !data || !len || !signature || !siglen) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
 +
 +      if(*siglen < MESHLINK_SIGLEN) {
 +              meshlink_errno = MESHLINK_EINVAL;
                return false;
 -      if(!ecdsa_sign(mesh->self->connection->ecdsa, data, len, signature))
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
 +      if(!ecdsa_sign(mesh->self->connection->ecdsa, data, len, signature)) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
 +      }
 +
        *siglen = MESHLINK_SIGLEN;
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return true;
  }
  
  bool meshlink_verify(meshlink_handle_t *mesh, meshlink_node_t *source, const void *data, size_t len, const void *signature, size_t siglen) {
 -      if(siglen != MESHLINK_SIGLEN)
 +      if(!mesh || !data || !len || !signature) {
 +              meshlink_errno = MESHLINK_EINVAL;
                return false;
 +      }
 +
 +      if(siglen != MESHLINK_SIGLEN) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
 +      bool rval = false;
 +
        struct node_t *n = (struct node_t *)source;
        node_read_ecdsa_public_key(mesh, n);
 -      if(!n->ecdsa)
 -              return false;
 -      return ecdsa_verify(((struct node_t *)source)->ecdsa, data, len, signature);
 +      if(!n->ecdsa) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              rval = false;
 +      } else {
 +              rval = ecdsa_verify(((struct node_t *)source)->ecdsa, data, len, signature);
 +      }
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      return rval;
  }
  
  static bool refresh_invitation_key(meshlink_handle_t *mesh) {
        char filename[PATH_MAX];
 +      
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
  
        snprintf(filename, sizeof filename, "%s" SLASH "invitations", mesh->confbase);
        if(mkdir(filename, 0700) && errno != EEXIST) {
 -              fprintf(stderr, "Could not create directory %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create directory %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        // Count the number of valid invitations, clean up old ones
        DIR *dir = opendir(filename);
        if(!dir) {
 -              fprintf(stderr, "Could not read directory %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not read directory %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
                        else
                                unlink(invname);
                } else {
 -                      fprintf(stderr, "Could not stat %s: %s\n", invname, strerror(errno));
 +                      logger(mesh, MESHLINK_DEBUG, "Could not stat %s: %s\n", invname, strerror(errno));
                        errno = 0;
                }
        }
  
        if(errno) {
 -              fprintf(stderr, "Error while reading directory %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Error while reading directory %s: %s\n", filename, strerror(errno));
                closedir(dir);
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
                }
        }
  
 -      if(mesh->invitation_key)
 +      if(mesh->invitation_key) {
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return true;
 +      }
  
        // Create a new key if necessary.
        FILE *f = fopen(filename, "r");
        if(!f) {
                if(errno != ENOENT) {
 -                      fprintf(stderr, "Could not read %s: %s\n", filename, strerror(errno));
 +                      logger(mesh, MESHLINK_DEBUG, "Could not read %s: %s\n", filename, strerror(errno));
 +                      meshlink_errno = MESHLINK_ESTORAGE;
 +                      pthread_mutex_unlock(&(mesh->mesh_mutex));
                        return false;
                }
  
                mesh->invitation_key = ecdsa_generate();
                if(!mesh->invitation_key) {
 -                      fprintf(stderr, "Could not generate a new key!\n");
 +                      logger(mesh, MESHLINK_DEBUG, "Could not generate a new key!\n");
 +                      meshlink_errno = MESHLINK_EINTERNAL;
 +                      pthread_mutex_unlock(&(mesh->mesh_mutex));
                        return false;
                }
                f = fopen(filename, "w");
                if(!f) {
 -                      fprintf(stderr, "Could not write %s: %s\n", filename, strerror(errno));
 +                      logger(mesh, MESHLINK_DEBUG, "Could not write %s: %s\n", filename, strerror(errno));
 +                      meshlink_errno = MESHLINK_ESTORAGE;
 +                      pthread_mutex_unlock(&(mesh->mesh_mutex));
                        return false;
                }
                chmod(filename, 0600);
        } else {
                mesh->invitation_key = ecdsa_read_pem_private_key(f);
                fclose(f);
 -              if(!mesh->invitation_key)
 -                      fprintf(stderr, "Could not read private key from %s\n", filename);
 +              if(!mesh->invitation_key) {
 +                      logger(mesh, MESHLINK_DEBUG, "Could not read private key from %s\n", filename);
 +                      meshlink_errno = MESHLINK_ESTORAGE;
 +              }
        }
  
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return mesh->invitation_key;
  }
  
  bool meshlink_add_address(meshlink_handle_t *mesh, const char *address) {
 +      if(!mesh || !address) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
 +      
 +      bool rval = false;
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
        for(const char *p = address; *p; p++) {
                if(isalnum(*p) || *p == '-' || *p == '.' || *p == ':')
                        continue;
 -              fprintf(stderr, "Invalid character in address: %s\n", address);
 +              logger(mesh, MESHLINK_DEBUG, "Invalid character in address: %s\n", address);
 +              meshlink_errno = MESHLINK_EINVAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
 -      return append_config_file(mesh, mesh->self->name, "Address", address);
 +      rval = append_config_file(mesh, mesh->self->name, "Address", address);
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      return rval;
  }
  
  char *meshlink_invite(meshlink_handle_t *mesh, const char *name) {
 +      if(!mesh) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +      
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
        // Check validity of the new node's name
        if(!check_id(name)) {
 -              fprintf(stderr, "Invalid name for node.\n");
 +              logger(mesh, MESHLINK_DEBUG, "Invalid name for node.\n");
 +              meshlink_errno = MESHLINK_EINVAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
  
        char filename[PATH_MAX];
        snprintf(filename, sizeof filename, "%s" SLASH "hosts" SLASH "%s", mesh->confbase, name);
        if(!access(filename, F_OK)) {
 -              fprintf(stderr, "A host config file for %s already exists!\n", name);
 +              logger(mesh, MESHLINK_DEBUG, "A host config file for %s already exists!\n", name);
 +              meshlink_errno = MESHLINK_EEXIST;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
  
        // Ensure no other nodes know about this name
        if(meshlink_get_node(mesh, name)) {
 -              fprintf(stderr, "A node with name %s is already known!\n", name);
 +              logger(mesh, MESHLINK_DEBUG, "A node with name %s is already known!\n", name);
 +              meshlink_errno = MESHLINK_EEXIST;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
  
        // Get the local address
        char *address = get_my_hostname(mesh);
        if(!address) {
 -              fprintf(stderr, "No Address known for ourselves!\n");
 +              logger(mesh, MESHLINK_DEBUG, "No Address known for ourselves!\n");
 +              meshlink_errno = MESHLINK_ERESOLV;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
  
 -      if(!refresh_invitation_key(mesh))
 +      if(!refresh_invitation_key(mesh)) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
 +      }
  
        char hash[64];
  
  
        b64encode_urlsafe(cookie, cookie, 18);
  
 +      free(fingerprint);
 +
        // Create a file containing the details of the invitation.
        snprintf(filename, sizeof filename, "%s" SLASH "invitations" SLASH "%s", mesh->confbase, cookiehash);
        int ifd = open(filename, O_RDWR | O_CREAT | O_EXCL, 0600);
        if(!ifd) {
 -              fprintf(stderr, "Could not create invitation file %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create invitation file %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
        FILE *f = fdopen(ifd, "w");
                }
                fclose(tc);
        } else {
 -              fprintf(stderr, "Could not create %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
  
        // Create an URL from the local address, key hash and cookie
        char *url;
        xasprintf(&url, "%s/%s%s", address, hash, cookie);
 +      free(address);
  
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return url;
  }
  
  bool meshlink_join(meshlink_handle_t *mesh, const char *invitation) {
 +      if(!mesh || !invitation) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
 +      
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
        //TODO: think of a better name for this variable, or of a different way to tokenize the invitation URL.
        char copy[strlen(invitation) + 1];
        strcpy(copy, invitation);
        }
  
        if(!port)
 -              port = "655";
 +              goto invalid;
  
        if(!b64decode(slash, mesh->hash, 18) || !b64decode(slash + 24, mesh->cookie, 18))
                goto invalid;
  
        // Generate a throw-away key for the invitation.
        ecdsa_t *key = ecdsa_generate();
 -      if(!key)
 +      if(!key) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
 +      }
  
        char *b64key = ecdsa_get_base64_public_key(key);
  
  
        // Connect to the meshlink daemon mentioned in the URL.
        struct addrinfo *ai = str2addrinfo(address, port, SOCK_STREAM);
 -      if(!ai)
 +      if(!ai) {
 +              meshlink_errno = MESHLINK_ERESOLV;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
 +      }
  
        mesh->sock = socket(ai->ai_family, ai->ai_socktype, ai->ai_protocol);
        if(mesh->sock <= 0) {
 -              fprintf(stderr, "Could not open socket: %s\n", strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not open socket: %s\n", strerror(errno));
 +              freeaddrinfo(ai);
 +              meshlink_errno = MESHLINK_ENETWORK;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        if(connect(mesh->sock, ai->ai_addr, ai->ai_addrlen)) {
 -              fprintf(stderr, "Could not connect to %s port %s: %s\n", address, port, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not connect to %s port %s: %s\n", address, port, strerror(errno));
                closesocket(mesh->sock);
 +              freeaddrinfo(ai);
 +              meshlink_errno = MESHLINK_ENETWORK;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
 -      fprintf(stderr, "Connected to %s port %s...\n", address, port);
 +      freeaddrinfo(ai);
 +
 +      logger(mesh, MESHLINK_DEBUG, "Connected to %s port %s...\n", address, port);
  
        // Tell him we have an invitation, and give him our throw-away key.
  
        mesh->blen = 0;
  
        if(!sendline(mesh->sock, "0 ?%s %d.%d", b64key, PROT_MAJOR, 1)) {
 -              fprintf(stderr, "Error sending request to %s port %s: %s\n", address, port, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Error sending request to %s port %s: %s\n", address, port, strerror(errno));
                closesocket(mesh->sock);
 +              meshlink_errno = MESHLINK_ENETWORK;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
 +      free(b64key);
 +
        char hisname[4096] = "";
        int code, hismajor, hisminor = 0;
  
        if(!recvline(mesh, sizeof mesh->line) || sscanf(mesh->line, "%d %s %d.%d", &code, hisname, &hismajor, &hisminor) < 3 || code != 0 || hismajor != PROT_MAJOR || !check_id(hisname) || !recvline(mesh, sizeof mesh->line) || !rstrip(mesh->line) || sscanf(mesh->line, "%d ", &code) != 1 || code != ACK || strlen(mesh->line) < 3) {
 -              fprintf(stderr, "Cannot read greeting from peer\n");
 +              logger(mesh, MESHLINK_DEBUG, "Cannot read greeting from peer\n");
                closesocket(mesh->sock);
 +              meshlink_errno = MESHLINK_ENETWORK;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        char *fingerprint = mesh->line + 2;
        char hishash[64];
        if(sha512(fingerprint, strlen(fingerprint), hishash)) {
 -              fprintf(stderr, "Could not create hash\n%s\n", mesh->line + 2);
 +              logger(mesh, MESHLINK_DEBUG, "Could not create hash\n%s\n", mesh->line + 2);
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
        if(memcmp(hishash, mesh->hash, 18)) {
 -              fprintf(stderr, "Peer has an invalid key!\n%s\n", mesh->line + 2);
 +              logger(mesh, MESHLINK_DEBUG, "Peer has an invalid key!\n%s\n", mesh->line + 2);
 +              meshlink_errno = MESHLINK_EPEER;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
  
        }
  
        ecdsa_t *hiskey = ecdsa_set_base64_public_key(fingerprint);
 -      if(!hiskey)
 +      if(!hiskey) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
 +      }
  
        // Start an SPTPS session
 -      if(!sptps_start(&mesh->sptps, mesh, true, false, key, hiskey, "meshlink invitation", 15, invitation_send, invitation_receive))
 +      if(!sptps_start(&mesh->sptps, mesh, true, false, key, hiskey, "meshlink invitation", 15, invitation_send, invitation_receive)) {
 +              meshlink_errno = MESHLINK_EINTERNAL;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
 +      }
  
        // Feed rest of input buffer to SPTPS
 -      if(!sptps_receive_data(&mesh->sptps, mesh->buffer, mesh->blen))
 +      if(!sptps_receive_data(&mesh->sptps, mesh->buffer, mesh->blen)) {
 +              meshlink_errno = MESHLINK_EPEER;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
 +      }
  
        int len;
  
                if(len < 0) {
                        if(errno == EINTR)
                                continue;
 -                      fprintf(stderr, "Error reading data from %s port %s: %s\n", address, port, strerror(errno));
 +                      logger(mesh, MESHLINK_DEBUG, "Error reading data from %s port %s: %s\n", address, port, strerror(errno));
 +                      meshlink_errno = MESHLINK_ENETWORK;
 +                      pthread_mutex_unlock(&(mesh->mesh_mutex));
                        return false;
                }
  
 -              if(!sptps_receive_data(&mesh->sptps, mesh->line, len))
 +              if(!sptps_receive_data(&mesh->sptps, mesh->line, len)) {
 +                      meshlink_errno = MESHLINK_EPEER;
 +                      pthread_mutex_unlock(&(mesh->mesh_mutex));
                        return false;
 +              }
        }
  
        sptps_stop(&mesh->sptps);
        closesocket(mesh->sock);
  
        if(!mesh->success) {
 -              fprintf(stderr, "Connection closed by peer, invitation cancelled.\n");
 +              logger(mesh, MESHLINK_DEBUG, "Connection closed by peer, invitation cancelled.\n");
 +              meshlink_errno = MESHLINK_EPEER;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return true;
  
  invalid:
 -      fprintf(stderr, "Invalid invitation URL or you are already connected to a Mesh ?\n");
 +      logger(mesh, MESHLINK_DEBUG, "Invalid invitation URL or you are already connected to a Mesh ?\n");
 +      meshlink_errno = MESHLINK_EINVAL;
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return false;
  }
  
  char *meshlink_export(meshlink_handle_t *mesh) {
 +      if(!mesh) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      
        char filename[PATH_MAX];
        snprintf(filename, sizeof filename, "%s" SLASH "hosts" SLASH "%s", mesh->confbase, mesh->self->name);
        FILE *f = fopen(filename, "r");
        if(!f) {
 -              fprintf(stderr, "Could not open %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not open %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
  
        char *buf = xmalloc(len);
        snprintf(buf, len, "Name = %s\n", mesh->self->name);
        if(fread(buf + len - fsize - 1, fsize, 1, f) != 1) {
 -              fprintf(stderr, "Error reading from %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Error reading from %s: %s\n", filename, strerror(errno));
                fclose(f);
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return NULL;
        }
  
        fclose(f);
        buf[len - 1] = 0;
 +      
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return buf;
  }
  
  bool meshlink_import(meshlink_handle_t *mesh, const char *data) {
 +      if(!mesh || !data) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return false;
 +      }
 +      
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +
        if(strncmp(data, "Name = ", 7)) {
 -              fprintf(stderr, "Invalid data\n");
 +              logger(mesh, MESHLINK_DEBUG, "Invalid data\n");
 +              meshlink_errno = MESHLINK_EPEER;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        char *end = strchr(data + 7, '\n');
        if(!end) {
 -              fprintf(stderr, "Invalid data\n");
 +              logger(mesh, MESHLINK_DEBUG, "Invalid data\n");
 +              meshlink_errno = MESHLINK_EPEER;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        memcpy(name, data + 7, len);
        name[len] = 0;
        if(!check_id(name)) {
 -              fprintf(stderr, "Invalid Name\n");
 +              logger(mesh, MESHLINK_DEBUG, "Invalid Name\n");
 +              meshlink_errno = MESHLINK_EPEER;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        char filename[PATH_MAX];
        snprintf(filename, sizeof filename, "%s" SLASH "hosts" SLASH "%s", mesh->confbase, name);
        if(!access(filename, F_OK)) {
 -              fprintf(stderr, "File %s already exists, not importing\n", filename);
 +              logger(mesh, MESHLINK_DEBUG, "File %s already exists, not importing\n", filename);
 +              meshlink_errno = MESHLINK_EEXIST;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        if(errno != ENOENT) {
 -              fprintf(stderr, "Error accessing %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Error accessing %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
        FILE *f = fopen(filename, "w");
        if(!f) {
 -              fprintf(stderr, "Could not create %s: %s\n", filename, strerror(errno));
 +              logger(mesh, MESHLINK_DEBUG, "Could not create %s: %s\n", filename, strerror(errno));
 +              meshlink_errno = MESHLINK_ESTORAGE;
 +              pthread_mutex_unlock(&(mesh->mesh_mutex));
                return false;
        }
  
  
        load_all_nodes(mesh);
  
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
        return true;
  }
  
  void meshlink_blacklist(meshlink_handle_t *mesh, meshlink_node_t *node) {
 -    node_t *n;
 -    n = (node_t*)node;
 -    n->status.blacklisted=true;
 -      fprintf(stderr, "Blacklisted %s.\n",node->name);
 +      if(!mesh || !node) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      
 +      node_t *n;
 +      n = (node_t*)node;
 +      n->status.blacklisted=true;
 +      logger(mesh, MESHLINK_DEBUG, "Blacklisted %s.\n",node->name);
  
        //Make blacklisting persistent in the config file
        append_config_file(mesh, n->name, "blacklisted", "yes");
 -    return;
  
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      return;
 +}
 +
 +void meshlink_whitelist(meshlink_handle_t *mesh, meshlink_node_t *node) {
 +      if(!mesh || !node) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      
 +      node_t *n = (node_t *)node;
 +      n->status.blacklisted = false;
 +
 +      //TODO: remove blacklisted = yes from the config file
 +
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      return;
 +}
 +
 +/* Hint that a hostname may be found at an address
 + * See header file for detailed comment.
 + */
 +void meshlink_hint_address(meshlink_handle_t *mesh, meshlink_node_t *node, const struct sockaddr *addr) {
 +      if(!mesh || !node || !addr)
 +              return;
 +      
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      
 +      char *host = NULL, *port = NULL, *str = NULL;
 +      sockaddr2str((const sockaddr_t *)addr, &host, &port);
 +
 +      if(host && port) {
 +              xasprintf(&str, "%s %s", host, port);
 +              if ( (strncmp ("fe80",host,4) != 0) && ( strncmp("127.",host,4) != 0 ) && ( strcmp("localhost",host) !=0 ) )
 +                      append_config_file(mesh, node->name, "Address", str);
 +              else
 +                      logger(mesh, MESHLINK_DEBUG, "Not adding Link Local IPv6 Address to config\n");
 +      }
 +
 +      free(str);
 +      free(host);
 +      free(port);
 +
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +      // @TODO do we want to fire off a connection attempt right away?
 +}
 +
 +/* Return an array of edges in the current network graph.
 + * Data captures the current state and will not be updated.
 + * Caller must deallocate data when done.
 + */
 +meshlink_edge_t **meshlink_get_all_edges_state(meshlink_handle_t *mesh, meshlink_edge_t **edges, size_t *nmemb) {
 +      if(!mesh || !nmemb || (*nmemb && !edges)) {
 +              meshlink_errno = MESHLINK_EINVAL;
 +              return NULL;
 +      }
 +
 +      pthread_mutex_lock(&(mesh->mesh_mutex));
 +      
 +      meshlink_edge_t **result = NULL;
 +      meshlink_edge_t *copy = NULL;
 +      int result_size = 0;
 +
 +      result_size = mesh->edges->count;
 +
 +      // if result is smaller than edges, we have to dealloc all the excess meshlink_edge_t
 +      if(result_size > *nmemb) {
 +              result = realloc(edges, result_size * sizeof (meshlink_edge_t*));
 +      } else {
 +              result = edges;
 +      }
 +
 +      if(result) {
 +              meshlink_edge_t **p = result;
 +              int n = 0;
 +              for splay_each(edge_t, e, mesh->edges) {
 +                      // skip edges that do not represent a two-directional connection
 +                      if((!e->reverse) || (e->reverse->to != e->from)) {
 +                              result_size--;
 +                              continue;
 +                      }
 +                      n++;
 +                      // the first *nmemb members of result can be re-used
 +                      if(n > *nmemb) {
 +                              copy = xzalloc(sizeof *copy);
 +                      }
 +                      else {
 +                              copy = *p;
 +                      }
 +                      copy->from = (meshlink_node_t*)e->from;
 +                      copy->to = (meshlink_node_t*)e->to;
 +                      copy->address = e->address.storage;
 +                      copy->options = e->options;
 +                      copy->weight = e->weight;
 +                      *p++ = copy;
 +              }
 +              // shrink result to the actual amount of memory used
 +              for(int i = *nmemb; i > result_size; i--) {
 +                      free(result[i - 1]);
 +              }
 +              result = realloc(result, result_size * sizeof (meshlink_edge_t*));
 +              *nmemb = result_size;
 +      } else {
 +              *nmemb = 0;
 +              free(result);
 +              meshlink_errno = MESHLINK_ENOMEM;
 +      }
 +
 +      pthread_mutex_unlock(&(mesh->mesh_mutex));
 +
 +      return result;
  }
  
+ static bool channel_pre_accept(struct utcp *utcp, uint16_t port) {
+       //TODO: implement
+       return false;
+ }
+ static void channel_accept(struct utcp_connection *utcp_connection, uint16_t port) {
+       //TODO: implement
+ }
+ static int channel_recv(struct utcp_connection *connection, const void *data, size_t len) {
+       meshlink_channel_t *channel = connection->priv;
+       node_t *n = channel->node;
+       meshlink_handle_t *mesh = n->mesh;
+       if(!channel->receive_cb)
+               return -1;
+       else {
+               channel->receive_cb(mesh, channel, data, len);
+               return 0;
+       }
+ }
+ static int channel_send(struct utcp *utcp, const void *data, size_t len) {
+       node_t *n = utcp->priv;
+       meshlink_handle_t *mesh = n->mesh;
+       return meshlink_send(mesh, (meshlink_node_t *)n, data, len) ? len : -1;
+ }
+ void meshlink_set_channel_accept_cb(meshlink_handle_t *mesh, meshlink_channel_accept_cb_t cb) {
+       mesh->channel_accept_cb = cb;
+ }
+ void meshlink_set_channel_receive_cb(meshlink_handle_t *mesh, meshlink_channel_t *channel, meshlink_channel_receive_cb_t cb) {
+       channel->receive_cb = cb;
+ }
+ meshlink_channel_t *meshlink_channel_open(meshlink_handle_t *mesh, meshlink_node_t *node, uint16_t port, meshlink_channel_receive_cb_t cb, const void *data, size_t len) {
+       node_t *n = (node_t *)node;
+       if(!n->utcp) {
+               n->utcp = utcp_init(channel_accept, channel_pre_accept, channel_send, n);
+               if(!n->utcp)
+                       return NULL;
+       }
+       meshlink_channel_t *channel = xzalloc(sizeof *channel);
+       channel->node = n;
+       channel->receive_cb = cb;
+       channel->c = utcp_connect(n->utcp, port, channel_recv, channel);
+       if(!channel->c) {
+               free(channel);
+               return NULL;
+       }
+       return channel;
+ }
+ void meshlink_channel_shutdown(meshlink_handle_t *mesh, meshlink_channel_t *channel, int direction) {
+       utcp_shutdown(channel->c, direction);
+ }
+ void meshlink_channel_close(meshlink_handle_t *mesh, meshlink_channel_t *channel) {
+       utcp_close(channel->c);
+       free(channel);
+ }
+ ssize_t meshlink_channel_send(meshlink_handle_t *mesh, meshlink_channel_t *channel, const void *data, size_t len) {
+       // TODO: locking.
+       // Ideally we want to put the data into the UTCP connection's send buffer.
+       // Then, preferrably only if there is room in the receiver window,
+       // kick the meshlink thread to go send packets.
+       return utcp_send(channel->c, data, len);
+ }
  static void __attribute__((constructor)) meshlink_init(void) {
        crypto_init();
  }
  static void __attribute__((destructor)) meshlink_exit(void) {
        crypto_exit();
  }
 +
 +
 +/// Device class traits
 +dev_class_traits_t dev_class_traits[_DEV_CLASS_MAX +1] = {
 +      { .min_connects = 3, .max_connects = 10000, .edge_weight = 1 }, // DEV_CLASS_BACKBONE
 +      { .min_connects = 3, .max_connects = 100, .edge_weight = 3 },   // DEV_CLASS_STATIONARY
 +      { .min_connects = 3, .max_connects = 3, .edge_weight = 6 },             // DEV_CLASS_PORTABLE
 +      { .min_connects = 1, .max_connects = 1, .edge_weight = 9 },             // DEV_CLASS_UNKNOWN
 +};
diff --combined src/meshlink_internal.h
index 994f25bf9bd5f077403ffc67ec5f484ce6165d50,0e035920c08345c98fd72db4c6142ee177b21cc5..517dc1e01cb8dbee31824f040f7c0a6140015f9e
@@@ -24,8 -24,8 +24,8 @@@
  
  #include "event.h"
  #include "hash.h"
 -#include "logger.h"
  #include "meshlink.h"
 +#include "meshlink_queue.h"
  #include "sockaddr.h"
  #include "sptps.h"
  
  
  #define MAXSOCKETS 8    /* Probably overkill... */
  
 +struct AvahiServer;
 +struct AvahiSServiceBrowser;
 +struct AvahiSimplePoll;
 +struct AvahiSEntryGroup;
 +
  typedef struct listen_socket_t {
        struct io_t tcp;
        struct io_t udp;
@@@ -62,22 -57,20 +62,24 @@@ typedef struct outpacketqueue 
  
  /// A handle for an instance of MeshLink.
  struct meshlink_handle {
 -      char *confbase;
        char *name;
 +      char *appname;
 +      dev_class_t devclass;
 +      void *priv;
 +
 +      char *confbase;
  
        meshlink_receive_cb_t receive_cb;
        meshlink_node_status_cb_t node_status_cb;
        meshlink_log_cb_t log_cb;
        meshlink_log_level_t log_level;
  
+       meshlink_channel_accept_cb_t channel_accept_cb;
        pthread_t thread;
        bool threadstarted;
        pthread_mutex_t outpacketqueue_mutex;
 -      pthread_mutex_t nodes_mutex;
 +      pthread_mutex_t mesh_mutex;
        event_loop_t loop;
        listen_socket_t listen_socket[MAXSOCKETS];
        int listen_sockets;
@@@ -92,7 -85,7 +94,7 @@@
        struct list_t *connections;
        struct list_t *outgoings;
  
 -      struct list_t *outpacketqueue;
 +      meshlink_queue_t outpacketqueue;
  
        struct splay_tree_t *past_request_tree;
        timeout_t past_request_timeout;
        struct connection_t *everyone;
        struct ecdsa *invitation_key;
  
 -      debug_t debug_level;
        int pinginterval;       /* seconds between pings */
        int pingtimeout;        /* seconds to wait for response */
        int maxtimeout;
        char line[4096];
        char buffer[4096];
        size_t blen;
 +
 +      pthread_t discovery_thread;
 +      bool discovery_threadstarted;
 +      struct AvahiServer *avahi_server;
 +      struct AvahiSServiceBrowser *avahi_browser;
 +      struct AvahiSimplePoll *avahi_poll;
 +      struct AvahiSEntryGroup *avahi_group;
 +      char* avahi_servicetype;
  };
  
  /// A handle for a MeshLink node.
@@@ -148,6 -134,13 +150,13 @@@ struct meshlink_node 
        void *priv;
  };
  
+ /// A channel.
+ struct meshlink_channel {
+       struct utcp_connection *c;
+       struct node_t *node;
+       meshlink_channel_receive_cb_t receive_cb;
+ };
  /// Header for data packets routed between nodes
  typedef struct meshlink_packethdr {
        uint8_t destination[16];
  } __attribute__ ((__packed__)) meshlink_packethdr_t;
  
  extern void meshlink_send_from_queue(event_loop_t* el,meshlink_handle_t *mesh);
 +extern meshlink_log_level_t global_log_level;
 +extern meshlink_log_cb_t global_log_cb;
 +
 +/// Device class traits
 +typedef struct {
 +      unsigned int min_connects;
 +      unsigned int max_connects;
 +      int edge_weight;
 +} dev_class_traits_t;
  
 +extern dev_class_traits_t dev_class_traits[];
  
  #endif // MESHLINK_INTERNAL_H
diff --combined src/node.h
index c86c3cee9a650225137f9c1f17f73cc96effd306,427bc83a000907c6f8fb7e65602dd704ab651397..5761e4d747e5544c2d5189fdaed074eaa597ac12
@@@ -23,6 -23,7 +23,7 @@@
  #include "event.h"
  #include "sockaddr.h"
  #include "sptps.h"
+ #include "utcp/utcp.h"
  
  typedef struct node_status_t {
        unsigned int unused_active:1;           /* 1 if active (not used for nodes) */
  
  typedef struct node_t {
        char *name;                             /* name of this node */
+       void *priv;
        uint32_t options;                       /* options turned on for this node */
-       void *priv;
 +      dev_class_t devclass;
  
        struct meshlink_handle *mesh;           /* The mesh this node belongs to */
  
@@@ -68,8 -69,6 +70,8 @@@
        struct splay_tree_t *edge_tree;                /* Edges with this node as one of the endpoints */
  
        struct connection_t *connection;        /* Connection associated with this node (if a direct connection exists) */
 +      time_t last_connect_try;
 +      time_t last_successfull_connection;
  
        uint32_t sent_seqno;                    /* Sequence number last sent to this node */
        uint32_t received_seqno;                /* Sequence number last received from this node */
@@@ -89,6 -88,8 +91,8 @@@
        float bandwidth;                        /* Last measured bandwidth */
        float packetloss;                       /* Last measured packet loss rate */
  
+       struct utcp *utcp;
        uint64_t in_packets;
        uint64_t in_bytes;
        uint64_t out_packets;
@@@ -101,7 -102,7 +105,7 @@@ extern node_t *new_node(void) __attribu
  extern void free_node(node_t *);
  extern void node_add(struct meshlink_handle *mesh, node_t *);
  extern void node_del(struct meshlink_handle *mesh, node_t *);
 -extern node_t *lookup_node(struct meshlink_handle *mesh, char *);
 +extern node_t *lookup_node(struct meshlink_handle *mesh, const char *);
  extern node_t *lookup_node_udp(struct meshlink_handle *mesh, const sockaddr_t *);
  extern void update_node_udp(struct meshlink_handle *mesh, node_t *, const sockaddr_t *);