diff options
author | Dan Carpenter <error27@gmail.com> | 2010-06-03 12:35:42 +0200 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2010-06-04 17:16:28 -0400 |
commit | 5b54470daded19d83ea2bbf5f6bc12662942cd63 (patch) | |
tree | 93ac73e7eba1231c863903cd814d12f4cb935b1e /fs/fcntl.c | |
parent | 7d683a09990ff095a91b6e724ecee0ff8733274a (diff) | |
download | kernel_goldelico_gta04-5b54470daded19d83ea2bbf5f6bc12662942cd63.zip kernel_goldelico_gta04-5b54470daded19d83ea2bbf5f6bc12662942cd63.tar.gz kernel_goldelico_gta04-5b54470daded19d83ea2bbf5f6bc12662942cd63.tar.bz2 |
fcntl: return -EFAULT if copy_to_user fails
copy_to_user() returns the number of bytes remaining, but we want to
return -EFAULT.
ret = fcntl(fd, F_SETOWN_EX, NULL);
With the original code ret would be 8 here.
V2: Takuya Yoshikawa pointed out a similar issue in f_getown_ex()
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/fcntl.c')
-rw-r--r-- | fs/fcntl.c | 7 |
1 files changed, 5 insertions, 2 deletions
@@ -274,7 +274,7 @@ static int f_setown_ex(struct file *filp, unsigned long arg) ret = copy_from_user(&owner, owner_p, sizeof(owner)); if (ret) - return ret; + return -EFAULT; switch (owner.type) { case F_OWNER_TID: @@ -332,8 +332,11 @@ static int f_getown_ex(struct file *filp, unsigned long arg) } read_unlock(&filp->f_owner.lock); - if (!ret) + if (!ret) { ret = copy_to_user(owner_p, &owner, sizeof(owner)); + if (ret) + ret = -EFAULT; + } return ret; } |