Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Add null pointer check for mConfig in onSessionLost"

Browse Source
This commit is contained in:
Benedict Wong
2021-04-22 18:20:16 +00:00
committed by Gerrit Code Review
parent 199b0018ff decca62338
commit 9fe8fcdb37
1 changed files with 13 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
services/core/java/com/android/server/connectivity/Vpn.java
View File

@@ -2696,19 +2696,21 @@ public class Vpn {
// prevent the NetworkManagementEventObserver from killing this VPN based on the // prevent the NetworkManagementEventObserver from killing this VPN based on the
// interface going down (which we expect). // interface going down (which we expect).
mInterface = null; mInterface = null;
mConfig.interfaze = null; if (mConfig != null) {
mConfig.interfaze = null;
// Set as unroutable to prevent traffic leaking while the interface is down. // Set as unroutable to prevent traffic leaking while the interface is down.
if (mConfig != null && mConfig.routes != null) { if (mConfig.routes != null) {
final List<RouteInfo> oldRoutes = new ArrayList<>(mConfig.routes); final List<RouteInfo> oldRoutes = new ArrayList<>(mConfig.routes);
mConfig.routes.clear(); mConfig.routes.clear();
for (final RouteInfo route : oldRoutes) { for (final RouteInfo route : oldRoutes) {
mConfig.routes.add(new RouteInfo(route.getDestination(), null /*gateway*/, mConfig.routes.add(new RouteInfo(route.getDestination(),
null /*iface*/, RTN_UNREACHABLE)); null /*gateway*/, null /*iface*/, RTN_UNREACHABLE));
} }
if (mNetworkAgent != null) { if (mNetworkAgent != null) {
mNetworkAgent.sendLinkProperties(makeLinkProperties()); mNetworkAgent.sendLinkProperties(makeLinkProperties());
}
} }
} }
} }