[lkml]   [2010]   [Aug]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] ARM: uaccess: Implement strict user copy checks
    On Wednesday 04 August 2010, Stephen Boyd wrote:
    > + bool "Strict user copy size checks"
    > + depends on DEBUG_KERNEL
    > + help
    > + Enabling this option turns a certain set of sanity checks for user
    > + copy operations into compile time errors.
    > +
    > + The copy_from_user() etc checks are there to help test if there
    > + are sufficient security checks on the length argument of
    > + the copy operation, by having gcc prove that the argument is
    > + within bounds.
    > +
    > + If unsure, or if you run an older (pre 4.4) gcc, say N.
    > +

    Do you actually need to disable this if running an older gcc? AFAICT, it
    should just have no effect at all in that case, so the comment is slightly

    Also, why turn this specific warning into an error but not any of the other
    warnings? Some architectures (alpha, sparc, mips, powerpc, sh) simply turn
    on -Werror for architecture specific code in general, which seems very
    useful. We can also make that a config option (probably arch independent)
    that we turn on for defconfig files that we know build without warnings.

    Unfortunately, there is a number of device drivers that have never been
    warning-free, so we can't just enable -Werror for all code.


     \ /
      Last update: 2010-08-11 05:07    [W:0.020 / U:4.868 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site