From 65d71894c9f254ea1a5eb836e454dd263b13a867 Mon Sep 17 00:00:00 2001 From: Eric Sandeen Date: Tue, 30 Mar 2010 11:42:51 -0500 Subject: [PATCH] e2fsck: make block counting variable in pass1 64 bits Justin reported that creating a 4T file with posix_fallocate led to fsck errors: e2fsck 1.41.10 (10-Feb-2009) Pass 1: Checking inodes, blocks, and sizes Inode 12, i_blocks is 8589935432, should be 840. Fix? yes This looks like a 32-bit overflow. commmit 8a8f36540bbf5d4397cf476e216e9a720b5c1d8e added handling of the high i_blocks number, but we accumulate blocks in the num_blocks field, and that's still just 32 bits. Note: we don't need to expand max_blocks for now, that's only used in the non-extents case, and those files have smaller max sizes. I haven't been able to replicate the problem, oddly, but Justin reports that this patch fixed his situation. Reported-by: Justin Maggard Signed-off-by: Eric Sandeen Signed-off-by: Theodore Ts'o --- e2fsck/pass1.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/e2fsck/pass1.c b/e2fsck/pass1.c index 3b65e8a..0d4c315 100644 --- a/e2fsck/pass1.c +++ b/e2fsck/pass1.c @@ -80,7 +80,7 @@ struct process_block_struct { ext2_ino_t ino; unsigned is_dir:1, is_reg:1, clear:1, suppress:1, fragmented:1, compressed:1, bbcheck:1; - blk_t num_blocks; + blk64_t num_blocks; blk_t max_blocks; e2_blkcnt_t last_block; e2_blkcnt_t last_db_block; -- 1.8.3.1