lkml.org 
[lkml]   [2013]   [Dec]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] fs/nilfs2: Integer overflow in nilfs_ioctl_wrap_copy()
From
Date
Hi,

On Sat, 2013-12-28 at 10:28 +0800, Wenliang Fan wrote:
> The local variable 'pos' comes from userspace. If a large number was
> passed, there would be an integer overflow in the following line:
> pos += n;
>
> Signed-off-by: Wenliang Fan <fanwlexca@gmail.com>
> ---
> fs/nilfs2/ioctl.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/fs/nilfs2/ioctl.c b/fs/nilfs2/ioctl.c
> index b44bdb2..a260a98 100644
> --- a/fs/nilfs2/ioctl.c
> +++ b/fs/nilfs2/ioctl.c
> @@ -65,6 +65,8 @@ static int nilfs_ioctl_wrap_copy(struct the_nilfs *nilfs,
> ret = 0;
> total = 0;
> pos = argv->v_index;
> + if (pos > ULONG_MAX - argv->v_nmembs)
> + return -EINVAL;

I am slightly confused. Does this patch means that you refused previous
one? Or this patch is addition to the previous one?

You need to prepare next version of the patch or to prepare patch set
from two patches. And it makes sense to provide changelog of your work,
I think.

Thanks,
Vyacheslav Dubeyko.

> for (i = 0; i < argv->v_nmembs; i += n) {
> n = (argv->v_nmembs - i < maxmembs) ?
> argv->v_nmembs - i : maxmembs;




\
 
 \ /
  Last update: 2013-12-30 08:01    [W:0.046 / U:0.396 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site