Re: [PATCH 1/5] Compiler Attributes: Add __alloc_size() for better bounds checking

From: Kees Cook
Date: Wed Aug 18 2021 - 13:58:47 EST


On Wed, Aug 18, 2021 at 03:07:48PM +0200, Miguel Ojeda wrote:
> On Wed, Aug 18, 2021 at 7:08 AM Kees Cook <keescook@xxxxxxxxxxxx> wrote:
> >
> > Clang can additionally use alloc_size to informt the results of
>
> Typo.
>
> > Additionally disables -Wno-alloc-size-larger-than since the allocators
>
> Disables -Walloc-size-larger-than?
>
> > already reject SIZE_MAX, and the compile-time warnings aren't helpful.
>
> Perhaps a bit more context here (and/or in the comment in the
> Makefile) would be nice: i.e. why are they not helpful (even if
> rejected by the allocators).

Thanks for the review! I'll get this all fixed for v2.

--
Kees Cook