Make sure to call the system completionHandler after the stop operation is fully finished

On iOS 13.4 the tunnel process is terminated when the completion handler is called. Postponing the call until we clear out the operation queue lets PacketTunnel deallocate before the process exits

1 files changed, 4 insertions, 2 deletions

diff --git a/ios/PacketTunnel/PacketTunnelProvider.swift b/ios/PacketTunnel/PacketTunnelProvider.swift
index 3657009909..784e3fc8fe 100644
--- a/ios/PacketTunnel/PacketTunnelProvider.swift
+++ b/ios/PacketTunnel/PacketTunnelProvider.swift
@@ -84,12 +84,14 @@ class PacketTunnelProvider: NEPacketTunnelProvider {
                 case .failure(let error):
                     self.logger.error(chainedError: error, message: "Failed to stop the tunnel")
                 }
-
-                completionHandler()
                 finish()
             }
         }
 
+        operation.addDidFinishBlockObserver { (op) in
+            completionHandler()
+        }
+
         exclusivityController.addOperation(operation, categories: [.exclusive])
     }