From b1ed2802fb944bb5e3dea3b4a80c05db3a9df7c3 Mon Sep 17 00:00:00 2001
From: Jakob Borg <jakob@kastelo.net>
Date: Sat, 21 Sep 2024 09:27:23 +0200
Subject: [PATCH] fix(connections): skip point-to-point interfaces when listing
 LANs (fixes #9719) (#9720)

Point-to-point interfaces are typically VPNs and similar which, for our
purposes, do not qualify as LANs.
---
 lib/osutil/net.go | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/lib/osutil/net.go b/lib/osutil/net.go
index eeb8bafde..4400d8f23 100644
--- a/lib/osutil/net.go
+++ b/lib/osutil/net.go
@@ -11,21 +11,26 @@ import (
 )
 
 func GetLans() ([]*net.IPNet, error) {
-	ifs, err := net.Interfaces()
+	intfs, err := net.Interfaces()
 	if err != nil {
 		return nil, err
 	}
 	var addrs []net.Addr
 
-	for _, currentIf := range ifs {
-		if currentIf.Flags&net.FlagRunning == 0 {
+	for _, intf := range intfs {
+		if intf.Flags&net.FlagRunning == 0 {
 			continue
 		}
-		currentAddrs, err := currentIf.Addrs()
+		if intf.Flags&net.FlagPointToPoint != 0 {
+			// Point-to-point interfaces are typically VPNs and similar
+			// which, for our purposes, do not qualify as LANs.
+			continue
+		}
+		intfAddrs, err := intf.Addrs()
 		if err != nil {
 			return nil, err
 		}
-		addrs = append(addrs, currentAddrs...)
+		addrs = append(addrs, intfAddrs...)
 	}
 
 	nets := make([]*net.IPNet, 0, len(addrs))