Commit 21696381 authored Feb 26, 2016 by Svet Ganov

Refresh the native app ops manager's token if needed.

The app ops manager serivce keeps track of started app ops
by a process by stashing them in a binder object that is
passed to the remote process to hold on to. Hence, the data
is in the system process and is referenced by the client
process it tracks to prevent garbage collaction and limit
risk of resource leaks. It is possible that the client
process is longer lived than the system seriver, e.g. camera
server starts an app op, the system server dies for some
reason, the camera server's app ops manager ends up with
a stale token. The solution is to check the validity of the
token and refresh it if needed.

bug:27344593

Change-Id: I0abb39e777b7b66b1b5cefede7e7b41c4404c4b1

parent a4c2c3ba

libs/binder/AppOpsManager.cpp

+1 −1

Original line number	Diff line number	Diff line
		@@ -28,7 +28,7 @@ static sp<IBinder> gToken;

		static const sp<IBinder>& getToken(const sp<IAppOpsService>& service) {
		pthread_mutex_lock(&gTokenMutex);
		if (gToken == NULL) {
		if (gToken == NULL \|\| gToken->pingBinder() != NO_ERROR) {
		gToken = service->getToken(new BBinder());
		}
		pthread_mutex_unlock(&gTokenMutex);