diff options
| author | Alan Modra <amodra@gmail.com> | 2022-08-06 14:57:27 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2022-08-06 18:43:24 +0930 |
| commit | 578a7392c33c7c7cde5559278e519c75047318c9 (patch) | |
| tree | a2c254b92055bfbfd070f2d60cb9d14ab6d6d35b /bfd/section.c | |
| parent | 77b38f6db98e046232ecaac11b61c453965fb75a (diff) | |
| download | binutils-gdb-578a7392c33c7c7cde5559278e519c75047318c9.tar.gz | |
objcopy section alignment
bfd_set_section_alignment currently always returns true. This patch
changes it to return false on silly alignment values, avoiding yet
another way to trigger ubsan errors like coffcode.h:3192:12: runtime
error: shift exponent 299 is too large for 32-bit type 'int'. We'll
catch that one in objcopy.c:setup_sections. However, setup_sections
gives up on other setup operations that are necessary even after an
error of some sort. Change that to keep going, which might change the
error message but that shouldn't matter in the least.
bfd/
* section.c (bfd_set_section_alignment): Return false and
don't set alignment_power for stupidly large alignments.
* bfd-in2.h: Regenerate.
* coffcode.h (coff_compute_section_file_positions): Don't use
an int constant when calculating alignment.
binutils/
* objcopy.c (setup_section): Keep on going after hitting
non-fatal errors.
Diffstat (limited to 'bfd/section.c')
| -rw-r--r-- | bfd/section.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/bfd/section.c b/bfd/section.c index 5a487ce6c6f..c7a02d729f2 100644 --- a/bfd/section.c +++ b/bfd/section.c @@ -631,6 +631,8 @@ CODE_FRAGMENT .static inline bool .bfd_set_section_alignment (asection *sec, unsigned int val) .{ +. if (val >= sizeof (bfd_vma) * 8 - 1) +. return false; . sec->alignment_power = val; . return true; .} |