Use ext2fs_file_acl_block() instead of using .i_file_acl directly
This provides support for 48-bit file acl blocks.
Signed-off-by: Valerie Aurora Henson <vaurora@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
diff --git a/e2fsck/message.c b/e2fsck/message.c
index 77f9756..6bda4f5 100644
--- a/e2fsck/message.c
+++ b/e2fsck/message.c
@@ -311,7 +311,7 @@
printf("%u", inode->i_faddr);
break;
case 'f':
- printf("%u", inode->i_file_acl);
+ printf("%llu", ext2fs_file_acl_block(inode));
break;
case 'd':
printf("%u", (LINUX_S_ISDIR(inode->i_mode) ?
diff --git a/e2fsck/pass1.c b/e2fsck/pass1.c
index 49b1863..c12e5f8 100644
--- a/e2fsck/pass1.c
+++ b/e2fsck/pass1.c
@@ -1026,7 +1026,7 @@
(inode->i_block[EXT2_IND_BLOCK] ||
inode->i_block[EXT2_DIND_BLOCK] ||
inode->i_block[EXT2_TIND_BLOCK] ||
- inode->i_file_acl)) {
+ ext2fs_file_acl_block(inode))) {
inodes_to_process[process_inode_count].ino = ino;
inodes_to_process[process_inode_count].inode = *inode;
process_inode_count++;
@@ -1204,7 +1204,8 @@
ret = (ib_a->inode.i_block[EXT2_IND_BLOCK] -
ib_b->inode.i_block[EXT2_IND_BLOCK]);
if (ret == 0)
- ret = ib_a->inode.i_file_acl - ib_b->inode.i_file_acl;
+ ret = ext2fs_file_acl_block(&(ib_a->inode)) -
+ ext2fs_file_acl_block(&ib_b->inode);
if (ret == 0)
ret = ib_a->ino - ib_b->ino;
return ret;
@@ -1368,7 +1369,7 @@
int count;
region_t region = 0;
- blk = inode->i_file_acl;
+ blk = ext2fs_file_acl_block(inode);
if (blk == 0)
return 0;
@@ -1443,7 +1444,7 @@
if (pctx->errcode && fix_problem(ctx, PR_1_READ_EA_BLOCK, pctx))
goto clear_extattr;
header = (struct ext2_ext_attr_header *) block_buf;
- pctx->blk = inode->i_file_acl;
+ pctx->blk = ext2fs_file_acl_block(inode);
if (((ctx->ext_attr_ver == 1) &&
(header->h_magic != EXT2_EXT_ATTR_MAGIC_v1)) ||
((ctx->ext_attr_ver == 2) &&
@@ -1531,7 +1532,7 @@
clear_extattr:
if (region)
region_free(region);
- inode->i_file_acl = 0;
+ ext2fs_file_acl_block_set(inode, 0);
e2fsck_write_inode(ctx, ino, inode, "check_ext_attr");
return 0;
}
@@ -1840,7 +1841,8 @@
}
}
- if (inode->i_file_acl && check_ext_attr(ctx, pctx, block_buf)) {
+ if (ext2fs_file_acl_block(inode) &&
+ check_ext_attr(ctx, pctx, block_buf)) {
if (ctx->flags & E2F_FLAG_SIGNAL_MASK)
goto out;
pb.num_blocks++;
diff --git a/e2fsck/pass1b.c b/e2fsck/pass1b.c
index 9aeab50..003827d 100644
--- a/e2fsck/pass1b.c
+++ b/e2fsck/pass1b.c
@@ -82,8 +82,8 @@
struct block_el *block_list;
};
-static int process_pass1b_block(ext2_filsys fs, blk_t *blocknr,
- e2_blkcnt_t blockcnt, blk_t ref_blk,
+static int process_pass1b_block(ext2_filsys fs, blk64_t *blocknr,
+ e2_blkcnt_t blockcnt, blk64_t ref_blk,
int ref_offset, void *priv_data);
static void delete_file(e2fsck_t ctx, ext2_ino_t ino,
struct dup_inode *dp, char *block_buf);
@@ -307,12 +307,15 @@
if (ext2fs_inode_has_valid_blocks(&inode) ||
(ino == EXT2_BAD_INO))
- pctx.errcode = ext2fs_block_iterate2(fs, ino,
+ pctx.errcode = ext2fs_block_iterate3(fs, ino,
BLOCK_FLAG_READ_ONLY, block_buf,
process_pass1b_block, &pb);
- if (inode.i_file_acl)
- process_pass1b_block(fs, &inode.i_file_acl,
+ if (ext2fs_file_acl_block(&inode)) {
+ blk64_t blk = ext2fs_file_acl_block(&inode);
+ process_pass1b_block(fs, &blk,
BLOCK_COUNT_EXTATTR, 0, 0, &pb);
+ ext2fs_file_acl_block_set(&inode, blk);
+ }
if (pb.dup_blocks) {
end_problem_latch(ctx, PR_LATCH_DBLOCK);
if (ino >= EXT2_FIRST_INODE(fs->super) ||
@@ -327,9 +330,9 @@
}
static int process_pass1b_block(ext2_filsys fs EXT2FS_ATTR((unused)),
- blk_t *block_nr,
+ blk64_t *block_nr,
e2_blkcnt_t blockcnt EXT2FS_ATTR((unused)),
- blk_t ref_blk EXT2FS_ATTR((unused)),
+ blk64_t ref_blk EXT2FS_ATTR((unused)),
int ref_offset EXT2FS_ATTR((unused)),
void *priv_data)
{
@@ -554,9 +557,9 @@
}
static int delete_file_block(ext2_filsys fs,
- blk_t *block_nr,
+ blk64_t *block_nr,
e2_blkcnt_t blockcnt EXT2FS_ATTR((unused)),
- blk_t ref_block EXT2FS_ATTR((unused)),
+ blk64_t ref_block EXT2FS_ATTR((unused)),
int ref_offset EXT2FS_ATTR((unused)),
void *priv_data)
{
@@ -578,7 +581,7 @@
decrement_badcount(ctx, *block_nr, p);
} else
com_err("delete_file_block", 0,
- _("internal error: can't find dup_blk for %u\n"),
+ _("internal error: can't find dup_blk for %llu\n"),
*block_nr);
} else {
ext2fs_unmark_block_bitmap2(ctx->block_found_map, *block_nr);
@@ -605,7 +608,7 @@
e2fsck_read_inode(ctx, ino, &inode, "delete_file");
if (ext2fs_inode_has_valid_blocks(&inode))
- pctx.errcode = ext2fs_block_iterate2(fs, ino, BLOCK_FLAG_READ_ONLY,
+ pctx.errcode = ext2fs_block_iterate3(fs, ino, BLOCK_FLAG_READ_ONLY,
block_buf, delete_file_block, &pb);
if (pctx.errcode)
fix_problem(ctx, PR_1B_BLOCK_ITERATE, &pctx);
@@ -616,17 +619,18 @@
/* Inode may have changed by block_iterate, so reread it */
e2fsck_read_inode(ctx, ino, &inode, "delete_file");
e2fsck_clear_inode(ctx, ino, &inode, 0, "delete_file");
- if (inode.i_file_acl &&
+ if (ext2fs_file_acl_block(&inode) &&
(fs->super->s_feature_compat & EXT2_FEATURE_COMPAT_EXT_ATTR)) {
count = 1;
- pctx.errcode = ext2fs_adjust_ea_refcount(fs, inode.i_file_acl,
+ pctx.errcode = ext2fs_adjust_ea_refcount(fs,
+ ext2fs_file_acl_block(&inode),
block_buf, -1, &count);
if (pctx.errcode == EXT2_ET_BAD_EA_BLOCK_NUM) {
pctx.errcode = 0;
count = 1;
}
if (pctx.errcode) {
- pctx.blk = inode.i_file_acl;
+ pctx.blk = ext2fs_file_acl_block(&inode);
fix_problem(ctx, PR_1B_ADJ_EA_REFCOUNT, &pctx);
}
/*
@@ -637,9 +641,12 @@
*/
if ((count == 0) ||
ext2fs_test_block_bitmap2(ctx->block_dup_map,
- inode.i_file_acl))
- delete_file_block(fs, &inode.i_file_acl,
+ ext2fs_file_acl_block(&inode))) {
+ blk64_t blk = ext2fs_file_acl_block(&inode);
+ delete_file_block(fs, &blk,
BLOCK_COUNT_EXTATTR, 0, 0, &pb);
+ ext2fs_file_acl_block_set(&inode, blk);
+ }
}
}
@@ -651,9 +658,9 @@
};
static int clone_file_block(ext2_filsys fs,
- blk_t *block_nr,
+ blk64_t *block_nr,
e2_blkcnt_t blockcnt,
- blk_t ref_block EXT2FS_ATTR((unused)),
+ blk64_t ref_block EXT2FS_ATTR((unused)),
int ref_offset EXT2FS_ATTR((unused)),
void *priv_data)
{
@@ -711,7 +718,7 @@
return BLOCK_CHANGED;
} else
com_err("clone_file_block", 0,
- _("internal error: can't find dup_blk for %u\n"),
+ _("internal error: can't find dup_blk for %llu\n"),
*block_nr);
}
return 0;
@@ -724,7 +731,7 @@
errcode_t retval;
struct clone_struct cs;
struct problem_context pctx;
- blk_t blk;
+ blk64_t blk, new_blk;
dnode_t *n;
struct inode_el *ino_el;
struct dup_block *db;
@@ -744,7 +751,7 @@
pctx.ino = ino;
pctx.str = "clone_file";
if (ext2fs_inode_has_valid_blocks(&dp->inode))
- pctx.errcode = ext2fs_block_iterate2(fs, ino, 0, block_buf,
+ pctx.errcode = ext2fs_block_iterate3(fs, ino, 0, block_buf,
clone_file_block, &cs);
ext2fs_mark_bb_dirty(fs);
if (pctx.errcode) {
@@ -760,10 +767,12 @@
}
/* The inode may have changed on disk, so we have to re-read it */
e2fsck_read_inode(ctx, ino, &dp->inode, "clone file EA");
- blk = dp->inode.i_file_acl;
- if (blk && (clone_file_block(fs, &dp->inode.i_file_acl,
+ blk = ext2fs_file_acl_block(&dp->inode);
+ new_blk = blk;
+ if (blk && (clone_file_block(fs, &new_blk,
BLOCK_COUNT_EXTATTR, 0, 0, &cs) ==
BLOCK_CHANGED)) {
+ ext2fs_file_acl_block_set(&dp->inode, new_blk);
e2fsck_write_inode(ctx, ino, &dp->inode, "clone file EA");
/*
* If we cloned the EA block, find all other inodes
@@ -774,7 +783,7 @@
if (!n) {
com_err("clone_file", 0,
_("internal error: couldn't lookup EA "
- "block record for %u"), blk);
+ "block record for %llu"), blk);
retval = 0; /* OK to stumble on... */
goto errout;
}
@@ -792,8 +801,9 @@
goto errout;
}
di = (struct dup_inode *) dnode_get(n);
- if (di->inode.i_file_acl == blk) {
- di->inode.i_file_acl = dp->inode.i_file_acl;
+ if (ext2fs_file_acl_block(&di->inode) == blk) {
+ ext2fs_file_acl_block_set(&di->inode,
+ ext2fs_file_acl_block(&dp->inode));
e2fsck_write_inode(ctx, ino_el->inode,
&di->inode, "clone file EA");
decrement_badcount(ctx, blk, db);
diff --git a/e2fsck/pass2.c b/e2fsck/pass2.c
index b7ce352..51b7fad 100644
--- a/e2fsck/pass2.c
+++ b/e2fsck/pass2.c
@@ -1193,26 +1193,26 @@
e2fsck_read_bitmaps(ctx);
ext2fs_inode_alloc_stats2(fs, ino, -1, LINUX_S_ISDIR(inode.i_mode));
- if (inode.i_file_acl &&
+ if (ext2fs_file_acl_block(&inode) &&
(fs->super->s_feature_compat & EXT2_FEATURE_COMPAT_EXT_ATTR)) {
- pctx.errcode = ext2fs_adjust_ea_refcount(fs, inode.i_file_acl,
+ pctx.errcode = ext2fs_adjust_ea_refcount(fs, ext2fs_file_acl_block(&inode),
block_buf, -1, &count);
if (pctx.errcode == EXT2_ET_BAD_EA_BLOCK_NUM) {
pctx.errcode = 0;
count = 1;
}
if (pctx.errcode) {
- pctx.blk = inode.i_file_acl;
+ pctx.blk = ext2fs_file_acl_block(&inode);
fix_problem(ctx, PR_2_ADJ_EA_REFCOUNT, &pctx);
ctx->flags |= E2F_FLAG_ABORT;
return;
}
if (count == 0) {
ext2fs_unmark_block_bitmap2(ctx->block_found_map,
- inode.i_file_acl);
- ext2fs_block_alloc_stats(fs, inode.i_file_acl, -1);
+ ext2fs_file_acl_block(&inode));
+ ext2fs_block_alloc_stats(fs, ext2fs_file_acl_block(&inode), -1);
}
- inode.i_file_acl = 0;
+ ext2fs_file_acl_block_set(&inode, 0);
}
if (!ext2fs_inode_has_valid_blocks(&inode))
@@ -1264,10 +1264,10 @@
pctx.dir = dir;
pctx.inode = &inode;
- if (inode.i_file_acl &&
+ if (ext2fs_file_acl_block(&inode) &&
!(fs->super->s_feature_compat & EXT2_FEATURE_COMPAT_EXT_ATTR)) {
if (fix_problem(ctx, PR_2_FILE_ACL_ZERO, &pctx)) {
- inode.i_file_acl = 0;
+ ext2fs_file_acl_block_set(&inode, 0);
inode_modified++;
} else
not_fixed++;
@@ -1363,11 +1363,11 @@
not_fixed++;
}
- if (inode.i_file_acl &&
- ((inode.i_file_acl < fs->super->s_first_data_block) ||
- (inode.i_file_acl >= fs->super->s_blocks_count))) {
+ if (ext2fs_file_acl_block(&inode) &&
+ ((ext2fs_file_acl_block(&inode) < fs->super->s_first_data_block) ||
+ (ext2fs_file_acl_block(&inode) >= fs->super->s_blocks_count))) {
if (fix_problem(ctx, PR_2_FILE_ACL_BAD, &pctx)) {
- inode.i_file_acl = 0;
+ ext2fs_file_acl_block_set(&inode, 0);
inode_modified++;
} else
not_fixed++;
diff --git a/e2fsck/super.c b/e2fsck/super.c
index 2d5d661..6b08688 100644
--- a/e2fsck/super.c
+++ b/e2fsck/super.c
@@ -197,8 +197,8 @@
if (pb.truncated_blocks)
ext2fs_iblk_sub_blocks(fs, inode, pb.truncated_blocks);
- if (inode->i_file_acl) {
- retval = ext2fs_adjust_ea_refcount(fs, inode->i_file_acl,
+ if (ext2fs_file_acl_block(inode)) {
+ retval = ext2fs_adjust_ea_refcount(fs, ext2fs_file_acl_block(inode),
block_buf, -1, &count);
if (retval == EXT2_ET_BAD_EA_BLOCK_NUM) {
retval = 0;
@@ -211,8 +211,10 @@
return 1;
}
if (count == 0)
- ext2fs_block_alloc_stats(fs, inode->i_file_acl, -1);
- inode->i_file_acl = 0;
+ ext2fs_block_alloc_stats(fs,
+ ext2fs_file_acl_block(inode),
+ -1);
+ ext2fs_file_acl_block_set(inode, 0);
}
return 0;
}
diff --git a/lib/ext2fs/ext2_fs.h b/lib/ext2fs/ext2_fs.h
index 8b85d6c..114b001 100644
--- a/lib/ext2fs/ext2_fs.h
+++ b/lib/ext2fs/ext2_fs.h
@@ -346,7 +346,7 @@
__u32 i_block[EXT2_N_BLOCKS];/* Pointers to blocks */
__u32 i_generation; /* File version (for NFS) */
__u32 i_file_acl; /* File ACL */
- __u32 i_dir_acl; /* Directory ACL */
+ __u32 i_size_high; /* Formerly i_dir_acl, directory ACL */
__u32 i_faddr; /* Fragment address */
union {
struct {
@@ -393,7 +393,7 @@
__u32 i_block[EXT2_N_BLOCKS];/* Pointers to blocks */
__u32 i_generation; /* File version (for NFS) */
__u32 i_file_acl; /* File ACL */
- __u32 i_dir_acl; /* Directory ACL */
+ __u32 i_size_high; /* Formerly i_dir_acl, directory ACL */
__u32 i_faddr; /* Fragment address */
union {
struct {
@@ -422,7 +422,7 @@
__u32 i_version_hi; /* high 32 bits for 64-bit version */
};
-#define i_size_high i_dir_acl
+#define i_dir_acl i_size_high
#if defined(__KERNEL__) || defined(__linux__)
#define i_reserved1 osd1.linux1.l_i_reserved1
diff --git a/lib/ext2fs/valid_blk.c b/lib/ext2fs/valid_blk.c
index d0367e7..6a2aa47 100644
--- a/lib/ext2fs/valid_blk.c
+++ b/lib/ext2fs/valid_blk.c
@@ -39,7 +39,7 @@
* target is stored in the block entries.
*/
if (LINUX_S_ISLNK (inode->i_mode)) {
- if (inode->i_file_acl == 0) {
+ if (ext2fs_file_acl_block(inode) == 0) {
/* With no EA block, we can rely on i_blocks */
if (inode->i_blocks == 0)
return 0;
diff --git a/misc/e2image.c b/misc/e2image.c
index 24b9a1e..abbf641 100644
--- a/misc/e2image.c
+++ b/misc/e2image.c
@@ -511,9 +511,9 @@
break;
if (!inode.i_links_count)
continue;
- if (inode.i_file_acl) {
+ if (ext2fs_file_acl_block(&inode)) {
ext2fs_mark_block_bitmap2(meta_block_map,
- inode.i_file_acl);
+ ext2fs_file_acl_block(&inode));
}
if (!ext2fs_inode_has_valid_blocks(&inode))
continue;
diff --git a/misc/tune2fs.c b/misc/tune2fs.c
index 0911eb5..6da5c07 100644
--- a/misc/tune2fs.c
+++ b/misc/tune2fs.c
@@ -1185,13 +1185,14 @@
* Do we need to fix this ??
*/
- if (inode.i_file_acl &&
- ext2fs_test_block_bitmap2(bmap, inode.i_file_acl)) {
- blk = translate_block(inode.i_file_acl);
+ if (ext2fs_file_acl_block(&inode) &&
+ ext2fs_test_block_bitmap2(bmap,
+ ext2fs_file_acl_block(&inode))) {
+ blk = translate_block(ext2fs_file_acl_block(&inode));
if (!blk)
continue;
- inode.i_file_acl = blk;
+ ext2fs_file_acl_block_set(&inode, blk);
/*
* Write the inode to disk so that inode table
diff --git a/resize/resize2fs.c b/resize/resize2fs.c
index dc35001..2883af5 100644
--- a/resize/resize2fs.c
+++ b/resize/resize2fs.c
@@ -1375,11 +1375,11 @@
pb.is_dir = LINUX_S_ISDIR(inode->i_mode);
pb.changed = 0;
- if (inode->i_file_acl && rfs->bmap) {
+ if (ext2fs_file_acl_block(inode) && rfs->bmap) {
new_block = ext2fs_extent_translate(rfs->bmap,
- inode->i_file_acl);
+ ext2fs_file_acl_block(inode));
if (new_block) {
- inode->i_file_acl = new_block;
+ ext2fs_file_acl_block_set(inode, new_block);
retval = ext2fs_write_inode_full(rfs->old_fs,
ino, inode, inode_size);
if (retval) goto errout;
diff --git a/tests/f_bad_disconnected_inode/expect.1 b/tests/f_bad_disconnected_inode/expect.1
index 11862f6..1b233e7 100644
--- a/tests/f_bad_disconnected_inode/expect.1
+++ b/tests/f_bad_disconnected_inode/expect.1
@@ -15,13 +15,13 @@
Pass 4: Checking reference counts
Inode 2 ref count is 4, should be 3. Fix? yes
-i_file_acl for inode 13 (...) is 4218798225, should be zero.
+i_file_acl for inode 13 (...) is 239964041625745, should be zero.
Clear? yes
Inode 13 (...) has invalid mode (0117003).
Clear? yes
-i_file_acl for inode 14 (...) is 2892851642, should be zero.
+i_file_acl for inode 14 (...) is 178759431711162, should be zero.
Clear? yes
Inode 14 (...) has invalid mode (0154247).