From: Maxim Patlasov Date: Wed, 2 Oct 2013 11:01:07 +0000 (+0400) Subject: fuse: writepages: roll back changes if request not found X-Git-Tag: next-20131017~85^2~3 X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=7047a8af8ecac2619162f84d504c9ee39683f124;p=karo-tx-linux.git fuse: writepages: roll back changes if request not found fuse_writepage_in_flight() returns false if it fails to find request with given index in fi->writepages. Then the caller proceeds with populating data->orig_pages[] and incrementing req->num_pages. Hence, fuse_writepage_in_flight() must revert changes it made in request before returning false. Signed-off-by: Maxim Patlasov Signed-off-by: Miklos Szeredi --- diff --git a/fs/fuse/file.c b/fs/fuse/file.c index 135360e2b9e5..077b03844998 100644 --- a/fs/fuse/file.c +++ b/fs/fuse/file.c @@ -1649,7 +1649,6 @@ static bool fuse_writepage_in_flight(struct fuse_req *new_req, spin_lock(&fc->lock); list_del(&new_req->writepages_entry); - new_req->num_pages = 1; list_for_each_entry(old_req, &fi->writepages, writepages_entry) { BUG_ON(old_req->inode != new_req->inode); curr_index = old_req->misc.write.in.offset >> PAGE_CACHE_SHIFT; @@ -1659,9 +1658,12 @@ static bool fuse_writepage_in_flight(struct fuse_req *new_req, break; } } - if (!found) + if (!found) { + list_add(&new_req->writepages_entry, &fi->writepages); goto out_unlock; + } + new_req->num_pages = 1; for (tmp = old_req; tmp != NULL; tmp = tmp->misc.write.next) { BUG_ON(tmp->inode != new_req->inode); curr_index = tmp->misc.write.in.offset >> PAGE_CACHE_SHIFT;