tree f00e4ec38de0f616443b931e195b8436d1f6a84d
parent f5bec0d781499a3bde7aa54f2072019ce2709e70
author Hugo Benichi <hugobenichi@google.com> 1511747836 +0900
committer Hugo Benichi <hugobenichi@google.com> 1513126478 +0900

ConnectivityService: synchronize access on mLockdownEnabled

The mLockdownEnabled boolean and the mLockdownTracker objects are read
and mutated in many places involving vpn logic inside of
ConnectivityService. This includes codepaths run on the
ConnectivityService handler and codepaths run on Binder calls from
IConnectivityManager.aidl, however the access to these variables are not
synchronized.

This patch adds proper synchronization to mLockdownEnabled and
mLockdownTracker by moving access to them into the mVpns lock used for
all of vpn logic.

Bug: 18331877
Test: runtest frameworks-net
Change-Id: I4abde43b1036861f4486dd2b5567782d10204bd6