Re: [PATCH 3/4] [MTD] compat_ioctl cleanup

From: Arnd Bergmann
Date: Fri May 29 2009 - 11:02:48 EST


On Friday 29 May 2009, David Woodhouse wrote:
> On Wed, 2009-04-08 at 22:53 -0700, Kevin Cernekee wrote:
> >
> > +static long mtd_compat_ioctl(struct file *file, unsigned int cmd,
> > + unsigned long arg)
> > +{
> > + struct mtd_file_info *mfi = file->private_data;
> > + struct mtd_info *mtd = mfi->mtd;
> > + void __user *argp = (void __user *)arg;
>
> Shouldn't that be 'argp = compat_ptr(arg)' ?

Yes, that's right. I missed this when I looked at the patches.

Arnd <><
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/