lkml.org 
[lkml]   [2002]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: AUDIT: copy_from_user is a deathtrap.
Date
In message <E178gfl-0006Ip-00@the-village.bc.nu> you write:
> > > I would much rather fix these instances than add yet another
> > > interface.
> >
> > I'll accept that if someone's volunteering to audit the kernel for
> > them every six months.
> >
> > Sorry I wasn't clear: I'm saying *replace*, not add,
>
> Replace requires you audit every single use, and then work out how to
> handle those that do care about the length and the point it faulted.

Read my original post. I have done this.

> From what I've seen of the stuff that has been fixed we have a mix
> of the following
>
> 1. Misports of ancient verify_* code - eg the serial ones
> 2. Not checking the return code - 100% legal and standards compliant

No, the 400+ are all of form:

/* of course this returns 0 or -EFAULT! */
return copy_from_user(xxx);

Rusty.
--
Anyone who quotes me in their sig is an idiot. -- Rusty Russell.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:22    [W:1.344 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site