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.

setupapi: add SetDeviceRegistryPropertyString description

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

setupapi: unify ERROR_INSUFFICIENT_BUFFER handling

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

wintun: allow controlling GUID

setupapi: add DeviceInstanceID()

global: fixup TODO comment spacing

wintun: guid functions are upstream

wintun: simplify DeleteInterface method signature

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

wintun: don't run HrRenameConnection in separate thread

It's very slow, but unfortunately we haven't a choice. NLA needs this to
have completed.

tun: windows: obsolete 256 packets per exchange buffer limitation

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

setupapi, wintun: replace syscall with golang.org/x/sys/windows

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

wintun: set DI_QUIETINSTALL flag for GUI-less device management

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

setupapi: define PropChangeParams struct

This structure is required for calling DIF_PROPERTYCHANGE installer
class.

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

device: remove redundant return statements

More staticcheck fixes:

$ staticcheck ./... | grep S1023
device/noise-helpers.go:45:2: redundant return statement (S1023)
device/noise-helpers.go:54:2: redundant return statement (S1023)
device/noise-helpers.go:64:2: redundant return statement (S1023)

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

device: use bytes.Equal for equality check, simplify assertEqual

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

device, ratelimiter: replace uses of time.Now().Sub() with time.Since()

Simplification found by staticcheck:

$ staticcheck ./... | grep S1012
device/cookie.go:90:5: should use time.Since instead of time.Now().Sub (S1012)
device/cookie.go:127:5: should use time.Since instead of time.Now().Sub (S1012)
device/cookie.go:242:5: should use time.Since instead of time.Now().Sub (S1012)
device/noise-protocol.go:304:13: should use time.Since instead of time.Now().Sub (S1012)
device/receive.go:82:46: should use time.Since instead of time.Now().Sub (S1012)
device/send.go:132:5: should use time.Since instead of time.Now().Sub (S1012)
device/send.go:139:5: should use time.Since instead of time.Now().Sub (S1012)
device/send.go:235:59: should use time.Since instead of time.Now().Sub (S1012)
device/send.go:393:9: should use time.Since instead of time.Now().Sub (S1012)
ratelimiter/ratelimiter.go:79:10: should use time.Since instead of time.Now().Sub (S1012)
ratelimiter/ratelimiter.go:87:10: should use time.Since instead of time.Now().Sub (S1012)

Change applied using:

$ find . -type f -name "*.go" -exec sed -i "s/Now().Sub(/Since(/g" {} \;

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

ipc/winpipe: go fmt

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

tun: windows: mitigate infinite loop in Flush()

It's possible that for whatever reason, we keep returning EOF, resulting
in repeated close/open/write operations, except with empty packets.

device: add SendKeepalivesToPeersWithCurrentKeypair for handover

tai64n: account for whitening in test

device, tun: rearrange code and fix device tests

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

wintun: remove extra /

device: darwin actually doesn't need bound interfaces

device: make initiations per second match kernel implementation

device: timers: add jitter on ack failure reinitiation

wintun: revise GetInterface()

- Make foreign interface found error numeric to ease condition
  detection.
- Update GetInterface() documentation.
- Make tun.CreateTUN() quit when foreign interface found before
  attempting to create a Wintun interface with a duplicate name.
  Creation is futile.

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

wintun: never return nil, nil

ipc: use simplified fork of winio

setupapi: trim "Get" from getters

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

wintun: don't try to flush interface, but rather delete

device: fail to give bind if it doesn't exist

wintun: make certain methods private

version: bump snapshot

makefile: do not show warning on non-linux

wintun: change acronyms to uppercase

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

global: regroup all imports

wintun: registry: fix nits

wintun: registry: revise value reading

- Make getStringValueRetry() reusable for reading any value type. This
  merges code from GetIntegerValueWait().
- expandString() >> toString() and extend to support REG_MULTI_SZ
  (to return first value of REG_MULTI_SZ). Furthermore, doing our own
  UTF-16 to UTF-8 conversion works around a bug in windows/registry's
  GetStringValue() non-zero terminated string handling.
- Provide toInteger() analogous to toString()
- GetStringValueWait() tolerates and reads REG_MULTI_SZ too now. It
  returns REG_MULTI_SZ[0], making GetFirstStringValueWait() redundant.

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

wintun: call HrRenameConnection in another thread

wintun: add LUID accessor

wintun: enumerate faster by using COMPATDRIVER instead of CLASSDRIVER

wintun: destroy devinfolist after usage

wintun: registry: replace REG_NOTIFY with NOTIFY

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

wintun: IpConfig is a MULTI_SZ, and fix errors

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.