]> git.baikalelectronics.ru Git - kernel.git/commit
udf: Fix file corruption when appending just after end of preallocated extent
authorJan Kara <jack@suse.cz>
Mon, 23 Jan 2023 13:18:47 +0000 (14:18 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 10 Mar 2023 08:34:10 +0000 (09:34 +0100)
commit4a6a91a28dd1da8b162f41624e266378090ce6f9
treebd3820ae2e67f95a14711b896dfc94f6afefc166
parentca2ba28af90ff8c755fc26831d3afd05844d6a21
udf: Fix file corruption when appending just after end of preallocated extent

commit 91915c96778d7b5c6c284729c1aca981016c9255 upstream.

When we append new block just after the end of preallocated extent, the
code in inode_getblk() wrongly determined we're going to use the
preallocated extent which resulted in adding block into a wrong logical
offset in the file. Sequence like this manifests it:

xfs_io -f -c "pwrite 0x2cacf 0xd122" -c "truncate 0x2dd6f" \
  -c "pwrite 0x27fd9 0x69a9" -c "pwrite 0x32981 0x7244" <file>

The code that determined the use of preallocated extent is actually
stale because udf_do_extend_file() does not create preallocation anymore
so after calling that function we are sure there's no usable
preallocation. Just remove the faulty condition.

CC: stable@vger.kernel.org
Fixes: 189b02ac597c ("udf: Discard preallocation before extending file with a hole")
Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/udf/inode.c