From: Theodore Ts'o <tytso@mit.edu>
Date: Fri, 10 May 2019 23:21:03 +0000 (-0400)
Subject: e2fsck: remove an potentially ambiguous dangling else clause
X-Git-Tag: v1.45.1~1
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=a033d4ef03d4d28957caa8f3ef59a89ba9307603;p=tools%2Fe2fsprogs.git

e2fsck: remove an potentially ambiguous dangling else clause

This doesn't actually fix a bug or change behavior, but it removes a
clang warning.

Signed-off-by: Theodore Ts'o <tytso@mit.edu>
---

diff --git a/e2fsck/pass5.c b/e2fsck/pass5.c
index 8100909..3a5c88d 100644
--- a/e2fsck/pass5.c
+++ b/e2fsck/pass5.c
@@ -886,11 +886,12 @@ static void check_inode_end(e2fsck_t ctx)
 	 * be written back to disk.
 	 */
 check_intra_bg_tail:
-	if (!asked && fs->flags & EXT2_FLAG_IBITMAP_TAIL_PROBLEM)
+	if (!asked && fs->flags & EXT2_FLAG_IBITMAP_TAIL_PROBLEM) {
 		if (fix_problem(ctx, PR_5_INODE_BMAP_PADDING, &pctx))
 			ext2fs_mark_ib_dirty(fs);
 		else
 			ext2fs_unmark_valid(fs);
+	}
 }
 
 static void check_block_end(e2fsck_t ctx)