Merge "[CM] Fix NPE due to unvalidated callback value" am: ca7356d684
am: 51ddc176ab Change-Id: Idf439c1f6badd715776eaaced1a781b066ba6d37
This commit is contained in:
commit
407e5ecdaa
1 changed files with 34 additions and 13 deletions
|
|
@ -3449,6 +3449,11 @@ public class ConnectivityManager {
|
|||
final NetworkCallback callback;
|
||||
synchronized (sCallbacks) {
|
||||
callback = sCallbacks.get(request);
|
||||
if (callback == null) {
|
||||
Log.w(TAG,
|
||||
"callback not found for " + getCallbackName(message.what) + " message");
|
||||
return;
|
||||
}
|
||||
if (message.what == CALLBACK_UNAVAIL) {
|
||||
sCallbacks.remove(request);
|
||||
callback.networkRequest = ALREADY_UNREGISTERED;
|
||||
|
|
@ -3457,10 +3462,6 @@ public class ConnectivityManager {
|
|||
if (DBG) {
|
||||
Log.d(TAG, getCallbackName(message.what) + " for network " + network);
|
||||
}
|
||||
if (callback == null) {
|
||||
Log.w(TAG, "callback not found for " + getCallbackName(message.what) + " message");
|
||||
return;
|
||||
}
|
||||
|
||||
switch (message.what) {
|
||||
case CALLBACK_PRECHECK: {
|
||||
|
|
|
|||
Loading…
Reference in a new issue