uapi: allow unsetting device private key with /dev/null

version: bump snapshot

tun: darwin: ignore ENOMEM errors

Coauthored-by: Andrej Mihajlov <and@mullvad.net>

tun: windows: serialize write calls

README: update repo urls

device: SendmsgN mutates the input sockaddr

So we take a new granular lock to prevent concurrent writes from
racing.

WARNING: DATA RACE
Write at 0x00c0011f2740 by goroutine 27:
  golang.org/x/sys/unix.(*SockaddrInet4).sockaddr()
      /go/pkg/mod/golang.org/x/sys@v0.0.0-20191105231009-c1f44814a5cd/unix/syscall_linux.go:384
+0x114
  golang.org/x/sys/unix.SendmsgN()
      /go/pkg/mod/golang.org/x/sys@v0.0.0-20191105231009-c1f44814a5cd/unix/syscall_linux.go:1304
+0x288
  golang.zx2c4.com/wireguard/device.send4()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/conn_linux.go:485
+0x11f
  golang.zx2c4.com/wireguard/device.(*nativeBind).Send()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/conn_linux.go:268
+0x1d6
  golang.zx2c4.com/wireguard/device.(*Peer).SendBuffer()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/peer.go:151
+0x285
  golang.zx2c4.com/wireguard/device.(*Peer).SendHandshakeInitiation()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/send.go:163
+0x692
  golang.zx2c4.com/wireguard/device.(*Device).RoutineReadFromTUN()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/send.go:318
+0x4b8

Previous write at 0x00c0011f2740 by goroutine 386:
  golang.org/x/sys/unix.(*SockaddrInet4).sockaddr()
      /go/pkg/mod/golang.org/x/sys@v0.0.0-20191105231009-c1f44814a5cd/unix/syscall_linux.go:384
+0x114
  golang.org/x/sys/unix.SendmsgN()
      /go/pkg/mod/golang.org/x/sys@v0.0.0-20191105231009-c1f44814a5cd/unix/syscall_linux.go:1304
+0x288
  golang.zx2c4.com/wireguard/device.send4()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/conn_linux.go:485
+0x11f
  golang.zx2c4.com/wireguard/device.(*nativeBind).Send()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/conn_linux.go:268
+0x1d6
  golang.zx2c4.com/wireguard/device.(*Peer).SendBuffer()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/peer.go:151
+0x285
  golang.zx2c4.com/wireguard/device.(*Peer).SendHandshakeInitiation()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/send.go:163
+0x692
  golang.zx2c4.com/wireguard/device.expiredRetransmitHandshake()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/timers.go:110
+0x40c
  golang.zx2c4.com/wireguard/device.(*Peer).NewTimer.func1()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/timers.go:42
+0xd8

Goroutine 27 (running) created at:
  golang.zx2c4.com/wireguard/device.NewDevice()
      /go/pkg/mod/golang.zx2c4.com/wireguard@v0.0.20191012/device/device.go:322
+0x5e8
  main.main()
      /go/src/x/main.go:102 +0x58e

Goroutine 386 (finished) created at:
  time.goFunc()
      /usr/local/go/src/time/sleep.go:168 +0x51

Reported-by: Ben Burkert <ben@benburkert.com>

wintun: manage ring memory manually

It's large and Go's garbage collector doesn't deal with it especially
well.

constants: recalculate rekey max based on a one minute flood

Discussed-with: Mathias Hall-Andersen <mathias@hall-andersen.dk>

global: fix a few typos courtesy of codespell

Signed-off-by: Jonathan Tooker <jonathan.tooker@netprotect.com>

device: allow blackholing sockets

device: remove dead error reporting code

namespaceapi: remove tasteless comment

device: recheck counters while holding write lock

wintun: normalize variable names for their types

wintun: quickly ignore non-Wintun devices

Some devices take ~2 seconds to enumerate on Windows if we try to get
their instance name.  The hardware id property, on the other hand,
is available right away.

Signed-off-by: Avery Pennarun <apenwarr@gmail.com>
[zx2c4: inlined this to where it makes sense, reused setupapi const]

tun: match windows CreateTUN signature to the Linux variant

Signed-off-by: Avery Pennarun <apenwarr@gmail.com>
[zx2c4: fix default value]

rwcancel: handle EINTR and EAGAIN in unixSelect()

On my Chromebook (Linux 4.19.44 in a VM) and on an AWS EC2
machine, select() was sometimes returning EINTR. This is
harmless and just means you should try again. So let's try
again.

This eliminates a problem where the tunnel fails to come up
correctly and the program needs to be restarted.

Signed-off-by: Avery Pennarun <apenwarr@gmail.com>

device: test packets between two fake devices

Signed-off-by: David Crawshaw <crawshaw@tailscale.io>

version: bump snapshot

Makefile: remove v prefix

wintun: expose version

uapi: allow preventing creation of new peers when updating

This enables race-free updates for wg-dynamic and similar tools.

Suggested-by: Thomas Gschwantner <tharre3@gmail.com>

mod: bump versions

winpipe: use x/sys/windows instead of syscall

wintun: use correct length for security attributes

tun: windows: unify error message format

version: bump snapshot

main: simplify warnings

tun: openbsd: check for interface already being up

In some cases, we operate on an already-up interface, or the user brings
up the interface before we start monitoring. For those situations, we
should first check if the interface is already up.

This still technically races between the initial check and the start of
the route loop, but fixing that is a bit ugly and probably not worth it
at the moment.

Reported-by: Theo Buehler <tb@theobuehler.org>

namespaceapi: AddSIDToBoundaryDescriptor modifies the handle

wintun: take mutex first always

This prevents an ABA deadlock with setupapi's internal locks.

wintun: consider abandoned mutexes as released

ipc: windows: use protected prefix

winpipe: enforce ownership of client connection

wintun: put mutex into private namespace

namespaceapi: fix mistake

namespaceapi: initial version

Signed-off-by: Simon Rozman <simon@rozman.si>

wintun: take mutex so that deletion uses the right name

wintun: move ring constants into module

wintun: delete all interfaces is not used anymore

wintun: Wintun->Interface

wintun: keep reference to pool in wintun object

wintun: introduce adapter pools

This makes wintun package reusable for non-WireGuard applications.

Signed-off-by: Simon Rozman <simon@rozman.si>

wintun: simplify rename logic

wintun: give better errors when ndis interface listing fails

wintun: also check for numbered suffix and friendly name

wintun: upgrade deleting all interfaces and make it reusable

DeleteAllInterfaces() didn't check if SPDRP_DEVICEDESC == "WireGuard
Tunnel". It deleted _all_ Wintun adapters, not just WireGuard's.

Furthermore, the DeleteAllInterfaces() was upgraded into a new function
called DeleteMatchingInterfaces() for selectively deletion. This will
be used by WireGuard to clean stale Wintun adapters.

Signed-off-by: Simon Rozman <simon@rozman.si>

wintun: cleanup earlier

wintun: rename duplicate adapters instead of ourselves

wintun: match suffix numbers

device: getsockname on linux to determine port

It turns out Go isn't passing the pointer properly so we wound up with a
zero port every time.

wintun: make description consistent across fields

wintun: try multiple names until one isn't a duplicate

wintun: use nci.dll directly instead of buggy netshell

wintun: set friendly a bit better

This is still wrong, but NETSETUPPKEY_Driver_FriendlyName seems a bit
tricky to use.

wintun: also set friendly name after setting interface name

wintun: defer requires unique variable

wintun: set adapter description name

tun: windows: don't spin unless we really need it

version: bump snapshot

device: drop lock before expiring keys

uapi: skip peers with invalid keys

tun: windows: spin for only a millisecond/80

Performance stays the same as before.

wintun: merge opening device registry key

This also introduces waiting for key to appear on initial access.

See if this resolves the issue caused by HDD power-up delay resulting in
failure to create the adapter.

Signed-off-by: Simon Rozman <simon@rozman.si>

wintun: simplify checking reboot requirement

We never checked checkReboot() reported error anyway.

Signed-off-by: Simon Rozman <simon@rozman.si>

wintun: refactor `err == nil` error checking

Signed-off-by: Simon Rozman <simon@rozman.si>

wintun: handle error for deadgwdetect

tun: darwin: do not attempt to close tun.event twice

Previously it was possible for this to race. It turns out we really
don't need to set anything to -1 anyway.

wintun: get interface path properly with cfgmgr

tun: windows: style

wintun: simplify resolution of dev node

wintun: enable sharing of pnp node

tun: windows: close event handle on shutdown

ipc: windows: match SDDL of WDK and make monkeyable

tun: windows: get rid of retry logic

Things work fine on Windows 8.

tun: windows: use specific IOCTL code

device: do not crash on nil'd bind in windows binding

tun: windows: open file at startup time

tun: windows: silently drop packet when ring is full

tun: windows: switch to NDIS device object

wintun: calculate path of NDIS device object symbolic link

tun: openbsd: don't change MTU when it's already the expected size

Allows for running wireguard-go as non-root user.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

tun: windows: spin for a bit before falling back to event object

tun: windows: implement ring buffers

Signed-off-by: Simon Rozman <simon@rozman.si>

device: immediately rekey all peers after changing device private key

Reported-by: Derrick Pallas <derrick@pallas.us>

README: update windows info

tun: windows: registration of write buffer no longer required

tun: windows: decrease alignment to 4

tun: windows: delay initial write

Otherwise we provoke Wintun 0.3.

device: receive: uniform message for source address check

tun: windows: packetNum is unused

device: receive: simplify flush loop

tun: windows: inform wintun of maximum buffer length for writes

tun: windows: never retry open on Windows 10

tun: remove TUN prefix from types to reduce stutter elsewhere

Signed-off-by: Matt Layher <mdlayher@gmail.com>

device: update transfer counters correctly

The rule is to always update them to the full packet size minus UDP/IP
encapsulation for all authenticated packet types.

wintun: increase registry timeout

wintun: add helper for cleaning up

wintun: simplify error matching and remove dumb comments

wintun: fix comments and remove hwnd param

This now looks more idiomatic.