[[ this was already added to 3.4-stable, I am sending it to the stable 
mailing list for 3.2, 3.0 and others stable kernels ]]

This is backport of upstream commit 
bbd465df73f0d8ba41b8a0732766a243d0f5b356, modified for stable kernels 
2.6.39 - 3.7.

    This patch fixes warnings due to missing lock on write error path.

      WARNING: at fs/hpfs/hpfs_fn.h:353 hpfs_truncate+0x75/0x80 [hpfs]()
      Hardware name: empty
      Pid: 26563, comm: dd Tainted: P           O 3.9.4 #12
      Call Trace:
        hpfs_truncate+0x75/0x80 [hpfs]
        hpfs_write_begin+0x84/0x90 [hpfs]
        _hpfs_bmap+0x10/0x10 [hpfs]
        generic_file_buffered_write+0x121/0x2c0
        __generic_file_aio_write+0x1c7/0x3f0
        generic_file_aio_write+0x7c/0x100
        do_sync_write+0x98/0xd0
        hpfs_file_write+0xd/0x50 [hpfs]
        vfs_write+0xa2/0x160
        sys_write+0x51/0xa0
        page_fault+0x22/0x30
        system_call_fastpath+0x1a/0x1f

Signed-off-by: Mikulas Patocka <[email protected]>

---
 fs/hpfs/file.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Index: linux-3.4.66/fs/hpfs/file.c
===================================================================
--- linux-3.4.66.orig/fs/hpfs/file.c    2013-11-30 01:16:17.000000000 +0100
+++ linux-3.4.66/fs/hpfs/file.c 2013-11-30 01:19:16.000000000 +0100
@@ -116,9 +116,12 @@ static int hpfs_write_begin(struct file
                                hpfs_get_block,
                                &hpfs_i(mapping->host)->mmu_private);
        if (unlikely(ret)) {
-               loff_t isize = mapping->host->i_size;
+               loff_t isize;
+               hpfs_lock(mapping->host->i_sb);
+               isize = mapping->host->i_size;
                if (pos + len > isize)
                        vmtruncate(mapping->host, isize);
+               hpfs_unlock(mapping->host->i_sb);
        }
 
        return ret;
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to