wintun: poll for device key

It's actually pretty hard to guess where it is.

wintun: fix scope of error object

wintun: wait for interface registry key on device creation

By using RegNotifyChangeKeyValue(). Also disable dead gateway detection.

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

conn: remove scope when sanity checking IP address format

wintun: fix GUID leading zero padding

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

mod: update deps

setupapi: safer aliasing of slice types

wintun: work around GetInterface staleness bug

wintun: don't retry when not creating

The only time we're trying to counteract the race condition is when
we're creating a driver. When we're simply looking up all drivers, it
doesn't make sense to retry.

wintun: try harder to open registry key

This sucks. Can we please find a deterministic way of doing this
instead?

go.mod: use vendored winio

tun: freebsd: work around numerous kernel panics on shutdown

There are numerous race conditions. But even this will crash it:

while true; do ifconfig tun0 create; ifconfig tun0 destroy; done

It seems like LLv6 is related, which we're not using anyway, so
explicitly disable it on the interface.

setupapi: Fix struct size mismatches

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

setupapi: actually fix padding by rounding up to sizeof(void*)

tun: darwin: write routeSocket variable in helper

Otherwise the race detector "complains".

main: revise warnings

device: send: check packet length before freeing element

setupapi: Revise DrvInfoDetailData struct size calculation

Go adds trailing padding to DrvInfoDetailData struct in GOARCH=386 which
confuses SetupAPI expecting exactly sizeof(SP_DRVINFO_DETAIL_DATA).

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

windows: use proper constants from updated x/sys

conn: linux: RTA_MARK has moved to x/sys

tun: windows: Adopt new error codes returned by Wintun

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

tun: windows: do not sleep after OPERATION_ABORTED on write

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

main_windows: use proper version constant

version: put version in right place

version: bump snapshot

tun: windows: do not sleep after OPERATION_ABORTED

tun: windows: Retry R/W on ERROR_OPERATION_ABORTED

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

tun: windows: Attempt to reopen handle on all errors

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

wintun: rename device using undocumented API that netsh.exe uses

wintun: add more retry loops

tun: windows: cancel ongoing reads on closing and delete after close

This reverts commit 52ec440d7977fad966002c3710ed9df957943407 and adds
some spice.

wintun: query for NetCfgInstanceId several times

tun: windows: delete interface before deleting file handles

tun: windows: Make adapter rename asynchronous

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

tun: windows: Adapter devices renamed to WINTUN<LUID Index>

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

tun: windows: Increase unavailable adapter timeout to 30sec

5 seconds was too short when debugging.

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

tun: windows: Make writing persistent too

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

tun: windows: Fix paused adapter test

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

tun: windows: just open two file handles

receive: implement flush semantics

tun: windows: add dummy overlapped events back

These seem basically wrong to me, but we get crashes without them.

tun: windows: use new constants in sys

uapi: remove unhelpful log messages

wintun: Use native Win32 API for I/O

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

uapi: report endpoint error

tun: windows: wintun does iocp

tun: windows: temporary hack for forcing MTU

wintun: Poll more often

uapi: make ipcerror conform to interface

wintun: Make errors more descriptive

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

wintun: Return correct reboot-req flag on CreateInterface() error too

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

wintun: Fix double-quoted strings escaping on output

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

wintun: Introduce SetupAPI enumerator and machineName consts

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

Windows: Apply strict security descriptor on pipe server

Signed-off-by: Odd Stranne <odd@mullvad.net>

uapi: windows: work out pipe semantics

Pipes can be arranged like this, so that's fine. We also apply a strict
SDDL that can't be inherited and only gives access to local system.

Developed-with: Odd Stranne <odd@mullvad.net>

wintun: Cleanup

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

wintun: Refactor network registry key name generation

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

wintun: Revise interface creation wait

DIF_INSTALLDEVICE returns almost immediately, while the device
installation continues in the background. It might take a while, before
all registry keys and values are populated.

Previously, wireguard-go waited for HKLM\SYSTEM\CurrentControlSet\
Control\Class\{4D36E972-E325-11CE-BFC1-08002BE10318}\<id> registry key
only.

Followed by a SetInterfaceName() method of Wintun struct which tried to
access HKLM\SYSTEM\CurrentControlSet\Control\Network\
{4D36E972-E325-11CE-BFC1-08002BE10318}\<id>\Connection registry key
might not be available yet.

This commit loops until both registry keys are available before
returning from CreateInterface() function.

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

receive: squelch tear down error

tun: linux: work out netpoll trick

wintun: Resolve some of golint warnings

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

Elaborate the failing step when forwarding errors on return

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

Simplify reading NetCfgInstanceId from registry

As querying non-existing registry value and reading non-existing
registry string value both return ERROR_FILE_NOT_FOUND, we can
use later only.

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

Unify interface-specific network registry key open

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

tun: import mobile particularities

boundif: introduce API for socket binding

global: begin modularization

tun: windows: expose GUID

tun: allow special methods in NativeTun

tun: linux: netpoll is broken for tun's epoll

So this mostly reverts the switch to Sysconn for Linux.

Issue: https://github.com/golang/go/issues/30426

tun: linux: netlink sock needs cleaning up but file will be gc'd

tun: use netpoll instead of rwcancel

The new sysconn function of Go 1.12 makes this possible:

package main

import "log"
import "os"
import "unsafe"
import "time"
import "syscall"
import "sync"
import "golang.org/x/sys/unix"

func main() {
fd, err := os.OpenFile("/dev/net/tun", os.O_RDWR, 0)
if err != nil {
log.Fatal(err)
}

var ifr [unix.IFNAMSIZ + 64]byte
copy(ifr[:], []byte("cheese"))
*(*uint16)(unsafe.Pointer(&ifr[unix.IFNAMSIZ])) = unix.IFF_TUN

var errno syscall.Errno
s, _ := fd.SyscallConn()
s.Control(func(fd uintptr) {
_, _, errno = unix.Syscall(
unix.SYS_IOCTL,
fd,
uintptr(unix.TUNSETIFF),
uintptr(unsafe.Pointer(&ifr[0])),
)
})
if errno != 0 {
log.Fatal(errno)
}

b := [4]byte{}
wait := sync.WaitGroup{}
wait.Add(1)
go func() {
_, err := fd.Read(b[:])
log.Print("Read errored: ", err)
wait.Done()
}()
time.Sleep(time.Second)
log.Print("Closing")
err = fd.Close()
if err != nil {
log.Print("Close errored: " , err)
}
wait.Wait()
log.Print("Exiting")
}

tun: use sysconn instead of .Fd with Go 1.12

Rearrange imports

device: send persistent keepalive when bringing up device

Reported-by: Marcelo Bello

wintun: Read/write packet size from/to exchange buffer directly

Driver <-> user-space communication is local and using native endian.

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

wintun: Make two-step slicing a one step

Stop relying to Go compiler optimizations and calculate the end offset
directly.

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

wintun: Write exchange buffer increased back to 1MiB

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

wintun: Migrate from unsafe buffer handling to encoding/binary

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

wintun: Adopt new packet data alignment

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

wintun: Cleanup

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

wintun: Add TUN device locking

In case reading from TUN device detected TUN device was closed, it
closed the file handle and set tunFile to nil. The tunFile is
automatically reopened on retry, but... If another packet comes in the
WireGuard calls Write() method. With tunFile set to nil, this will
cause access violation.

Therefore, locking was introduced.

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

wintun: Move exchange buffer in separate struct on heap

This allows buffer alignment and keeps it together with its meta-data.

Furthermore, the write buffer has been reduced - as long as we flush
after _every_ write, we don't need a 1MiB write buffer.

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

wintun: Switch to dynamic packet sizes

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

Change package path

Bump dependencies for ARM ChaCha20

wintun: Auto-calculate TUN exchange buffer size

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

wintun: Simplify Read method()

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

wintun: Make constants private and adopt Go recommended case

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

wintun: Check for user close in read loop regardless the load

Do the WaitForSingleObject() always to provide high-load responsiveness.

Reorder events so TUN_SIGNAL_CLOSE has priority over
TUN_SIGNAL_DATA_AVAIL, to provide high-load responsiveness at all.

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

wintun: Adjust tunRWQueue.left member to match Wintun driver

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

setupapi: Merge _SP_DRVINFO_DETAIL_DATA and DrvInfoDetailData

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

setupapi: Merge SP_DRVINFO_DATA and DrvInfoData

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

setupapi: Rename SP_REMOVEDEVICE_PARAMS to RemoveDeviceParams

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

setupapi: Rename SP_CLASSINSTALL_HEADER to ClassInstallHeader

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

setupapi: Merge _SP_DEVINSTALL_PARAMS and DevInstallParams

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

setupapi: Merge _SP_DEVINFO_LIST_DETAIL_DATA and DevInfoListDetailData

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

setupapi: Rename SP_DEVINFO_DATA to DevInfoData

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

wintun: Detect if a foreign interface with the same name exists

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

wintun: Clean excessive setupapi.DevInfo.GetDeviceInfoListDetail() call

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