diff options
| author | 2024-10-03 17:00:48 +0000 | |
|---|---|---|
| committer | 2024-10-03 17:00:48 +0000 | |
| commit | 75844136d3849868edb22e1b23c30e2e03c48746 (patch) | |
| tree | 111145b3a1fe2e77b3732306bb637134f6e62545 | |
| parent | cf9208796d929be7bf93915b5d67165205cbaf91 (diff) | |
| parent | 36ed35e30511d045d815aa271c3806b1f32d5271 (diff) | |
Merge "[Binder][XIAOMI][Bugfix] Skip appops header in native parcel. [1/2]" into main
| -rw-r--r-- | core/java/android/app/AppOpsManager.java | 11 |
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++) { |