On Thu, Sep 22, 2022 at 11:23:46AM +0200, Miguel Ojeda wrote:
On Thu, Sep 22, 2022 at 5:10 AM Kees Cook keescook@chromium.org wrote:
-#ifdef __alloc_size__ -# define __alloc_size(x, ...) __alloc_size__(x, ## __VA_ARGS__) __malloc -#else -# define __alloc_size(x, ...) __malloc -#endif +#define __alloc_size(x, ...) __alloc_size__(x, ## __VA_ARGS__) __malloc +#define __realloc_size(x, ...) __alloc_size__(x, ## __VA_ARGS__)
These look unconditional now, so we could move it to `compiler_attributes.h` in a later patch (or an independent series).
I wasn't sure if this "composite macro" was sane there, especially since it would be using __malloc before it was defined, etc. Would you prefer I move it?