udf: fix silent AED tagLocation corruption
authorSteven J. Magnani <magnani@ieee.org>
Thu, 7 Jan 2021 23:41:16 +0000 (17:41 -0600)
committerJan Kara <jack@suse.cz>
Mon, 25 Jan 2021 17:09:25 +0000 (18:09 +0100)
commit63c9e47a1642fc817654a1bc18a6ec4bbcc0f056
treec474964d67997c72c15dee149fa4219c402a27b5
parent0a6dc67a6aa45f19bd4ff89b4f468fc50c4b8daa
udf: fix silent AED tagLocation corruption

When extending a file, udf_do_extend_file() may enter following empty
indirect extent. At the end of udf_do_extend_file() we revert prev_epos
to point to the last written extent. However if we end up not adding any
further extent in udf_do_extend_file(), the reverting points prev_epos
into the header area of the AED and following updates of the extents
(in udf_update_extents()) will corrupt the header.

Make sure that we do not follow indirect extent if we are not going to
add any more extents so that returning back to the last written extent
works correctly.

Link: https://lore.kernel.org/r/20210107234116.6190-2-magnani@ieee.org
Signed-off-by: Steven J. Magnani <magnani@ieee.org>
Signed-off-by: Jan Kara <jack@suse.cz>
fs/udf/inode.c