Merge pull request #696 from mirokuratczyk/master

Do not call nw_path_monitor_cancel on nil monitor
Psiphon-Labs · Oct 1, 2024 · 10ab761 · 10ab761
2 parents e9a9b00 + 2526e54
commit 10ab761
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/MobileLibrary/iOS/PsiphonTunnel/PsiphonTunnel/Network/Reachability/DefaultRouteMonitor.m b/MobileLibrary/iOS/PsiphonTunnel/PsiphonTunnel/Network/Reachability/DefaultRouteMonitor.m
@@ -192,8 +192,10 @@ - (void)stop API_AVAILABLE(macos(10.14), ios(12.0), watchos(5.0), tvos(12.0)) {
         // Note: this monitor cannot be used after being cancelled. Its update handler will not
         // fire again and cannot be restarted with nw_path_monitor_start. A new monitor must be
         // created and started.
-        nw_path_monitor_cancel(self->monitor);
-        self->monitor = nil;
+        if (self->monitor != nil) {
+            nw_path_monitor_cancel(self->monitor);
+            self->monitor = nil;
+        }
     }
 }