66f761c526eff2cb6115379f9b2439fe2b48d4a2 - platform/frameworks/base

commit	66f761c526eff2cb6115379f9b2439fe2b48d4a2	[log] [tgz]
author	Lifu Tang <lifu@google.com>	Mon Nov 13 15:51:54 2017 -0800
committer	Lifu Tang <lifu@google.com>	Tue Nov 14 15:18:17 2017 -0800
tree	02abf31b494972c2a45c822c44d5bda8b55b9423
parent	2dd2cbcb12a21adccd3c349590a01e7ed009f679 [diff]

Use callback to run binder service to avoid race

The binder interface should always be managed by ServiceWatcher, and
shouldn't be exposed to the caller. Otherwise the caller could cache an
unbound service, and could cause system crash.

Bug: 69008332
Test: build and manual test
Change-Id: I70d1fe0bace7b0fb1c3844c2c9113b1fcf95f784

services/core/java/com/android/server/ServiceWatcher.java[diff]
services/core/java/com/android/server/location/ActivityRecognitionProxy.java[diff]
services/core/java/com/android/server/location/FusedProxy.java[diff]
services/core/java/com/android/server/location/GeocoderProxy.java[diff]
services/core/java/com/android/server/location/GeofenceProxy.java[diff]
services/core/java/com/android/server/location/LocationProviderProxy.java[diff]

6 files changed

tree: 02abf31b494972c2a45c822c44d5bda8b55b9423