diff options
author | Christopher Tate <ctate@google.com> | 2015-01-08 18:42:33 -0800 |
---|---|---|
committer | Christopher Tate <ctate@google.com> | 2015-01-14 16:36:30 -0800 |
commit | 43a4a8c777fbb8f71540ac7fbe82674489ef557b (patch) | |
tree | ab4f4b3a27124b244aa8b3cbb013c674da07731d /core/java/android/app | |
parent | a2fa3d219bc148c196b0eb3cf7b3b1bd453e830b (diff) | |
download | frameworks_base-43a4a8c777fbb8f71540ac7fbe82674489ef557b.zip frameworks_base-43a4a8c777fbb8f71540ac7fbe82674489ef557b.tar.gz frameworks_base-43a4a8c777fbb8f71540ac7fbe82674489ef557b.tar.bz2 |
Fix redundant file backups
We'd observed a bug in which an unchanged file was nevertheless
being redundantly transmitted for backup on every backup pass.
The underlying issue turns out to have been the FileBackupHelper
base implementation's logic for diffing the prior-state file
set against the current state, in the case when there had been
deletions of prior files. In addition, there was also a
parallel bug in which file checksums were not calculated
properly in some cases, leading to at least one additional
redundant backup of the file in question.
Bug 18694053
Change-Id: Ie0dec06486b5fef4624561737019569c85d6b2a0
Diffstat (limited to 'core/java/android/app')
-rw-r--r-- | core/java/android/app/backup/BackupHelperDispatcher.java | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/core/java/android/app/backup/BackupHelperDispatcher.java b/core/java/android/app/backup/BackupHelperDispatcher.java index 5466db5..6811532 100644 --- a/core/java/android/app/backup/BackupHelperDispatcher.java +++ b/core/java/android/app/backup/BackupHelperDispatcher.java @@ -50,7 +50,6 @@ public class BackupHelperDispatcher { Header header = new Header(); TreeMap<String,BackupHelper> helpers = (TreeMap<String,BackupHelper>)mHelpers.clone(); FileDescriptor oldStateFD = null; - FileDescriptor newStateFD = newState.getFileDescriptor(); if (oldState != null) { oldStateFD = oldState.getFileDescriptor(); |