diff options
author | Maxim Patlasov <MPatlasov@parallels.com> | 2014-07-10 07:32:43 -0400 |
---|---|---|
committer | Miklos Szeredi <mszeredi@suse.cz> | 2014-07-14 10:17:57 -0400 |
commit | 27f1b36326bc8b6911e7052bc4b80a10234f0ec5 (patch) | |
tree | f1641fb962d5d99dd6f7b9903ca02b2486f35697 /fs | |
parent | 4237ba43b65aa989674c89fc4f2fe46eebc501ee (diff) |
fuse: release temporary page if fuse_writepage_locked() failed
tmp_page to be freed if fuse_write_file_get() returns NULL.
Signed-off-by: Maxim Patlasov <mpatlasov@parallels.com>
Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/fuse/file.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/fuse/file.c b/fs/fuse/file.c index 96d513e01a5d..35b6f31ecc38 100644 --- a/fs/fuse/file.c +++ b/fs/fuse/file.c | |||
@@ -1722,7 +1722,7 @@ static int fuse_writepage_locked(struct page *page) | |||
1722 | error = -EIO; | 1722 | error = -EIO; |
1723 | req->ff = fuse_write_file_get(fc, fi); | 1723 | req->ff = fuse_write_file_get(fc, fi); |
1724 | if (!req->ff) | 1724 | if (!req->ff) |
1725 | goto err_free; | 1725 | goto err_nofile; |
1726 | 1726 | ||
1727 | fuse_write_fill(req, req->ff, page_offset(page), 0); | 1727 | fuse_write_fill(req, req->ff, page_offset(page), 0); |
1728 | 1728 | ||
@@ -1750,6 +1750,8 @@ static int fuse_writepage_locked(struct page *page) | |||
1750 | 1750 | ||
1751 | return 0; | 1751 | return 0; |
1752 | 1752 | ||
1753 | err_nofile: | ||
1754 | __free_page(tmp_page); | ||
1753 | err_free: | 1755 | err_free: |
1754 | fuse_request_free(req); | 1756 | fuse_request_free(req); |
1755 | err: | 1757 | err: |