commit e88a85e0d0a25c943e974114557770ba10b81847
author Eino-Ville Talvala <etalvala@google.com> Tue Feb 19 12:54:57 2013 -0800
committer Eino-Ville Talvala <etalvala@google.com> Tue Feb 19 18:10:05 2013 -0800
tree d8872089aea8322195ac333e576a3c9407ac6c60
parent c4905eb49d20667381f3cda7c6f6894234404bd3

Fixes to native app ops services

Bug: 8181262
Change-Id: I3d3f20453f6c6f2756c988363c5b5453ef309cb6

libs/binder/AppOpsManager.cpp

diff --git a/libs/binder/AppOpsManager.cpp b/libs/binder/AppOpsManager.cpp
index 5d36dd2..7ac1b11 100644
--- a/libs/binder/AppOpsManager.cpp
+++ b/libs/binder/AppOpsManager.cpp
@@ -32,23 +32,21 @@
     int64_t startTime = 0;
     mLock.lock();
     sp<IAppOpsService> service = mService;
-    while (true) {
-        if (service == NULL || !service->asBinder()->isBinderAlive()) {
-            sp<IBinder> binder = defaultServiceManager()->checkService(_appops);
-            if (binder == NULL) {
-                // Wait for the app ops service to come back...
-                if (startTime == 0) {
-                    startTime = uptimeMillis();
-                    ALOGI("Waiting for app ops service");
-                } else if ((uptimeMillis()-startTime) > 10000) {
-                    ALOGW("Waiting too long for app ops service, giving up");
-                    return NULL;
-                }
-                sleep(1);
-            } else {
-                service = interface_cast<IAppOpsService>(binder);
-                mService = service;
+    while (service == NULL || !service->asBinder()->isBinderAlive()) {
+        sp<IBinder> binder = defaultServiceManager()->checkService(_appops);
+        if (binder == NULL) {
+            // Wait for the app ops service to come back...
+            if (startTime == 0) {
+                startTime = uptimeMillis();
+                ALOGI("Waiting for app ops service");
+            } else if ((uptimeMillis()-startTime) > 10000) {
+                ALOGW("Waiting too long for app ops service, giving up");
+                return NULL;
             }
+            sleep(1);
+        } else {
+            service = interface_cast<IAppOpsService>(binder);
+            mService = service;
         }
     }
     mLock.unlock();

libs/binder/IAppOpsService.cpp

diff --git a/libs/binder/IAppOpsService.cpp b/libs/binder/IAppOpsService.cpp
index d8922c9..282b30f 100644
--- a/libs/binder/IAppOpsService.cpp
+++ b/libs/binder/IAppOpsService.cpp
@@ -45,8 +45,8 @@
         data.writeString16(packageName);
         remote()->transact(CHECK_OPERATION_TRANSACTION, data, &reply);
         // fail on exception
-        if (reply.readExceptionCode() != 0) return 0;
-        return reply.readInt32() != 0;
+        if (reply.readExceptionCode() != 0) return MODE_ERRORED;
+        return reply.readInt32();
     }
 
     virtual int32_t noteOperation(int32_t code, int32_t uid, const String16& packageName) {
@@ -57,8 +57,8 @@
         data.writeString16(packageName);
         remote()->transact(NOTE_OPERATION_TRANSACTION, data, &reply);
         // fail on exception
-        if (reply.readExceptionCode() != 0) return 0;
-        return reply.readInt32() != 0;
+        if (reply.readExceptionCode() != 0) return MODE_ERRORED;
+        return reply.readInt32();
     }
 
     virtual int32_t startOperation(int32_t code, int32_t uid, const String16& packageName) {
@@ -69,8 +69,8 @@
         data.writeString16(packageName);
         remote()->transact(START_OPERATION_TRANSACTION, data, &reply);
         // fail on exception
-        if (reply.readExceptionCode() != 0) return 0;
-        return reply.readInt32() != 0;
+        if (reply.readExceptionCode() != 0) return MODE_ERRORED;
+        return reply.readInt32();
     }
 
     virtual void finishOperation(int32_t code, int32_t uid, const String16& packageName) {