On Sun, Sep 16, 2012 at 05:15:44PM +0000, Christos Zoulas wrote:
> In article <[email protected]>,
> Manuel Bouyer  <[email protected]> wrote:
> >-=-=-=-=-=-
> >
> >On Sun, Sep 16, 2012 at 03:23:22PM +0200, Manuel Bouyer wrote:
> >> Hello,
> >> the attached patch adds a pass-through ioctl interface, with the
> >
> >As several of you noctied I forgot (once again) to attach the patch.
> >Here it is.
> 
> As thor mentioned, the commands and arguments need sanity checking.
> For example the malloc'ed len is unchecked and comes from userland.

I'll have a look

> I think that a better way to multiplex the ioctl is to determine
> what device you have from the fd.

I agree, but I don't know how to do this (is there a better way than
hardcoding mfi's major number in compat_linux), can you give details on how
you would do this ?

-- 
Manuel Bouyer <[email protected]>
     NetBSD: 26 ans d'experience feront toujours la difference
--

Reply via email to