Fix monitor

SagerNet · Nov 12, 2024 · 858d089 · 858d089
1 parent d105263
commit 858d089
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/monitor_darwin.go b/monitor_darwin.go
@@ -107,11 +107,10 @@ func (m *networkUpdateMonitor) Close() error {
 }
 
 func (m *defaultInterfaceMonitor) checkUpdate() error {
-	err := m.interfaceFinder.Update()
-	if err != nil {
-		return E.Cause(err, "update interfaces")
-	}
-	var defaultInterface *control.Interface
+	var (
+		defaultInterface *control.Interface
+		err              error
+	)
 	if m.underNetworkExtension {
 		defaultInterface, err = m.getDefaultInterfaceBySocket()
 		if err != nil {

diff --git a/monitor_shared.go b/monitor_shared.go
@@ -74,7 +74,12 @@ func (m *defaultInterfaceMonitor) delayCheckUpdate() {
 }
 
 func (m *defaultInterfaceMonitor) postCheckUpdate() {
-	err := m.checkUpdate()
+	err := m.interfaceFinder.Update()
+	if err != nil {
+		m.logger.Error("update interface: ", err)
+		return
+	}
+	err = m.checkUpdate()
 	if errors.Is(err, ErrNoRoute) {
 		if !m.noRoute {
 			m.noRoute = true