Remove unnecessary status bitfield conversions.
[tinc] / src / tincctl.c
index 4dd9726..a3978bc 100644 (file)
@@ -1,6 +1,6 @@
 /*
     tincctl.c -- Controlling a running tincd
-    Copyright (C) 2007-2021 Guus Sliepen <guus@tinc-vpn.org>
+    Copyright (C) 2007-2022 Guus Sliepen <guus@tinc-vpn.org>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -72,7 +72,8 @@ bool force = false;
 bool tty = true;
 bool confbasegiven = false;
 char *scriptinterpreter = NULL;
-char *scriptextension = "";
+static char defaultextension[] = "";
+char *scriptextension = defaultextension;
 static char *prompt;
 char *device = NULL;
 char *iface = NULL;
@@ -90,88 +91,95 @@ static struct option const long_options[] = {
 };
 
 static void version(void) {
-       printf("%s version %s (built %s %s, protocol %d.%d)\n", PACKAGE,
-              BUILD_VERSION, BUILD_DATE, BUILD_TIME, PROT_MAJOR, PROT_MINOR);
-       printf("Features:"
+       static const char *message =
+               "%s version %s (built %s %s, protocol %d.%d)\n"
+               "Features:"
 #ifdef HAVE_READLINE
-              " readline"
+               " readline"
 #endif
 #ifdef HAVE_CURSES
-              " curses"
+               " curses"
 #endif
 #ifndef DISABLE_LEGACY
-              " legacy_protocol"
+               " legacy_protocol"
 #endif
-              "\n\n");
-       printf("Copyright (C) 1998-2018 Ivo Timmermans, Guus Sliepen and others.\n"
-              "See the AUTHORS file for a complete list.\n\n"
-              "tinc comes with ABSOLUTELY NO WARRANTY.  This is free software,\n"
-              "and you are welcome to redistribute it under certain conditions;\n"
-              "see the file COPYING for details.\n");
+               "\n\n"
+               "Copyright (C) 1998-2018 Ivo Timmermans, Guus Sliepen and others.\n"
+               "See the AUTHORS file for a complete list.\n"
+               "\n"
+               "tinc comes with ABSOLUTELY NO WARRANTY.  This is free software,\n"
+               "and you are welcome to redistribute it under certain conditions;\n"
+               "see the file COPYING for details.\n";
+
+       printf(message, PACKAGE, BUILD_VERSION, BUILD_DATE, BUILD_TIME, PROT_MAJOR, PROT_MINOR);
 }
 
 static void usage(bool status) {
        if(status) {
                fprintf(stderr, "Try `%s --help\' for more information.\n", program_name);
        } else {
-               printf("Usage: %s [options] command\n\n", program_name);
-               printf("Valid options are:\n"
-                      "  -b, --batch             Don't ask for anything (non-interactive mode).\n"
-                      "  -c, --config=DIR        Read configuration options from DIR.\n"
-                      "  -n, --net=NETNAME       Connect to net NETNAME.\n"
-                      "      --pidfile=FILENAME  Read control cookie from FILENAME.\n"
-                      "      --force             Force some commands to work despite warnings.\n"
-                      "      --help              Display this help and exit.\n"
-                      "      --version           Output version information and exit.\n"
-                      "\n"
-                      "Valid commands are:\n"
-                      "  init [name]                Create initial configuration files.\n"
-                      "  get VARIABLE               Print current value of VARIABLE\n"
-                      "  set VARIABLE VALUE         Set VARIABLE to VALUE\n"
-                      "  add VARIABLE VALUE         Add VARIABLE with the given VALUE\n"
-                      "  del VARIABLE [VALUE]       Remove VARIABLE [only ones with watching VALUE]\n"
-                      "  start [tincd options]      Start tincd.\n"
-                      "  stop                       Stop tincd.\n"
-                      "  restart [tincd options]    Restart tincd.\n"
-                      "  reload                     Partially reload configuration of running tincd.\n"
-                      "  pid                        Show PID of currently running tincd.\n"
+               static const char *message =
+                       "Usage: %s [options] command\n"
+                       "\n"
+                       "Valid options are:\n"
+                       "  -b, --batch             Don't ask for anything (non-interactive mode).\n"
+                       "  -c, --config=DIR        Read configuration options from DIR.\n"
+                       "  -n, --net=NETNAME       Connect to net NETNAME.\n"
+                       "      --pidfile=FILENAME  Read control cookie from FILENAME.\n"
+                       "      --force             Force some commands to work despite warnings.\n"
+                       "      --help              Display this help and exit.\n"
+                       "      --version           Output version information and exit.\n"
+                       "\n"
+                       "Valid commands are:\n"
+                       "  init [name]                Create initial configuration files.\n"
+                       "  get VARIABLE               Print current value of VARIABLE\n"
+                       "  set VARIABLE VALUE         Set VARIABLE to VALUE\n"
+                       "  add VARIABLE VALUE         Add VARIABLE with the given VALUE\n"
+                       "  del VARIABLE [VALUE]       Remove VARIABLE [only ones with watching VALUE]\n"
+                       "  start [tincd options]      Start tincd.\n"
+                       "  stop                       Stop tincd.\n"
+                       "  restart [tincd options]    Restart tincd.\n"
+                       "  reload                     Partially reload configuration of running tincd.\n"
+                       "  pid                        Show PID of currently running tincd.\n"
 #ifdef DISABLE_LEGACY
-                      "  generate-keys              Generate a new Ed25519 public/private key pair.\n"
+                       "  generate-keys              Generate a new Ed25519 public/private key pair.\n"
 #else
-                      "  generate-keys [bits]       Generate new RSA and Ed25519 public/private key pairs.\n"
-                      "  generate-rsa-keys [bits]   Generate a new RSA public/private key pair.\n"
+                       "  generate-keys [bits]       Generate new RSA and Ed25519 public/private key pairs.\n"
+                       "  generate-rsa-keys [bits]   Generate a new RSA public/private key pair.\n"
 #endif
-                      "  generate-ed25519-keys      Generate a new Ed25519 public/private key pair.\n"
-                      "  dump                       Dump a list of one of the following things:\n"
-                      "    [reachable] nodes        - all known nodes in the VPN\n"
-                      "    edges                    - all known connections in the VPN\n"
-                      "    subnets                  - all known subnets in the VPN\n"
-                      "    connections              - all meta connections with ourself\n"
-                      "    [di]graph                - graph of the VPN in dotty format\n"
-                      "    invitations              - outstanding invitations\n"
-                      "  info NODE|SUBNET|ADDRESS   Give information about a particular NODE, SUBNET or ADDRESS.\n"
-                      "  purge                      Purge unreachable nodes\n"
-                      "  debug N                    Set debug level\n"
-                      "  retry                      Retry all outgoing connections\n"
-                      "  disconnect NODE            Close meta connection with NODE\n"
+                       "  generate-ed25519-keys      Generate a new Ed25519 public/private key pair.\n"
+                       "  dump                       Dump a list of one of the following things:\n"
+                       "    [reachable] nodes        - all known nodes in the VPN\n"
+                       "    edges                    - all known connections in the VPN\n"
+                       "    subnets                  - all known subnets in the VPN\n"
+                       "    connections              - all meta connections with ourself\n"
+                       "    [di]graph                - graph of the VPN in dotty format\n"
+                       "    invitations              - outstanding invitations\n"
+                       "  info NODE|SUBNET|ADDRESS   Give information about a particular NODE, SUBNET or ADDRESS.\n"
+                       "  purge                      Purge unreachable nodes\n"
+                       "  debug N                    Set debug level\n"
+                       "  retry                      Retry all outgoing connections\n"
+                       "  disconnect NODE            Close meta connection with NODE\n"
 #ifdef HAVE_CURSES
-                      "  top                        Show real-time statistics\n"
+                       "  top                        Show real-time statistics\n"
 #endif
-                      "  pcap [snaplen]             Dump traffic in pcap format [up to snaplen bytes per packet]\n"
-                      "  log [level]                Dump log output [up to the specified level]\n"
-                      "  export                     Export host configuration of local node to standard output\n"
-                      "  export-all                 Export all host configuration files to standard output\n"
-                      "  import                     Import host configuration file(s) from standard input\n"
-                      "  exchange                   Same as export followed by import\n"
-                      "  exchange-all               Same as export-all followed by import\n"
-                      "  invite NODE [...]          Generate an invitation for NODE\n"
-                      "  join INVITATION            Join a VPN using an INVITATION\n"
-                      "  network [NETNAME]          List all known networks, or switch to the one named NETNAME.\n"
-                      "  fsck                       Check the configuration files for problems.\n"
-                      "  sign [FILE]                Generate a signed version of a file.\n"
-                      "  verify NODE [FILE]         Verify that a file was signed by the given NODE.\n"
-                      "\n");
-               printf("Report bugs to tinc@tinc-vpn.org.\n");
+                       "  pcap [snaplen]             Dump traffic in pcap format [up to snaplen bytes per packet]\n"
+                       "  log [level]                Dump log output [up to the specified level]\n"
+                       "  export                     Export host configuration of local node to standard output\n"
+                       "  export-all                 Export all host configuration files to standard output\n"
+                       "  import                     Import host configuration file(s) from standard input\n"
+                       "  exchange                   Same as export followed by import\n"
+                       "  exchange-all               Same as export-all followed by import\n"
+                       "  invite NODE [...]          Generate an invitation for NODE\n"
+                       "  join INVITATION            Join a VPN using an INVITATION\n"
+                       "  network [NETNAME]          List all known networks, or switch to the one named NETNAME.\n"
+                       "  fsck                       Check the configuration files for problems.\n"
+                       "  sign [FILE]                Generate a signed version of a file.\n"
+                       "  verify NODE [FILE]         Verify that a file was signed by the given NODE.\n"
+                       "\n"
+                       "Report bugs to tinc@tinc-vpn.org.\n";
+
+               printf(message, program_name);
        }
 }
 
@@ -282,8 +290,12 @@ ask_filename:
 
        if(filename[0] != '/') {
 #endif
+
                /* The directory is a relative path or a filename. */
-               getcwd(directory, sizeof(directory));
+               if(!getcwd(directory, sizeof(directory))) {
+                       fprintf(stderr, "Could not get current directory: %s\n", strerror(errno));
+                       return NULL;
+               }
 
                if((size_t)snprintf(buf2, sizeof(buf2), "%s" SLASH "%s", directory, filename) >= sizeof(buf2)) {
                        fprintf(stderr, "Filename too long: %s" SLASH "%s\n", directory, filename);
@@ -463,15 +475,15 @@ bool recvline(int fd, char *line, size_t len) {
        }
 
        while(!(newline = memchr(buffer, '\n', blen))) {
-               int result = recv(fd, buffer + blen, sizeof(buffer) - blen, 0);
+               ssize_t nrecv = recv(fd, buffer + blen, sizeof(buffer) - blen, 0);
 
-               if(result == -1 && sockerrno == EINTR) {
+               if(nrecv == -1 && sockerrno == EINTR) {
                        continue;
-               } else if(result <= 0) {
+               } else if(nrecv <= 0) {
                        return false;
                }
 
-               blen += result;
+               blen += nrecv;
        }
 
        if((size_t)(newline - buffer) >= len) {
@@ -490,15 +502,15 @@ bool recvline(int fd, char *line, size_t len) {
 
 static bool recvdata(int fd, char *data, size_t len) {
        while(blen < len) {
-               int result = recv(fd, buffer + blen, sizeof(buffer) - blen, 0);
+               ssize_t nrecv = recv(fd, buffer + blen, sizeof(buffer) - blen, 0);
 
-               if(result == -1 && sockerrno == EINTR) {
+               if(nrecv == -1 && sockerrno == EINTR) {
                        continue;
-               } else if(result <= 0) {
+               } else if(nrecv <= 0) {
                        return false;
                }
 
-               blen += result;
+               blen += nrecv;
        }
 
        memcpy(data, buffer, len);
@@ -508,10 +520,10 @@ static bool recvdata(int fd, char *data, size_t len) {
        return true;
 }
 
-bool sendline(int fd, char *format, ...) {
+bool sendline(int fd, const char *format, ...) {
        static char buffer[4096];
        char *p = buffer;
-       int blen;
+       ssize_t blen;
        va_list ap;
 
        va_start(ap, format);
@@ -527,16 +539,16 @@ bool sendline(int fd, char *format, ...) {
        blen++;
 
        while(blen) {
-               int result = send(fd, p, blen, MSG_NOSIGNAL);
+               ssize_t nsend = send(fd, p, blen, MSG_NOSIGNAL);
 
-               if(result == -1 && sockerrno == EINTR) {
+               if(nsend == -1 && sockerrno == EINTR) {
                        continue;
-               } else if(result <= 0) {
+               } else if(nsend <= 0) {
                        return false;
                }
 
-               p += result;
-               blen -= result;
+               p += nsend;
+               blen -= nsend;
        }
 
        return true;
@@ -577,11 +589,12 @@ static void pcap(int fd, FILE *out, uint32_t snaplen) {
        char line[32];
 
        while(recvline(fd, line, sizeof(line))) {
-               int code, req, len;
-               int n = sscanf(line, "%d %d %d", &code, &req, &len);
+               int code, req;
+               unsigned long len;
+               int n = sscanf(line, "%d %d %lu", &code, &req, &len);
                gettimeofday(&tv, NULL);
 
-               if(n != 3 || code != CONTROL || req != REQ_PCAP || len < 0 || (size_t)len > sizeof(data)) {
+               if(n != 3 || code != CONTROL || req != REQ_PCAP || len > sizeof(data)) {
                        break;
                }
 
@@ -883,12 +896,10 @@ static int cmd_start(int argc, char *argv[]) {
 
 #endif
 
-       char *default_c = "tincd";
-
        if(slash++) {
                xasprintf(&c, "%.*stincd", (int)(slash - program_name), program_name);
        } else {
-               c = default_c;
+               c = xstrdup("tincd");
        }
 
        int nargc = 0;
@@ -920,10 +931,7 @@ static int cmd_start(int argc, char *argv[]) {
        int status = spawnvp(_P_WAIT, c, nargv);
 
        free(nargv);
-
-       if(c != default_c) {
-               free(c);
-       }
+       free(c);
 
        if(status == -1) {
                fprintf(stderr, "Error starting %s: %s\n", c, strerror(errno));
@@ -937,11 +945,7 @@ static int cmd_start(int argc, char *argv[]) {
        if(socketpair(AF_UNIX, SOCK_STREAM, 0, pfd)) {
                fprintf(stderr, "Could not create umbilical socket: %s\n", strerror(errno));
                free(nargv);
-
-               if(c != default_c) {
-                       free(c);
-               }
-
+               free(c);
                return 1;
        }
 
@@ -950,11 +954,7 @@ static int cmd_start(int argc, char *argv[]) {
        if(pid == -1) {
                fprintf(stderr, "Could not fork: %s\n", strerror(errno));
                free(nargv);
-
-               if(c != default_c) {
-                       free(c);
-               }
-
+               free(c);
                return 1;
        }
 
@@ -970,7 +970,6 @@ static int cmd_start(int argc, char *argv[]) {
 
        free(nargv);
 
-       int status = -1, result;
 #ifdef SIGINT
        signal(SIGINT, SIG_IGN);
 #endif
@@ -978,7 +977,7 @@ static int cmd_start(int argc, char *argv[]) {
        // Pass all log messages from the umbilical to stderr.
        // A nul-byte right before closure means tincd started successfully.
        bool failure = true;
-       char buf[1024];
+       uint8_t buf[1024];
        ssize_t len;
 
        while((len = read(pfd[0], buf, sizeof(buf))) > 0) {
@@ -988,7 +987,9 @@ static int cmd_start(int argc, char *argv[]) {
                        len--;
                }
 
-               write(2, buf, len);
+               if(write(2, buf, len) != len) {
+                       // Nothing we can do about it.
+               }
        }
 
        if(len) {
@@ -998,7 +999,8 @@ static int cmd_start(int argc, char *argv[]) {
        close(pfd[0]);
 
        // Make sure the child process is really gone.
-       result = waitpid(pid, &status, 0);
+       int status = -1;
+       pid_t result = waitpid(pid, &status, 0);
 
 #ifdef SIGINT
        signal(SIGINT, SIG_DFL);
@@ -1010,9 +1012,7 @@ static int cmd_start(int argc, char *argv[]) {
                fprintf(stderr, "Error starting %s\n", c);
        }
 
-       if(c != default_c) {
-               free(c);
-       }
+       free(c);
 
        return failed ? EXIT_FAILURE : EXIT_SUCCESS;
 #endif
@@ -1099,7 +1099,7 @@ static int dump_invitations(void) {
        while((ent = readdir(dir))) {
                char buf[MAX_STRING_SIZE];
 
-               if(b64decode(ent->d_name, buf, 24) != 18) {
+               if(b64decode_tinc(ent->d_name, buf, 24) != 18) {
                        continue;
                }
 
@@ -1243,7 +1243,7 @@ static int cmd_dump(int argc, char *argv[]) {
                char nexthop[4096];
                int cipher, digest, maclength, compression, distance, socket, weight;
                short int pmtu, minmtu, maxmtu;
-               unsigned int options, status_int;
+               unsigned int options;
                node_status_t status;
                long int last_state_change;
                int udp_ping_rtt;
@@ -1251,15 +1251,13 @@ static int cmd_dump(int argc, char *argv[]) {
 
                switch(req) {
                case REQ_DUMP_NODES: {
-                       int n = sscanf(line, "%*d %*d %4095s %4095s %4095s port %4095s %d %d %d %d %x %x %4095s %4095s %d %hd %hd %hd %ld %d %"PRIu64" %"PRIu64" %"PRIu64" %"PRIu64, node, id, host, port, &cipher, &digest, &maclength, &compression, &options, &status_int, nexthop, via, &distance, &pmtu, &minmtu, &maxmtu, &last_state_change, &udp_ping_rtt, &in_packets, &in_bytes, &out_packets, &out_bytes);
+                       int n = sscanf(line, "%*d %*d %4095s %4095s %4095s port %4095s %d %d %d %d %x %"PRIx32" %4095s %4095s %d %hd %hd %hd %ld %d %"PRIu64" %"PRIu64" %"PRIu64" %"PRIu64, node, id, host, port, &cipher, &digest, &maclength, &compression, &options, &status.value, nexthop, via, &distance, &pmtu, &minmtu, &maxmtu, &last_state_change, &udp_ping_rtt, &in_packets, &in_bytes, &out_packets, &out_bytes);
 
                        if(n != 22) {
                                fprintf(stderr, "Unable to parse node dump from tincd: %s\n", line);
                                return 1;
                        }
 
-                       memcpy(&status, &status_int, sizeof(status));
-
                        if(do_graph) {
                                const char *color = "black";
 
@@ -1282,7 +1280,7 @@ static int cmd_dump(int argc, char *argv[]) {
                                }
 
                                printf("%s id %s at %s port %s cipher %d digest %d maclength %d compression %d options %x status %04x nexthop %s via %s distance %d pmtu %d (min %d max %d) rx %"PRIu64" %"PRIu64" tx %"PRIu64" %"PRIu64,
-                                      node, id, host, port, cipher, digest, maclength, compression, options, status_int, nexthop, via, distance, pmtu, minmtu, maxmtu, in_packets, in_bytes, out_packets, out_bytes);
+                                      node, id, host, port, cipher, digest, maclength, compression, options, status.value, nexthop, via, distance, pmtu, minmtu, maxmtu, in_packets, in_bytes, out_packets, out_bytes);
 
                                if(udp_ping_rtt != -1) {
                                        printf(" rtt %d.%03d", udp_ping_rtt / 1000, udp_ping_rtt % 1000);
@@ -1302,7 +1300,7 @@ static int cmd_dump(int argc, char *argv[]) {
                        }
 
                        if(do_graph) {
-                               float w = 1 + 65536.0 / weight;
+                               float w = 1.0f + 65536.0f / (float)weight;
 
                                if(do_graph == 1 && strcmp(node1, node2) > 0) {
                                        printf(" \"%s\" -- \"%s\" [w = %f, weight = %f];\n", node1, node2, w, w);
@@ -1328,14 +1326,14 @@ static int cmd_dump(int argc, char *argv[]) {
                break;
 
                case REQ_DUMP_CONNECTIONS: {
-                       int n = sscanf(line, "%*d %*d %4095s %4095s port %4095s %x %d %x", node, host, port, &options, &socket, &status_int);
+                       int n = sscanf(line, "%*d %*d %4095s %4095s port %4095s %x %d %x", node, host, port, &options, &socket, &status.value);
 
                        if(n != 6) {
                                fprintf(stderr, "Unable to parse connection dump from tincd.\n");
                                return 1;
                        }
 
-                       printf("%s at %s port %s options %x socket %d status %x\n", node, host, port, options, socket, status_int);
+                       printf("%s at %s port %s options %x socket %d status %x\n", node, host, port, options, socket, status.value);
                }
                break;
 
@@ -1553,8 +1551,8 @@ static int cmd_pid(int argc, char *argv[]) {
        return 0;
 }
 
-int rstrip(char *value) {
-       int len = strlen(value);
+size_t rstrip(char *value) {
+       size_t len = strlen(value);
 
        while(len && strchr("\t\r\n ", value[len - 1])) {
                value[--len] = 0;
@@ -1578,7 +1576,7 @@ char *get_my_name(bool verbose) {
        char *value;
 
        while(fgets(buf, sizeof(buf), f)) {
-               int len = strcspn(buf, "\t =");
+               size_t len = strcspn(buf, "\t =");
                value = buf + len;
                value += strspn(value, "\t ");
 
@@ -1617,7 +1615,7 @@ ecdsa_t *get_pubkey(FILE *f) {
        char *value;
 
        while(fgets(buf, sizeof(buf), f)) {
-               int len = strcspn(buf, "\t =");
+               size_t len = strcspn(buf, "\t =");
                value = buf + len;
                value += strspn(value, "\t ");
 
@@ -1944,9 +1942,8 @@ static int cmd_config(int argc, char *argv[]) {
 
                // Parse line in a simple way
                char *bvalue;
-               int len;
 
-               len = strcspn(buf2, "\t =");
+               size_t len = strcspn(buf2, "\t =");
                bvalue = buf2 + len;
                bvalue += strspn(bvalue, "\t ");
 
@@ -2127,7 +2124,7 @@ int check_port(const char *name) {
        fprintf(stderr, "Warning: could not bind to port 655. ");
 
        for(int i = 0; i < 100; i++) {
-               int port = 0x1000 + (rand() & 0x7fff);
+               uint16_t port = 0x1000 + prng(0x8000);
 
                if(try_bind(port)) {
                        char filename[PATH_MAX];
@@ -2170,7 +2167,7 @@ static int cmd_init(int argc, char *argv[]) {
                                return 1;
                        }
 
-                       int len = rstrip(buf);
+                       size_t len = rstrip(buf);
 
                        if(!len) {
                                fprintf(stderr, "No name given!\n");
@@ -2787,7 +2784,7 @@ static int cmd_sign(int argc, char *argv[]) {
        long t = time(NULL);
        char *trailer;
        xasprintf(&trailer, " %s %ld", name, t);
-       int trailer_len = strlen(trailer);
+       size_t trailer_len = strlen(trailer);
 
        data = xrealloc(data, len + trailer_len);
        memcpy(data + len, trailer, trailer_len);
@@ -2802,7 +2799,7 @@ static int cmd_sign(int argc, char *argv[]) {
                return 1;
        }
 
-       b64encode(sig, sig, 64);
+       b64encode_tinc(sig, sig, 64);
        ecdsa_free(key);
 
        fprintf(stdout, "Signature = %s %ld %s\n", name, t, sig);
@@ -2902,7 +2899,7 @@ static int cmd_verify(int argc, char *argv[]) {
 
        char *trailer;
        xasprintf(&trailer, " %s %ld", signer, t);
-       int trailer_len = strlen(trailer);
+       size_t trailer_len = strlen(trailer);
 
        data = xrealloc(data, len + trailer_len);
        memcpy(data + len, trailer, trailer_len);
@@ -2936,7 +2933,7 @@ static int cmd_verify(int argc, char *argv[]) {
 
        fclose(fp);
 
-       if(b64decode(sig, sig, 86) != 64 || !ecdsa_verify(key, newline, len + trailer_len - (newline - data), sig)) {
+       if(b64decode_tinc(sig, sig, 86) != 64 || !ecdsa_verify(key, newline, len + trailer_len - (newline - data), sig)) {
                fprintf(stderr, "Invalid signature\n");
                free(data);
                ecdsa_free(key);
@@ -3158,6 +3155,7 @@ static int cmd_shell(int argc, char *argv[]) {
 
 #ifdef HAVE_READLINE
        rl_readline_name = "tinc";
+       rl_basic_word_break_characters = "\t\n ";
        rl_completion_entry_function = complete_nothing;
        rl_attempted_completion_function = completion;
        rl_filename_completion_desired = 0;
@@ -3170,7 +3168,6 @@ static int cmd_shell(int argc, char *argv[]) {
                if(tty) {
                        free(copy);
                        free(line);
-                       rl_basic_word_break_characters = "\t\n ";
                        line = readline(prompt);
                        copy = line ? xstrdup(line) : NULL;
                } else {
@@ -3261,7 +3258,7 @@ static int cmd_shell(int argc, char *argv[]) {
        return result;
 }
 
-static void cleanup() {
+static void cleanup(void) {
        free(tinc_conf);
        free(hosts_dir);
        free_names();
@@ -3302,8 +3299,8 @@ int main(int argc, char *argv[]) {
 #endif
 
        gettimeofday(&now, NULL);
-       srand(now.tv_sec + now.tv_usec);
        crypto_init();
+       prng_init();
 
        if(optind >= argc) {
                return cmd_shell(argc, argv);