]> git.baikalelectronics.ru Git - kernel.git/commit
udf: Fix race between write(2) and close(2)
authorJan Kara <jack@suse.cz>
Tue, 9 Sep 2014 11:03:03 +0000 (13:03 +0200)
committerJan Kara <jack@suse.cz>
Wed, 17 Sep 2014 09:59:11 +0000 (11:59 +0200)
commitcc34f69ad3f464ac9e3deaddf64b308586ae6e65
treeb3e110ddf9b2e35e92f586418005a0fc0002f9a4
parent8677373a5a4328d8d42c833a0cbe25246d4b2b05
udf: Fix race between write(2) and close(2)

Currently write(2) updating i_size and close(2) of the file can race in
such a way that udf_truncate_tail_extent() called from
udf_file_release() sees old i_size but already new extents added by the
running write call. This results in complaints like:
  UDF-fs: warning (device vdb2): udf_truncate_tail_extent: Too long extent
    after EOF in inode 877: i_size: 0 lbcount: 1073739776 extent 0+1073739776
  UDF-fs: error (device vdb2): udf_truncate_tail_extent: Extent after EOF
    in inode 877

Fix the problem by grabbing i_mutex in udf_file_release() to be sure
i_size is consistent with current state of extent list. Also avoid
truncating tail extent unnecessarily when the file is still open for
writing.

Signed-off-by: Jan Kara <jack@suse.cz>
fs/udf/file.c