summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Nate Myren <ntmyren@google.com> 2024-10-03 17:00:48 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2024-10-03 17:00:48 +0000
commit75844136d3849868edb22e1b23c30e2e03c48746 (patch)
tree111145b3a1fe2e77b3732306bb637134f6e62545
parentcf9208796d929be7bf93915b5d67165205cbaf91 (diff)
parent36ed35e30511d045d815aa271c3806b1f32d5271 (diff)
Merge "[Binder][XIAOMI][Bugfix] Skip appops header in native parcel. [1/2]" into main
-rw-r--r--core/java/android/app/AppOpsManager.java11
1 files changed, 11 insertions, 0 deletions
diff --git a/core/java/android/app/AppOpsManager.java b/core/java/android/app/AppOpsManager.java
index 5214d2c9c02a..091d5ab8534d 100644
--- a/core/java/android/app/AppOpsManager.java
+++ b/core/java/android/app/AppOpsManager.java
@@ -10193,6 +10193,9 @@ public class AppOpsManager {
}
p.writeInt(Parcel.EX_HAS_NOTED_APPOPS_REPLY_HEADER);
+ final int sizePosition = p.dataPosition();
+ // Write size placeholder. With this size we can easily skip it in native.
+ p.writeInt(0);
int numAttributionWithNotesAppOps = notedAppOps.size();
p.writeInt(numAttributionWithNotesAppOps);
@@ -10209,6 +10212,12 @@ public class AppOpsManager {
}
}
}
+
+ final int payloadPosition = p.dataPosition();
+ p.setDataPosition(sizePosition);
+ // Total header size including 4 bytes size itself.
+ p.writeInt(payloadPosition - sizePosition);
+ p.setDataPosition(payloadPosition);
}
/**
@@ -10222,6 +10231,8 @@ public class AppOpsManager {
* @hide
*/
public static void readAndLogNotedAppops(@NonNull Parcel p) {
+ // Skip size.
+ p.readInt();
int numAttributionsWithNotedAppOps = p.readInt();
for (int i = 0; i < numAttributionsWithNotedAppOps; i++) {