commit aa50c2224a7b5a6c98cf5d428e979c8dad0d826e
author Tobias Thierer <tobiast@google.com> Mon Oct 19 20:01:28 2020 +0100
committer Tobias Thierer <tobiast@google.com> Tue Oct 20 16:12:38 2020 +0000
tree b34a1ab3d5c325ab59c3e6294fc671b0b68c4f28
parent 89eed4d26d55175e0fc5ff4d284347140224a6b3

UBMS.mRunInitIntent: Mark as immutable.

No production code relies on the mutability. Code in fakeandroid's
AlarmManager added in http://ag/3826144 relies on the mutability,
and backup/internal/PerformInitializeTask sets a (now immutable)
PendingIntent, but fakeandroid uses a different AlarmManager instance
that the real system so these should be unrelated (I think - reviewer,
please double-check my thinking).

Fixes: 170163281
Test: Treehugger
Change-Id: I3dd45cc270c2fd8ab9d4b3402edc6cc5d765fd5a

services/backup/java/com/android/server/backup/UserBackupManagerService.java

diff --git a/services/backup/java/com/android/server/backup/UserBackupManagerService.java b/services/backup/java/com/android/server/backup/UserBackupManagerService.java
index aa8532b..2ff66b5 100644
--- a/services/backup/java/com/android/server/backup/UserBackupManagerService.java
+++ b/services/backup/java/com/android/server/backup/UserBackupManagerService.java
@@ -650,7 +650,7 @@
                         context,
                         /* requestCode */ 0,
                         initIntent,
-                        /* flags */ 0,
+                        /* flags */ PendingIntent.FLAG_IMMUTABLE,
                         UserHandle.of(userId));
 
         // Set up the backup-request journaling