commit 213dbe30a1b0cfaa45c4a5cef141b96e6ffce3b1
author Eric Biggers <ebiggers@google.com> Mon Aug 10 17:52:39 2020 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Aug 10 17:52:39 2020 +0000
tree b9055c95ff9150fd3454fe72cd19d8bb658bc075
parent 5de675c93be181256ecc0361e885fa922693cfb8
parent 4ac43f0e163572d68313411ef734f364fb253f3a

Merge "EncryptInPlace: ensure that backup superblocks get encrypted"

EncryptInplace.cpp

diff --git a/EncryptInplace.cpp b/EncryptInplace.cpp
index 9d304da..b1bd11d 100644
--- a/EncryptInplace.cpp
+++ b/EncryptInplace.cpp
@@ -205,9 +205,16 @@
         data->count = 0;
 
         for (block = 0; block < block_count; block++) {
-            int used = (aux_info.bg_desc[i].bg_flags & EXT4_BG_BLOCK_UNINIT)
-                           ? 0
-                           : bitmap_get_bit(block_bitmap, block);
+            int used;
+
+            if (aux_info.bg_desc[i].bg_flags & EXT4_BG_BLOCK_UNINIT) {
+                // In block groups with an uninitialized block bitmap, we only
+                // need to encrypt the backup superblock (if one is present).
+                used = (ext4_bg_has_super_block(i) && block < 1 + aux_info.bg_desc_blocks);
+            } else {
+                used = bitmap_get_bit(block_bitmap, block);
+            }
+
             update_progress(data, used);
             if (used) {
                 if (data->count == 0) {