• Kees Cook's avatar
    lkdtm/heap: Hide allocation size from -Warray-bounds · f260fd59
    Kees Cook authored
    With the kmalloc() size annotations, GCC is smart enough to realize that
    LKDTM is intentionally writing past the end of the buffer. This is on
    purpose, of course, so hide the buffer from the optimizer. Silences:
    
    ../drivers/misc/lkdtm/heap.c: In function 'lkdtm_SLAB_LINEAR_OVERFLOW':
    ../drivers/misc/lkdtm/heap.c:59:13: warning: array subscript 256 is outside array bounds of 'void[1020]' [-Warray-bounds]
       59 |         data[1024 / sizeof(u32)] = 0x12345678;
          |         ~~~~^~~~~~~~~~~~~~~~~~~~
    In file included from ../drivers/misc/lkdtm/heap.c:7:
    In function 'kmalloc',
        inlined from 'lkdtm_SLAB_LINEAR_OVERFLOW' at ../drivers/misc/lkdtm/heap.c:54:14:
    ../include/linux/slab.h:581:24: note: at offset 1024 into object of size 1020 allocated by 'kmem_cache_alloc_trace'
      581 |                 return kmem_cache_alloc_trace(
          |                        ^~~~~~~~~~~~~~~~~~~~~~~
      582 |                                 kmalloc_caches[kmalloc_type(flags)][index],
          |                                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      583 |                                 flags, size);
          |                                 ~~~~~~~~~~~~
    
    Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    Cc: Arnd Bergmann <arnd@arndb.de>
    Signed-off-by: default avatarKees Cook <keescook@chromium.org>
    f260fd59
heap.c 8.09 KB