diff options
| author | Alan Modra <amodra@gmail.com> | 2022-05-20 14:59:05 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2022-05-20 16:11:25 +0930 |
| commit | bd7d326debef642c0a98f859807d794d8d68afc7 (patch) | |
| tree | e590bc7d3c850d6742191d435ada233a4631e50c /bfd/elflink.c | |
| parent | ef5d515048ba13a06815c8fccc420e6e8098da03 (diff) | |
| download | gdb-bd7d326debef642c0a98f859807d794d8d68afc7.zip gdb-bd7d326debef642c0a98f859807d794d8d68afc7.tar.gz gdb-bd7d326debef642c0a98f859807d794d8d68afc7.tar.bz2 | |
Tidy warn-execstack handling
Make ld and bfd values consistent by swapping values 0 and 2 in
link_info.warn_execstack. This has the benefit of making the value an
"extended" boolean, with 0 meaning no warning, 1 meaning warn, other
values a conditional warning.
Yes, this patch introduces fails on arm/aarch64. Not a problem with
this patch but an arm/aarch64 before_parse problem.
bfd/
* elflink.c (bfd_elf_size_dynamic_sections): Adjust
warn_execstack test.
include/
* bfdlink.h (warn_execstack): Swap 0 and 2 meaning.
ld/
* configure.ac (DEFAULT_LD_WARN_EXECSTACK): Use values of 0,
1, 2 consistent with link_info.warn_execstack.
* ld.texi: Typo fixes.
* lexsup.c (parse_args): Adjust setting of link_info.warn_execstack.
(elf_static_list_options): Adjust help message conditions.
* configure: Regenerate.
Diffstat (limited to 'bfd/elflink.c')
| -rw-r--r-- | bfd/elflink.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bfd/elflink.c b/bfd/elflink.c index 4d6fe66..96eb36a 100644 --- a/bfd/elflink.c +++ b/bfd/elflink.c @@ -7179,7 +7179,7 @@ warning: enabling an executable stack because of -z execstack command line optio { if (exec) { - if (info->warn_execstack != 2) + if (info->warn_execstack != 0) { /* PR 29072: Because an executable stack is a serious security risk, make sure that the user knows that it is |