From 2a7e23738eb6f5a3acd4cad0bcfe7a471fe02ad3 Mon Sep 17 00:00:00 2001
From: "Jason A. Donenfeld" <Jason@zx2c4.com>
Date: Thu, 29 Aug 2019 12:30:00 -0600
Subject: manager: adjust for wintun api changes

Also get rid of the "Local Area Connection" hack and "/wintun
/deleteall".

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
---
 manager/adaptercleanup.go   | 63 ---------------------------------------------
 manager/interfacecleanup.go | 58 +++++++++++++++++++++++++++++++++++++++++
 2 files changed, 58 insertions(+), 63 deletions(-)
 delete mode 100644 manager/adaptercleanup.go
 create mode 100644 manager/interfacecleanup.go

(limited to 'manager')

diff --git a/manager/adaptercleanup.go b/manager/adaptercleanup.go
deleted file mode 100644
index 779e4b80..00000000
--- a/manager/adaptercleanup.go
+++ /dev/null
@@ -1,63 +0,0 @@
-/* SPDX-License-Identifier: MIT
- *
- * Copyright (C) 2019 WireGuard LLC. All Rights Reserved.
- */
-
-package manager
-
-import (
-	"log"
-	"strings"
-
-	"golang.org/x/sys/windows"
-	"golang.org/x/sys/windows/svc"
-	"golang.org/x/sys/windows/svc/mgr"
-
-	"golang.zx2c4.com/wireguard/tun/wintun"
-	"golang.zx2c4.com/wireguard/windows/services"
-)
-
-const unnamedWintunInterface = "Local Area Connection"
-
-func cleanupStaleAdapters() {
-	defer printPanic()
-
-	m, err := mgr.Connect()
-	if err != nil {
-		return
-	}
-	defer m.Disconnect()
-
-	wintun.DeleteMatchingInterfaces(func(wintun *wintun.Wintun) bool {
-		interfaceName, err := wintun.InterfaceName()
-		if err != nil {
-			log.Printf("Removing Wintun interface %s because determining interface name failed: %v", wintun.GUID().String(), err)
-			return true
-		}
-		if strings.HasPrefix(interfaceName, unnamedWintunInterface) {
-			return false
-		}
-		serviceName, err := services.ServiceNameOfTunnel(interfaceName)
-		if err != nil {
-			log.Printf("Removing Wintun interface ‘%s’ because determining tunnel service name failed: %v", interfaceName, err)
-			return true
-		}
-		service, err := m.OpenService(serviceName)
-		if err == windows.ERROR_SERVICE_DOES_NOT_EXIST {
-			log.Printf("Removing Wintun interface ‘%s’ because no service for it exists", interfaceName)
-			return true
-		} else if err != nil {
-			return false
-		}
-		defer service.Close()
-		status, err := service.Query()
-		if err != nil {
-			return false
-		}
-		if status.State == svc.Stopped {
-			log.Printf("Removing Wintun interface ‘%s’ because its service is stopped", interfaceName)
-			return true
-		}
-		return false
-	})
-}
diff --git a/manager/interfacecleanup.go b/manager/interfacecleanup.go
new file mode 100644
index 00000000..7b13b10c
--- /dev/null
+++ b/manager/interfacecleanup.go
@@ -0,0 +1,58 @@
+/* SPDX-License-Identifier: MIT
+ *
+ * Copyright (C) 2019 WireGuard LLC. All Rights Reserved.
+ */
+
+package manager
+
+import (
+	"log"
+
+	"golang.org/x/sys/windows"
+	"golang.org/x/sys/windows/svc"
+	"golang.org/x/sys/windows/svc/mgr"
+	"golang.zx2c4.com/wireguard/tun/wintun"
+
+	"golang.zx2c4.com/wireguard/tun"
+	"golang.zx2c4.com/wireguard/windows/services"
+)
+
+func cleanupStaleAdapters() {
+	defer printPanic()
+
+	m, err := mgr.Connect()
+	if err != nil {
+		return
+	}
+	defer m.Disconnect()
+
+	tun.WintunPool.DeleteMatchingInterfaces(func(wintun *wintun.Interface) bool {
+		interfaceName, err := wintun.Name()
+		if err != nil {
+			log.Printf("Removing Wintun interface %s because determining interface name failed: %v", wintun.GUID().String(), err)
+			return true
+		}
+		serviceName, err := services.ServiceNameOfTunnel(interfaceName)
+		if err != nil {
+			log.Printf("Removing Wintun interface ‘%s’ because determining tunnel service name failed: %v", interfaceName, err)
+			return true
+		}
+		service, err := m.OpenService(serviceName)
+		if err == windows.ERROR_SERVICE_DOES_NOT_EXIST {
+			log.Printf("Removing Wintun interface ‘%s’ because no service for it exists", interfaceName)
+			return true
+		} else if err != nil {
+			return false
+		}
+		defer service.Close()
+		status, err := service.Query()
+		if err != nil {
+			return false
+		}
+		if status.State == svc.Stopped {
+			log.Printf("Removing Wintun interface ‘%s’ because its service is stopped", interfaceName)
+			return true
+		}
+		return false
+	})
+}
-- 
cgit v1.2.3-59-g8ed1b