brianhansonxyz/vpnhotspotmod
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Be more careful about checking upstreams were removed

Browse Source
This commit is contained in:
Mygod
2018-12-14 00:56:28 +08:00
parent c3cb26ce35
commit 366ae7dab6
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
mobile/src/main/java/be/mygod/vpnhotspot/net/Routing.kt
View File

@@ -90,7 +90,8 @@ class Routing(val downstream: String, ownerAddress: InterfaceAddress? = null) {
override fun onLost() = synchronized(this@Routing) { override fun onLost() = synchronized(this@Routing) {
val subrouting = subrouting ?: return val subrouting = subrouting ?: return
check(upstreams.remove(subrouting.upstream)) // we could be removing fallback subrouting which no collision could ever happen, check before removing
if (subrouting.upstream != null) check(upstreams.remove(subrouting.upstream))
subrouting.close() subrouting.close()
TrafficRecorder.update() // record stats before removing rules to prevent stats losing TrafficRecorder.update() // record stats before removing rules to prevent stats losing
subrouting.revert() subrouting.revert()