Merge "ConnectivityManager: fix potential NPE"

am: b688dd18ab

Change-Id: If562fc6a921569ad9d3141cac72b6f395a30da9e
This commit is contained in:
Hugo Benichi
2017-02-15 01:05:24 +00:00
committed by android-build-merger

View File

@@ -1432,8 +1432,9 @@ public class ConnectivityManager {
private void sendExpireMsgForFeature(NetworkCapabilities netCap, int seqNum, int delay) { private void sendExpireMsgForFeature(NetworkCapabilities netCap, int seqNum, int delay) {
if (delay >= 0) { if (delay >= 0) {
Log.d(TAG, "sending expire msg with seqNum " + seqNum + " and delay " + delay); Log.d(TAG, "sending expire msg with seqNum " + seqNum + " and delay " + delay);
Message msg = sCallbackHandler.obtainMessage(EXPIRE_LEGACY_REQUEST, seqNum, 0, netCap); CallbackHandler handler = getHandler();
sCallbackHandler.sendMessageDelayed(msg, delay); Message msg = handler.obtainMessage(EXPIRE_LEGACY_REQUEST, seqNum, 0, netCap);
handler.sendMessageDelayed(msg, delay);
} }
} }
@@ -2868,19 +2869,19 @@ public class ConnectivityManager {
} }
} }
static final HashMap<NetworkRequest, NetworkCallback> sCallbacks = new HashMap<>(); private static final HashMap<NetworkRequest, NetworkCallback> sCallbacks = new HashMap<>();
static CallbackHandler sCallbackHandler; private static CallbackHandler sCallbackHandler;
private final static int LISTEN = 1; private static final int LISTEN = 1;
private final static int REQUEST = 2; private static final int REQUEST = 2;
private NetworkRequest sendRequestForNetwork(NetworkCapabilities need, private NetworkRequest sendRequestForNetwork(NetworkCapabilities need,
NetworkCallback callback, int timeoutMs, int action, int legacyType) { NetworkCallback callback, int timeoutMs, int action, int legacyType) {
return sendRequestForNetwork(need, callback, getHandler(), timeoutMs, action, legacyType); return sendRequestForNetwork(need, callback, timeoutMs, action, legacyType, getHandler());
} }
private NetworkRequest sendRequestForNetwork(NetworkCapabilities need, private NetworkRequest sendRequestForNetwork(NetworkCapabilities need, NetworkCallback callback,
NetworkCallback callback, Handler handler, int timeoutMs, int action, int legacyType) { int timeoutMs, int action, int legacyType, CallbackHandler handler) {
if (callback == null) { if (callback == null) {
throw new IllegalArgumentException("null NetworkCallback"); throw new IllegalArgumentException("null NetworkCallback");
} }