Commit d71f2236 authored by Gustavo A. R. Silva's avatar Gustavo A. R. Silva Committed by Kees Cook

gcc-plugins: randstruct: Update code comment in relayout_struct()

Update code comment to clarify that the only element whose layout is
not randomized is a proper C99 flexible-array member. This update is
complementary to commit 1ee60356 ("gcc-plugins: randstruct: Only
warn about true flexible arrays")
Signed-off-by: default avatar"Gustavo A. R. Silva" <gustavoars@kernel.org>
Link: https://lore.kernel.org/r/ZWJr2MWDjXLHE8ap@work
Fixes: 1ee60356 ("gcc-plugins: randstruct: Only warn about true flexible arrays")
Signed-off-by: default avatarKees Cook <keescook@chromium.org>
parent 4e86f32a
...@@ -339,8 +339,7 @@ static int relayout_struct(tree type) ...@@ -339,8 +339,7 @@ static int relayout_struct(tree type)
/* /*
* enforce that we don't randomize the layout of the last * enforce that we don't randomize the layout of the last
* element of a struct if it's a 0 or 1-length array * element of a struct if it's a proper flexible array
* or a proper flexible array
*/ */
if (is_flexible_array(newtree[num_fields - 1])) { if (is_flexible_array(newtree[num_fields - 1])) {
has_flexarray = true; has_flexarray = true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment