From b49ea8635bad8de5d70224683b7062e33eb51bf5 Mon Sep 17 00:00:00 2001
From: Mygod <contact-github@mygod.be>
Date: Sat, 23 Jun 2018 18:07:15 +0800
Subject: [PATCH] Fix crash on intent is null

---
 .../java/be/mygod/vpnhotspot/TetheringService.kt | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/mobile/src/main/java/be/mygod/vpnhotspot/TetheringService.kt b/mobile/src/main/java/be/mygod/vpnhotspot/TetheringService.kt
index c95169b2..950193ad 100644
--- a/mobile/src/main/java/be/mygod/vpnhotspot/TetheringService.kt
+++ b/mobile/src/main/java/be/mygod/vpnhotspot/TetheringService.kt
@@ -85,13 +85,15 @@ class TetheringService : IpNeighbourMonitoringService(), UpstreamMonitor.Callbac
 
     override fun onBind(intent: Intent?) = binder
 
-    override fun onStartCommand(intent: Intent, flags: Int, startId: Int): Int {
-        val iface = intent.getStringExtra(EXTRA_ADD_INTERFACE)
-        synchronized(routings) {
-            if (iface != null) routings[iface] = null
-            routings.remove(intent.getStringExtra(EXTRA_REMOVE_INTERFACE))?.stop()
-            updateRoutingsLocked()
-        }
+    override fun onStartCommand(intent: Intent?, flags: Int, startId: Int): Int {
+        if (intent != null) {
+            val iface = intent.getStringExtra(EXTRA_ADD_INTERFACE)
+            synchronized(routings) {
+                if (iface != null) routings[iface] = null
+                routings.remove(intent.getStringExtra(EXTRA_REMOVE_INTERFACE))?.stop()
+                updateRoutingsLocked()
+            }
+        } else if (routings.isEmpty()) stopSelf(startId)
         return START_NOT_STICKY
     }