Re: [PATCH v2 2/5] treewide: Replace open-coded flex arrays in unions

From: Kees Cook
Date: Thu Aug 26 2021 - 11:39:58 EST




On August 26, 2021 12:36:27 AM PDT, Vincent MAILHOL <mailhol.vincent@xxxxxxxxxx> wrote:
>On Thu. 26 Aug 2021 at 14:04, Kees Cook <keescook@xxxxxxxxxxxx> wrote:
>> In support of enabling -Warray-bounds and -Wzero-length-bounds and
>> correctly handling run-time memcpy() bounds checking, replace all
>> open-coded flexible arrays (i.e. 0-element arrays) in unions with the
>> flex_array() helper macro.
>
>Nitpick: the commit description says flex_array() but the code is
>actually DECLARE_FLEX_ARRAY() or __DECLARE_FLEX_ARRAY().

Ah yes, thanks! I had renamed the macro but missed these references. Now fixed!

-Kees