diff options
| author | Hugo Benichi <hugobenichi@google.com> | 2017-02-03 14:18:44 +0900 |
|---|---|---|
| committer | Hugo Benichi <hugobenichi@google.com> | 2017-02-14 16:01:40 +0900 |
| commit | 6f260f3691c2b33dca87f8179426fb7941319bc7 (patch) | |
| tree | 7dd2dc9f2aef61e31e74651bf1fbfd70697c8edf /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | ed6d75d5f31dd69d2c91cd82cddd5ee1db49b2c7 (diff) | |
ConnectivityManager: fix potential NPE
ConnectivityManager static sCallbackHandler is referenced and directly
used in a way that is not ensuring its proper initialization.
This patch fixes this potential NPE by using getHandler() instead.
Also this patch changes sendRequestForNetwork's signature to only accept
the subtype CallbackHandler instead of Handler: without using
CallbackHandler the NetworkCallbacks are not triggered properly and
bookkeeping of sCallbacks does not happen. sendRequestForNetwork's
signature now makes this explicit.
This step prepares the addition of overloaded versions of
registerNetworkCallback and cie that takes custom Handlers.
Test: build, flashed, manually checked connectivity
Change-Id: I52e8a2cb5075e7aef7b35e30c9845cacba927d13
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions