From: Anand V. Avati <avati@gluster.com>
Date: Thu, 22 Oct 2009 13:24:52 +0000 (-0700)
Subject: fuse: prevent fuse_put_request on invalid pointer
X-Git-Tag: upstream/snapshot3+hdmi~16652^2~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f60311d5f7670d9539b424e4ed8b5c0872fc9e83;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

fuse: prevent fuse_put_request on invalid pointer

fuse_direct_io() has a loop where requests are allocated in each
iteration. if allocation fails, the loop is broken out and follows
into an unconditional fuse_put_request() on that invalid pointer.

Signed-off-by: Anand V. Avati <avati@gluster.com>
Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
Cc: stable@kernel.org
---

diff --git a/fs/fuse/file.c b/fs/fuse/file.c
index a3492f7..5887a63 100644
--- a/fs/fuse/file.c
+++ b/fs/fuse/file.c
@@ -1063,7 +1063,8 @@ ssize_t fuse_direct_io(struct file *file, const char __user *buf,
 				break;
 		}
 	}
-	fuse_put_request(fc, req);
+	if (!IS_ERR(req))
+		fuse_put_request(fc, req);
 	if (res > 0)
 		*ppos = pos;