oweals/tinc.git
15 years agoRevert "Raise default crypto algorithms to AES256 and SHA256."
Guus Sliepen [Sun, 11 Oct 2009 11:56:04 +0000 (13:56 +0200)]
Revert "Raise default crypto algorithms to AES256 and SHA256."

Although it would be better to have the new defaults, only the most recent
releases of most of the platforms supported by tinc come with a version of
OpenSSL that supports SHA256. To ensure people can compile tinc and that nodes
can interact with each other, we revert the default back to Blowfish and SHA1.

This reverts commit 4bb3793e38b7c7f24dd308801e7f6dbb02cf02d2.

15 years agoRemove code duplication when checking ADD_EDGE/DEL_EDGE messages.
Guus Sliepen [Sun, 11 Oct 2009 11:54:05 +0000 (13:54 +0200)]
Remove code duplication when checking ADD_EDGE/DEL_EDGE messages.

15 years agoDon't disconnect clients in TunnelServer mode who send unauthorised ADD_SUBNETs.
Guus Sliepen [Sun, 11 Oct 2009 11:51:10 +0000 (13:51 +0200)]
Don't disconnect clients in TunnelServer mode who send unauthorised ADD_SUBNETs.

So that we are liberal in what we accept.

15 years agoRemoved last gettext function.
Borg [Sat, 3 Oct 2009 11:06:00 +0000 (13:06 +0200)]
Removed last gettext function.

15 years agoRemove autogenerated files from EXTRA_DIST.
Guus Sliepen [Tue, 29 Sep 2009 14:25:20 +0000 (16:25 +0200)]
Remove autogenerated files from EXTRA_DIST.

Apparently they were once necessary, but autoconf now includes them
automatically.  Some of them are not used anymore, and this caused make dist to
fail.

15 years agoUpdate the NEWS.
Guus Sliepen [Sat, 26 Sep 2009 10:51:52 +0000 (12:51 +0200)]
Update the NEWS.

15 years agoAdd more authors to the copyright headers.
Guus Sliepen [Fri, 25 Sep 2009 19:14:56 +0000 (21:14 +0200)]
Add more authors to the copyright headers.

Git's log and blame tools were used to find out which files had significant
contributions from authors who sent in patches that were applied before we used
git.

15 years agoDrop support for localisation.
Guus Sliepen [Thu, 24 Sep 2009 22:54:07 +0000 (00:54 +0200)]
Drop support for localisation.

Localised messages don't make much sense for a daemon, and there is only the
Dutch translation which costs time to maintain.

15 years agoRemove checkpoint tracing.
Guus Sliepen [Thu, 24 Sep 2009 22:33:04 +0000 (00:33 +0200)]
Remove checkpoint tracing.

This feature is not necessary anymore since we have tools like valgrind today
that can catch stack overflow errors before they make a backtrace in gdb
impossible.

15 years agoK&R style braces.
Guus Sliepen [Thu, 24 Sep 2009 22:14:03 +0000 (00:14 +0200)]
K&R style braces.

This is essentially commit f02d3ed3e135b5326003e7f69f8331ff6a3cc219 from the
1.1 branch, making it easier to merge between master and 1.1.

15 years agoUpdate the address of the Free Software Foundation in all copyright headers.
Guus Sliepen [Thu, 24 Sep 2009 22:01:00 +0000 (00:01 +0200)]
Update the address of the Free Software Foundation in all copyright headers.

15 years agoRemove Ivo's old email addresses.
Guus Sliepen [Thu, 24 Sep 2009 21:42:30 +0000 (23:42 +0200)]
Remove Ivo's old email addresses.

15 years agoRemove all occurences of $Id$.
Guus Sliepen [Thu, 24 Sep 2009 21:39:16 +0000 (23:39 +0200)]
Remove all occurences of $Id$.

15 years agoUpdate copyright information.
Guus Sliepen [Thu, 24 Sep 2009 21:29:46 +0000 (23:29 +0200)]
Update copyright information.

- Update year numbers in copyright headers.
- Add copyright information for Michael Tokarev and Florian Forster to the
  copyright headers of files to which they have contributed significantly.
- Mention Michael and Florian in AUTHORS.
- Mention that tinc is GPLv3 or later if compiled with the --enable-tunemu
  flag.

15 years agoSend large packets we cannot handle properly via TCP.
Guus Sliepen [Tue, 15 Sep 2009 21:22:13 +0000 (23:22 +0200)]
Send large packets we cannot handle properly via TCP.

During the path MTU discovery phase, we might not know the maximum MTU yet, but
we do know a safe minimum.  If we encounter a packet that is larger than that
the minimum, we now send it via TCP instead to ensure it arrives.  We also
allow large packets that we cannot fragment or create ICMP replies for to be
sent via TCP.

15 years agoRaise default RSA key length to 2048 bits.
Guus Sliepen [Tue, 15 Sep 2009 21:04:52 +0000 (23:04 +0200)]
Raise default RSA key length to 2048 bits.

15 years agoUse a mutex to allow the TAP reader to process packets faster on Windows.
Guus Sliepen [Tue, 15 Sep 2009 20:59:01 +0000 (22:59 +0200)]
Use a mutex to allow the TAP reader to process packets faster on Windows.

The TAP-Win32 device is not a socket, and select() under Windows only works
with sockets.  Tinc used a separate thread to read from the TAP-Win32 device,
and passed this via a local socket to the main thread which could then select()
from it. We now use a global mutex, which is only unlocked when the main thread
is waiting for select(), to allow the TAP reader thread to process packets
directly.

15 years agoRemove extra {.
Guus Sliepen [Tue, 15 Sep 2009 20:58:16 +0000 (22:58 +0200)]
Remove extra {.

15 years agoRaise default crypto algorithms to AES256 and SHA256.
Guus Sliepen [Tue, 15 Sep 2009 10:08:05 +0000 (12:08 +0200)]
Raise default crypto algorithms to AES256 and SHA256.

In light of the recent improvements of attacks on SHA1, the default hash
algorithm in tinc is now SHA256. At the same time, the default symmetric
encryption algorithm has been changed to AES256.

15 years agoUse access() instead of stat() for checking whether scripts exist.
Guus Sliepen [Mon, 14 Sep 2009 22:36:07 +0000 (00:36 +0200)]
Use access() instead of stat() for checking whether scripts exist.

15 years agoRemove dropin random() function, as it is not used anymore.
Guus Sliepen [Mon, 14 Sep 2009 22:28:20 +0000 (00:28 +0200)]
Remove dropin random() function, as it is not used anymore.

15 years agoAllow compiling for Windows XP and higher.
Guus Sliepen [Mon, 14 Sep 2009 22:24:31 +0000 (00:24 +0200)]
Allow compiling for Windows XP and higher.

This allows us to use getaddrinfo(), getnameinfo() and related functions, which
allow tinc to make connections over existing IPv6 networks. These functions are
not available on Windows 2000 however. By default, support is enabled, but when
compiling for Windows 2000 the configure switch --with-windows2000 should be
used.

Since getaddrinfo() et al. are not functions but macros on Windows, we have to
use AC_CHECK_DECLS() instead of AC_CHECK_FUNCS() in configure.in.

15 years agoAlso do not use drand48(), it is not available on Windows.
Guus Sliepen [Mon, 14 Sep 2009 21:28:28 +0000 (23:28 +0200)]
Also do not use drand48(), it is not available on Windows.

15 years agoUse only rand(), not random().
Guus Sliepen [Mon, 14 Sep 2009 21:06:00 +0000 (23:06 +0200)]
Use only rand(), not random().

We used both rand() and random() in our code. Since it returns an int, we have
to use %x in our format strings instead of %lx. This fixes a crash under
Windows when cross-compiling tinc with a recent version of MinGW.

15 years agoApparently it's impolite to ask GCC to subtract two pointers.
Guus Sliepen [Sun, 13 Sep 2009 12:08:59 +0000 (14:08 +0200)]
Apparently it's impolite to ask GCC to subtract two pointers.

If two pointers do not belong to the same array, pointer subtraction gives
nonsensical results, depending on the level of optimisation and the
architecture one is compiling for. It is apparently not just subtracting the
pointer values and dividing by the size of the object, but uses some kind of
higher magic not intended for mere mortals. GCC will not warn about this at
all. Casting to void * is also a no-no, because then GCC does warn that strict
aliasing rules are being broken. The only safe way to query the ordering of two
pointers is to use the (in)equality operators.

The unsafe implementation of connection_compare() has probably caused the "old
connection_t for ... still lingering" messages. Our implementation of AVL trees
is augmented with a doubly linked list, which is normally what is traversed.
Only when deleting an old connection the tree itself is traversed.

15 years agoRemove superfluous call to avl_delete().
Guus Sliepen [Sun, 13 Sep 2009 12:07:40 +0000 (14:07 +0200)]
Remove superfluous call to avl_delete().

15 years agoHandle unicast packets larger than PMTU in switch mode.
Guus Sliepen [Sat, 12 Sep 2009 12:19:36 +0000 (14:19 +0200)]
Handle unicast packets larger than PMTU in switch mode.

If PMTUDiscovery is enabled, and we see a unicast packet that is larger than
the path MTU in switch mode, treat it just like we would do in router mode.

15 years agoAllow PMTUDiscovery in switch and hub modes again.
Guus Sliepen [Sat, 12 Sep 2009 11:40:32 +0000 (13:40 +0200)]
Allow PMTUDiscovery in switch and hub modes again.

PMTUDiscovery was disabled in commit d5b56bbba56480b5565ffb38496175a7c1df60ac
because tinc did not handle packets larger than the path MTU in switch and hub
modes. We now allow it again in preparation of proper support, but default to
off.

15 years agoPut Subnet weight in a separate environment variable.
Guus Sliepen [Sat, 12 Sep 2009 11:34:11 +0000 (13:34 +0200)]
Put Subnet weight in a separate environment variable.

Commit 5674bba5c54c1aee3a4ac5b3aba6b3ebded91bbc introduced weighted Subnets,
but the weight was included in the SUBNET variable passed to subnet-up/down
scripts. This makes it harder to use in those scripts. The weight is now
stripped from the SUBNET variable and put in the WEIGHT variabel.

15 years agoDon't stat() on iPhone/iPod.
Guus Sliepen [Thu, 10 Sep 2009 17:51:08 +0000 (19:51 +0200)]
Don't stat() on iPhone/iPod.

Grzegorz Dymarek noted that tinc segfaults at the stat() call in
execute_script() on the iPhone.  We can omit the stat() call for the moment,
the subsequent call to system() will fail with just a warning.

15 years agoAdd support for iPhones and recent iPods.
Guus Sliepen [Thu, 10 Sep 2009 17:32:54 +0000 (19:32 +0200)]
Add support for iPhones and recent iPods.

This is a slightly modified patch from Grzegorz Dymarek that allows tinc to use
the tunemu device, which allows tinc to be compiled for iPhones and recent
iPods. To enable support for tunemu, the --enable-tunemu option has to be used
when running the configure script.

15 years agoAnother safe bitfield conversion.
Guus Sliepen [Wed, 9 Sep 2009 12:51:36 +0000 (14:51 +0200)]
Another safe bitfield conversion.

15 years agoAdd the GPL license to the repository.
Guus Sliepen [Wed, 9 Sep 2009 11:23:16 +0000 (13:23 +0200)]
Add the GPL license to the repository.

Tinc is licensed under the GPL version 2 or later. To ensure autoconf does not
install the wrong license if COPYING is missing, we have to put the right one
in place.

15 years agoConvert bitfields to integers in a safe way.
Guus Sliepen [Wed, 9 Sep 2009 10:04:08 +0000 (12:04 +0200)]
Convert bitfields to integers in a safe way.

This is commit eb391c52eed46f3f03b404553df417851fc0cb90 redone, but without the
non-standard anonymous union.

15 years agoEnsure tinc compiles with gcc -std=c99.
Guus Sliepen [Tue, 8 Sep 2009 19:45:24 +0000 (21:45 +0200)]
Ensure tinc compiles with gcc -std=c99.

We use a lot of C99 features already, but also some extensions which are not in
the standard.

15 years agoUNIX signal numbers start at 1.
Guus Sliepen [Tue, 8 Sep 2009 16:21:52 +0000 (18:21 +0200)]
UNIX signal numbers start at 1.

15 years agoReplace asprintf() by xasprintf().
Guus Sliepen [Tue, 8 Sep 2009 16:18:36 +0000 (18:18 +0200)]
Replace asprintf() by xasprintf().

15 years agoCheck the return value of fscanf() when reading a PID file.
Guus Sliepen [Tue, 8 Sep 2009 16:18:16 +0000 (18:18 +0200)]
Check the return value of fscanf() when reading a PID file.

15 years agoAdd xasprintf() and xvasprintf().
Guus Sliepen [Tue, 8 Sep 2009 16:16:58 +0000 (18:16 +0200)]
Add xasprintf() and xvasprintf().

These functions wrap asprintf() and vasprintf(), and check the return value. If
the function failed, tinc will exit with an error message, similar to xmalloc()
and friends.

15 years agoRemove extra semicolon in my definition of setpriority()
Michael Tokarev [Sat, 5 Sep 2009 13:24:41 +0000 (17:24 +0400)]
Remove extra semicolon in my definition of setpriority()

15 years agoAlways remove a node from the UDP tree before freeing it.
Guus Sliepen [Tue, 8 Sep 2009 14:35:28 +0000 (16:35 +0200)]
Always remove a node from the UDP tree before freeing it.

Valgrind caught tinc reading free'd memory during a purge(). This was caused by
first removing it from the main node tree, which will already call free_node(),
and then removing it from the UDP tree. This might cause spurious segmentation
faults.

15 years agoChange level of some debug messages, zero pointer after freeing hostname.
Guus Sliepen [Thu, 11 Jun 2009 17:39:25 +0000 (19:39 +0200)]
Change level of some debug messages, zero pointer after freeing hostname.

15 years agoDo not log errors when recvfrom() returns EAGAIN or EINTR.
Guus Sliepen [Thu, 11 Jun 2009 17:26:34 +0000 (19:26 +0200)]
Do not log errors when recvfrom() returns EAGAIN or EINTR.

Although we select() before we call recvfrom(), it sometimes happens that
select() tells us we can read but a subsequent read fails anyway. This is
harmless.

15 years agoRemove pending MTU probe events when a node's reachability status changes.
Guus Sliepen [Thu, 11 Jun 2009 17:07:54 +0000 (19:07 +0200)]
Remove pending MTU probe events when a node's reachability status changes.

15 years agoDon't try to send MTU probes to unreachable nodes.
Guus Sliepen [Thu, 11 Jun 2009 16:36:08 +0000 (18:36 +0200)]
Don't try to send MTU probes to unreachable nodes.

If there is an outstanding MTU probe event for a node which is not reachable
anymore, a UDP packet would be sent to that node, which caused a key request to
be sent to that node, which triggered a NULL pointer dereference. Probes and
other UDP packets to unreachable nodes are now dropped.

15 years agoProperly set HMAC length for incoming packets.
Guus Sliepen [Fri, 5 Jun 2009 14:14:31 +0000 (16:14 +0200)]
Properly set HMAC length for incoming packets.

15 years agotry outgoing connections before chroot/drop_privs
Michael Tokarev [Fri, 5 Jun 2009 09:33:58 +0000 (13:33 +0400)]
try outgoing connections before chroot/drop_privs

When chrooted, we either need to force-initialize resolver
and/or nsswitch somehow (no clean way) or resolve all the
names we want before entering chroot jail.  The latter
looks cleaner, easier and it is actually safe because
we still don't talk with the remote nodes there, only
initiating outgoing connections.

15 years agocleanup setpriority thing to make it readable
Michael Tokarev [Fri, 5 Jun 2009 07:58:17 +0000 (11:58 +0400)]
cleanup setpriority thing to make it readable

15 years agoAdd some const where appropriate.
Guus Sliepen [Thu, 28 May 2009 21:18:22 +0000 (23:18 +0200)]
Add some const where appropriate.

15 years agoAdd ProcessPriority option.
Guus Sliepen [Thu, 28 May 2009 20:51:30 +0000 (22:51 +0200)]
Add ProcessPriority option.

This option can be set to low, normal or high. On UNIX flavours, this changes
the nice value of the process by +10, 0 and -10 respectively. On Windows, it
sets the priority to BELOW_NORMAL_PRIORITY_CLASS, NORMAL_PRIORITY_CLASS and
HIGH_PRIORITY_CLASS respectively.

A high priority might help to reduce latency and packet loss on the VPN.

15 years agosrc/net_socket.c: Bind outgoing TCP sockets to `BindToAddress'.
Florian Forster [Wed, 27 May 2009 12:20:24 +0000 (14:20 +0200)]
src/net_socket.c: Bind outgoing TCP sockets to `BindToAddress'.

If a host has multiple addresses on an interface, the source address of the TCP
connection(s) was picked by the operating system while the UDP packets used a
bound socket, i. e. the source address was the address specified by the user.
This caused problems because the receiving code requires the TCP connection and
the UDP connection to originate from the same IP address.

This patch adds support for the `BindToInterface' and `BindToAddress' options
to the setup of outgoing TCP connections.

Tested with Debian Etch on x86 and Debian Lenny on x86_64.

Signed-off-by: Florian Forster <octo@verplant.org>
15 years agosrc/linux/device.c: Fix segfault when running without `--net'.
Florian Forster [Wed, 27 May 2009 07:27:44 +0000 (09:27 +0200)]
src/linux/device.c: Fix segfault when running without `--net'.

If running without `--net', the (global) variable `netname' is NULL. This
creates a segmentation fault because this NULL-pointer is passed to strdup:

 Program terminated with signal 11, Segmentation fault.
 #0  0xb7d30463 in strlen () from /lib/tls/i686/cmov/libc.so.6
 (gdb) bt
 #0  0xb7d30463 in strlen () from /lib/tls/i686/cmov/libc.so.6
 #1  0xb7d30175 in strdup () from /lib/tls/i686/cmov/libc.so.6
 #2  0x0805bf47 in xstrdup (s=0x0) at xmalloc.c:118  <---
 #3  0x0805be33 in setup_device () at device.c:66
 #4  0x0805072e in setup_myself () at net_setup.c:432
 #5  0x08050db2 in setup_network () at net_setup.c:536
 #6  0x0805b27f in main (argc=Cannot access memory at address 0x0) at tincd.c:580

This patch fixes this by checking `netname' in `setup_device'. An alternative
would be to check for NULL-pointers in `xstrdup' and return NULL in this case.

Signed-off-by: Florian Forster <octo@verplant.org>
15 years agotunnelserver: log which ADD_SUBNET was refused
Michael Tokarev [Sun, 24 May 2009 13:23:24 +0000 (17:23 +0400)]
tunnelserver: log which ADD_SUBNET was refused

Add some logging about refused ADD_SUBNET
(it causes subsequent client disconnect so it's
important to know which subnet was at fault).

Maybe we should just ignore it completely.

15 years agoDo not forward broadcast packets when TunnelServer is enabled.
Guus Sliepen [Mon, 25 May 2009 13:04:33 +0000 (15:04 +0200)]
Do not forward broadcast packets when TunnelServer is enabled.

First of all, the idea behind the TunnelServer option is to hide all other
nodes from each other, so we shouldn't forward broadcast packets from them
anyway. The other reason is that since edges from other nodes are ignored, the
calculated minimum spanning tree might not be correct, which can result in
routing loops.

15 years agoUse packet size before decompression to calculate path MTU.
Guus Sliepen [Mon, 25 May 2009 10:19:37 +0000 (12:19 +0200)]
Use packet size before decompression to calculate path MTU.

Since compression can either grow or shrink a packet, the size of an MTU probe
after decompression might not reflect the real path MTU. Now we use the size
before decompression, which is independent of the compression algorithm, and
substract a safety margin such that the calculated path MTU will be safe even
for packets which grow as much as possible after compression.

15 years agoAdd declaration for sockaddrcmp_noport().
Guus Sliepen [Mon, 25 May 2009 10:19:08 +0000 (12:19 +0200)]
Add declaration for sockaddrcmp_noport().

15 years agoFix ans_key exchange in recent changes
Michael Tokarev [Sun, 24 May 2009 18:32:24 +0000 (22:32 +0400)]
Fix ans_key exchange in recent changes

send_ans_key() was using the wrong in vs. outkeylength to
terminate the key being sent, so it was always empty.

15 years agoUse xrealloc instead of if(ptr) ptr = xmalloc().
Guus Sliepen [Sun, 24 May 2009 17:35:51 +0000 (19:35 +0200)]
Use xrealloc instead of if(ptr) ptr = xmalloc().

15 years agoFix initialisation of packet decryption context broken by commit 3308d13e7e3bf20cfeaf...
Guus Sliepen [Sun, 24 May 2009 17:31:31 +0000 (19:31 +0200)]
Fix initialisation of packet decryption context broken by commit 3308d13e7e3bf20cfeaf6f2ab17228a9820cea66.

Instead of a single, global decryption context, each node has its own context.
However, in send_ans_key(), the global context was initialised. This commit
fixes that and removes the global context completely.

Also only set status.validkey after all checks have been evaluated.

15 years agodon't log every strange packet coming to the UDP port
Michael Tokarev [Thu, 21 May 2009 21:10:16 +0000 (01:10 +0400)]
don't log every strange packet coming to the UDP port

it's a sure way to fill up syslog.  Only log those if
debug level is up to PROTOCOL

15 years agoFix link to Mattias Nissler's tun/tap driver for MacOS/X.
Guus Sliepen [Sun, 24 May 2009 15:13:00 +0000 (17:13 +0200)]
Fix link to Mattias Nissler's tun/tap driver for MacOS/X.

Thanks to Martin Christof Kindsmüller for spotting.

15 years agoIf PMTUDiscovery is not set, do not forward packets via TCP unnecessarily.
Guus Sliepen [Sun, 24 May 2009 13:58:47 +0000 (15:58 +0200)]
If PMTUDiscovery is not set, do not forward packets via TCP unnecessarily.

15 years agoignore indirect edge registrations in tunnelserver mode
Michael Tokarev [Thu, 21 May 2009 21:01:35 +0000 (01:01 +0400)]
ignore indirect edge registrations in tunnelserver mode

In tunnelserver mode we're not interested to hear about
our client edges, just like in case of subnets.  Just
ignore all requests which are not about our node or the
client node.

The fix is very similar to what was done for subnets.

Note that we don't need to add the "unknown" nodes to
the list in tunnelserver mode too, so move allocation
of new nodes down the line.

15 years agoTunnelServer: Don't disconnect client on DEL_SUBNET too
Michael Tokarev [Wed, 20 May 2009 14:40:04 +0000 (18:40 +0400)]
TunnelServer: Don't disconnect client on DEL_SUBNET too

Similar changes as was in 2327d3f6eb5982bcc922ff1ab1ec436ba6aeffdc
but for del_subnet_h().

Before, we vere returning false (and causing disconnect of the
client) in case of tunnelserver and the client sending DEL_SUBNET
for non-his subnet or for subnet which owner isn't in our connection
list.

After the mentioned change to add_subnet_h() that routine does not
add such indirect owners to the connection list anymore, so that
was ok (owner == NULL and we return true).

But if we too has a connection with the node about which the client
is sending DEL_SUBNET notification, say, because that client lost
connection with that other node, we'll disconnect this client from
us too, returning false for indirect DEL_SUBNET.

Fix that by allowing and ignoring indirect DEL_SUBNET in tunnelserver
mode.

Also rearranged the function a bit, to match add_subnet_h() (in
particular, syntax-check everything first, see if we've seen this
request before).

And also fix some comments.

15 years agoformat 'not supported on this platform' error message
Michael Tokarev [Mon, 18 May 2009 13:34:30 +0000 (17:34 +0400)]
format 'not supported on this platform' error message

Format it in a similar way in all places, to make translation happier.
No functional changes.

15 years agochange error messages in droppriv code to match the rest
Michael Tokarev [Mon, 18 May 2009 13:00:00 +0000 (17:00 +0400)]
change error messages in droppriv code to match the rest

Change formatting of error messages about failed syscalls
to be the same as in other places in tincd.

Also suggest a change in "$foo not supported on this platform"
message as it's now used more than once.

15 years agobugfix: chdir(/) after chroot
Michael Tokarev [Mon, 18 May 2009 12:53:08 +0000 (16:53 +0400)]
bugfix: chdir(/) after chroot

Fix the famous chdir(".") vs chdir("/") after chroot(something).

15 years agobugfix: move mlock to after detach() so it works for child, not parent
Michael Tokarev [Mon, 18 May 2009 12:49:39 +0000 (16:49 +0400)]
bugfix: move mlock to after detach() so it works for child, not parent

mlock()/mlockall() are not persistent across fork(), and it's
done in parent process before daemon() which does fork().  So
basically, current --mlock does nothing useful.

Move mlock() to after detach() so it works for child process
instead of parent.

Also, check if the platform supports mlock right when processing
options (since else we'll have to die after startup, not at
startup, the error message will be in log only).

15 years agobugfix: initialize pid (as read from pidfile) to zero
Michael Tokarev [Mon, 18 May 2009 12:28:55 +0000 (16:28 +0400)]
bugfix: initialize pid (as read from pidfile) to zero

If we didn't read any number from a pid file, we'll return
an unitialized variable to the caller, and it will treat
that garbage as a pid of a process (possible to kill).

Fix that.

15 years agoImplement privilege dropping
Michael Tokarev [Mon, 18 May 2009 12:25:41 +0000 (16:25 +0400)]
Implement privilege dropping

Add two options, -R/--chroot and -U/--user=user, to chroot to the
config directory (where tinc.conf is located) and to perform
setuid to the user specified, after all the initialization is done.

What's left is handling of pid file since we can't remove it anymore.

15 years agoRename setup_network_connections() and split out try_outgoing_connections()
Michael Tokarev [Mon, 18 May 2009 12:25:10 +0000 (16:25 +0400)]
Rename setup_network_connections() and split out try_outgoing_connections()

In preparation of chroot/setuid operations, split out call to
try_outgoing_connections() from setup_network_connections()
(which was the last call in setup_network_connections()).
This is because dropping privileges should be done in-between
setup_network_connections() and try_outgoing_connections().

This patch renames setup_network_connections() to setup_network()
and moves call to try_outgoing_connections() into main routine.

No functional changes.

15 years agoHandle UDP packets from different and ports than advertised.
Guus Sliepen [Thu, 2 Apr 2009 23:05:23 +0000 (01:05 +0200)]
Handle UDP packets from different and ports than advertised.

Previously, tinc used a fixed address and port for each node for UDP packet
exchange.  The port was the one advertised by that node as its listening port.
However, due to NAT the port might be different.  Now, tinc sends a different
session key to each node. This way, the sending node can be determined from
incoming packets by checking the MAC against all session keys. If a match is
found, the address and port for that node are updated.

15 years agoUse a simple Random Early Drop algorithm in send_tcppacket().
Guus Sliepen [Mon, 9 Mar 2009 13:04:31 +0000 (14:04 +0100)]
Use a simple Random Early Drop algorithm in send_tcppacket().

15 years agoDisable PMTUDiscovery in switch and hub modes.
Guus Sliepen [Mon, 9 Mar 2009 12:48:54 +0000 (13:48 +0100)]
Disable PMTUDiscovery in switch and hub modes.

In switch and hub modes, tinc does not generate ICMP packets in response to
packets that are larger than the path MTU.  However, if PMTUDiscovery is
enabled, the IP_MTU_DISCOVER and IPV6_MTU_DISCOVER option is set on the UDP
sockets, which causes all UDP packets to be sent with the DF bit set, causing
large packets to be dropped, even if they would otherwise be routed fine.

15 years agoUpdate THANKS and copyright information.
Guus Sliepen [Thu, 5 Mar 2009 13:12:36 +0000 (14:12 +0100)]
Update THANKS and copyright information.

15 years agoAllow weight to be assigned to Subnets.
Guus Sliepen [Thu, 5 Mar 2009 12:34:13 +0000 (13:34 +0100)]
Allow weight to be assigned to Subnets.

Tinc allows multiple nodes to own the same Subnet, but did not have a sensible
way to decide which one to send packets to. Tinc also did not check the
reachability of nodes when deciding where to route packets to, so it would not
automatically fail over to a reachable node.

Tinc now assigns a weight to each Subnet. The default weight is 10, with lower
weights having higher priority.  The Subnets are now internally sorted in the
same way as the kernel's routing table, and the Subnets are search linearly,
skipping those of unreachable nodes. A small cache of recently used addresses
is used to speed up the lookup functions.

15 years agoEnable PMTUDiscovery only if BOTH sides wants it.
Michael Tokarev [Sat, 28 Feb 2009 13:37:51 +0000 (16:37 +0300)]
Enable PMTUDiscovery only if BOTH sides wants it.

Don't enable PMTUDiscovery if at least one side does not support it.
Before it was enabled if at least one side supported it, now both are required.

15 years agoHandle neighbor solicitation requests without link layer addresses.
Guus Sliepen [Tue, 17 Feb 2009 13:43:05 +0000 (14:43 +0100)]
Handle neighbor solicitation requests without link layer addresses.

Apparently FreeBSD likes to send out neighbor solicitation requests, even on a
tun interface where this is completely pointless. These requests do not have an
option header containing a link layer address, so the proxy-neighborsol code
was treating these requests as invalid. We now handle such requests, and send
back equally pointless replies, also without a link layer address. This seems
to satisfy FreeBSD.

15 years agoAllow tunnelserver to work with clients that have other peers.
Michael Tokarev [Mon, 9 Feb 2009 22:51:10 +0000 (23:51 +0100)]
Allow tunnelserver to work with clients that have other peers.

In TunnelServer mode, tinc server disconnects any client if it announces
indirect subnets -- subnets that are not theirs (e.g. subnets for nodes
the CLIENT has connections now, even if those nodes are known to the server
too).  Fix that by ignoring such (indirect) announces instead.

While we're at it, move check for such indirect subnet registration to
before allocating new node structure, as in TunnelServer mode we don't
really need to know that other node.

15 years agoDisable old RSA keys when generating new ones.
Guus Sliepen [Tue, 3 Feb 2009 13:54:45 +0000 (14:54 +0100)]
Disable old RSA keys when generating new ones.

When generating an RSA keypair, the new public and private keys are appended to
files. However, when OpenSSL reads keys it only reads the first in a file, not
the last. Instead of printing an easily ignored warning, tinc now disables old
keys when appending new ones.

15 years agoValidate Name before using it in a filename when generating a keypair.
Guus Sliepen [Tue, 20 Jan 2009 13:21:50 +0000 (14:21 +0100)]
Validate Name before using it in a filename when generating a keypair.

15 years agoAllow reading config files with CRLF endings on Unix systems.
Guus Sliepen [Tue, 20 Jan 2009 13:20:44 +0000 (14:20 +0100)]
Allow reading config files with CRLF endings on Unix systems.

15 years agoRemove unused definitions from net.h.
Guus Sliepen [Tue, 20 Jan 2009 12:19:31 +0000 (13:19 +0100)]
Remove unused definitions from net.h.

15 years agoUse a global list to track outgoing connections.
Guus Sliepen [Tue, 20 Jan 2009 12:12:41 +0000 (13:12 +0100)]
Use a global list to track outgoing connections.

Previously an outgoing_t was maintained for each outgoing connection,
but the pointer to it was either stored in a connection_t or in an event_t.
This made it very hard to keep track of and to clean up.

Now a list is created when tinc starts and reads all the ConnectTo variables,
and which is recreated when tinc receives a HUP signal.

15 years agoAdd missing cleanup functions in close_network_connections().
Guus Sliepen [Mon, 19 Jan 2009 22:17:28 +0000 (23:17 +0100)]
Add missing cleanup functions in close_network_connections().

15 years agoChange flush_events() to expire_events().
Guus Sliepen [Mon, 19 Jan 2009 21:50:05 +0000 (22:50 +0100)]
Change flush_events() to expire_events().

The former function made a totally bogus shallow copy of the event_tree, called
the handler of each event and then deleted the whole tree.  This should've
caused tinc to crash when an ALARM signal was sent more than once, but for some
reason it didn't. It also behaved incorrectly when a handler added a new event.

The new function just moves the expiration time of all events to the past.

15 years agoMove free()s at the end om main() to the proper destructor functions.
Guus Sliepen [Fri, 9 Jan 2009 11:36:06 +0000 (12:36 +0100)]
Move free()s at the end om main() to the proper destructor functions.

15 years agoOnly send packets via UDP if UDP communication is possible.
Guus Sliepen [Sat, 3 Jan 2009 21:33:55 +0000 (22:33 +0100)]
Only send packets via UDP if UDP communication is possible.

When no session key is known for a node, or when it is doing PMTU discovery but
no MTU probes have returned yet, packets are sent via TCP. Some logic is added
to make sure intermediate nodes continue forwarding via TCP.  The per-node
packet queue is now no longer necessary and has been removed.

15 years agoConsistently allocate device and iface variables on the heap.
Guus Sliepen [Sat, 3 Jan 2009 21:06:10 +0000 (22:06 +0100)]
Consistently allocate device and iface variables on the heap.

This fixes a segfault when no Device has been specified and tinc exits, and it
would try to free() a static string. Thanks to Borg for spottin.

15 years agoUpdate documentation for git.
Guus Sliepen [Sat, 27 Dec 2008 10:09:43 +0000 (11:09 +0100)]
Update documentation for git.

15 years agoReleasing 1.0.9. release-1.0.9
Guus Sliepen [Fri, 26 Dec 2008 13:47:34 +0000 (13:47 +0000)]
Releasing 1.0.9.

15 years agoAdd missing parentheses in check for IPv4 multicast addresses.
Guus Sliepen [Fri, 26 Dec 2008 12:46:45 +0000 (12:46 +0000)]
Add missing parentheses in check for IPv4 multicast addresses.

15 years agoApply patch from Max Rijevski fixing a memory leak when closing connections.
Guus Sliepen [Tue, 23 Dec 2008 23:14:37 +0000 (23:14 +0000)]
Apply patch from Max Rijevski fixing a memory leak when closing connections.
It also cleans up more when stopping tinc, helping tools like valgrind.

15 years agoHandle broadcast and multicast packets in router mode.
Guus Sliepen [Tue, 23 Dec 2008 22:31:38 +0000 (22:31 +0000)]
Handle broadcast and multicast packets in router mode.
Multicast packets are treated as broadcast packets.
Based on a patch from Max Rijevski.

15 years agoUpdate the manpage as well, and some whitespace to make its source more legible.
Guus Sliepen [Mon, 22 Dec 2008 21:49:23 +0000 (21:49 +0000)]
Update the manpage as well, and some whitespace to make its source more legible.

15 years agoUpdate documentation.
Guus Sliepen [Mon, 22 Dec 2008 21:29:21 +0000 (21:29 +0000)]
Update documentation.
- TCPOnly is not experimental.
- Do not mention old Linux kernels and Ethertap anymore.
- Document the DeviceType, PMTU and PMTUDiscovery options.

15 years agoEnable PMTU discovery by default.
Guus Sliepen [Mon, 22 Dec 2008 20:35:45 +0000 (20:35 +0000)]
Enable PMTU discovery by default.

15 years agoUpdate copyright information.
Guus Sliepen [Mon, 22 Dec 2008 20:27:52 +0000 (20:27 +0000)]
Update copyright information.

15 years agoUpdate Dutch translation.
Guus Sliepen [Mon, 22 Dec 2008 19:43:49 +0000 (19:43 +0000)]
Update Dutch translation.

15 years agoMake sure IPv6 sockets are IPv6 only.
Guus Sliepen [Mon, 22 Dec 2008 19:40:40 +0000 (19:40 +0000)]
Make sure IPv6 sockets are IPv6 only.
This will get rid of the "Can't bind to 0.0.0.0 port 655/tcp: Address already
in use" message on Linux.