diff options
| author | Gustavo A. R. Silva <gustavoars@kernel.org> | 2023-11-25 15:49:12 -0600 |
|---|---|---|
| committer | Kees Cook <keescook@chromium.org> | 2023-11-27 16:30:05 -0800 |
| commit | d71f22365a9caca82d424f3a33445de46567e198 (patch) | |
| tree | 213b7b9bef4fd6e660eaa90b13e410d8b8862d38 /scripts | |
| parent | 4e86f32a13af1970d21be94f659cae56bbe487ee (diff) | |
| download | lwn-d71f22365a9caca82d424f3a33445de46567e198.tar.gz lwn-d71f22365a9caca82d424f3a33445de46567e198.zip | |
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 1ee60356c2dc ("gcc-plugins: randstruct: Only
warn about true flexible arrays")
Signed-off-by: "Gustavo A. R. Silva" <gustavoars@kernel.org>
Link: https://lore.kernel.org/r/ZWJr2MWDjXLHE8ap@work
Fixes: 1ee60356c2dc ("gcc-plugins: randstruct: Only warn about true flexible arrays")
Signed-off-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/gcc-plugins/randomize_layout_plugin.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/scripts/gcc-plugins/randomize_layout_plugin.c b/scripts/gcc-plugins/randomize_layout_plugin.c index 910bd21d08f4..746ff2d272f2 100644 --- a/scripts/gcc-plugins/randomize_layout_plugin.c +++ b/scripts/gcc-plugins/randomize_layout_plugin.c @@ -339,8 +339,7 @@ static int relayout_struct(tree type) /* * enforce that we don't randomize the layout of the last - * element of a struct if it's a 0 or 1-length array - * or a proper flexible array + * element of a struct if it's a proper flexible array */ if (is_flexible_array(newtree[num_fields - 1])) { has_flexarray = true; |