commit a0ae2f99a166b5adbf30542ae4cbe55b2276586f
author Nate Myren <ntmyren@google.com> Thu Apr 01 15:19:26 2021 -0700
committer Nate Myren <ntmyren@google.com> Thu Apr 01 15:19:26 2021 -0700
tree c0c49f0e46f6574941fea4e033588ba829a5a0b6
parent 2ce1b7e95170f000a25175423af7732abf0294ab

Update native AppOpsService to unpack SyncNotedAppOp

Still return only mode, but unpack the "null attribution tag" byte from
SyncNotedAppOp first.

Bug: 184111263
Test: Presubmit
Change-Id: I53cf9df803b23c30b0721998ff1ff18d928afd75

libs/binder/IAppOpsService.cpp

diff --git a/libs/binder/IAppOpsService.cpp b/libs/binder/IAppOpsService.cpp
index 1897969..44aa55e 100644
--- a/libs/binder/IAppOpsService.cpp
+++ b/libs/binder/IAppOpsService.cpp
@@ -63,6 +63,7 @@
         remote()->transact(NOTE_OPERATION_TRANSACTION, data, &reply);
         // fail on exception
         if (reply.readExceptionCode() != 0) return MODE_ERRORED;
+        reply.readByte();
         return reply.readInt32();
     }
 
@@ -84,6 +85,7 @@
         remote()->transact(START_OPERATION_TRANSACTION, data, &reply);
         // fail on exception
         if (reply.readExceptionCode() != 0) return MODE_ERRORED;
+        reply.readByte();
         return reply.readInt32();
     }