diff options
| author | Nick Piggin <npiggin@suse.de> | 2007-02-20 21:57:54 (GMT) |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-02-21 01:10:13 (GMT) |
| commit | ffda9d302267dbb7fc9bc38f6e4c1b3d61a536a9 (patch) | |
| tree | d3b95c0b7dc2edc602e71e2ff8389de79ea243ac /fs/buffer.c | |
| parent | bc56bba8f31bd99f350a5ebfd43d50f411b620c7 (diff) | |
| download | linux-fsl-qoriq-ffda9d302267dbb7fc9bc38f6e4c1b3d61a536a9.tar.xz | |
[PATCH] fs: fix __block_write_full_page error case buffer submission
Andrew noticed that unlocking the page before submitting all buffers for
writeout could cause problems if the IO completes before we've finished
messing around with the page buffers, and they subsequently get freed.
Even if there were no bug, it is a good idea to bring the error case
into line with the common case here.
Signed-off-by: Nick Piggin <npiggin@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/buffer.c')
| -rw-r--r-- | fs/buffer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/buffer.c b/fs/buffer.c index f99c509..e95fe4f 100644 --- a/fs/buffer.c +++ b/fs/buffer.c @@ -1743,7 +1743,6 @@ recover: SetPageError(page); BUG_ON(PageWriteback(page)); set_page_writeback(page); - unlock_page(page); do { struct buffer_head *next = bh->b_this_page; if (buffer_async_write(bh)) { @@ -1753,6 +1752,7 @@ recover: } bh = next; } while (bh != head); + unlock_page(page); goto done; } |