diff options
| author |   Andrew Waterman <andrew@sifive.com> | 2016-12-18 22:53:49 -0800 |
|---|---|---|
| committer |   Alan Modra <amodra@gmail.com> | 2016-12-20 12:26:33 +1030 |
| commit | 1d61f7949f77796ee407466f3ca7f42dcde9251b (patch) | |
| tree | 211118dfa45c495a60a49de0bde53c6973489c29 | |
| parent | Rework RISC-V relocations (diff) | |
| download | binutils-gdb-1d61f7949f77796ee407466f3ca7f42dcde9251b.tar.gz binutils-gdb-1d61f7949f77796ee407466f3ca7f42dcde9251b.tar.bz2 binutils-gdb-1d61f7949f77796ee407466f3ca7f42dcde9251b.zip | |
Fix an integer overflow in RISC-V relocation handling
* elfnn-riscv.c (bfd_riscv_get_max_alignment): Return bfd_vma
instead of unsigned int.
| -rw-r--r-- | bfd/ChangeLog | 5 | ||||
| -rw-r--r-- | bfd/elfnn-riscv.c | 4 |
2 files changed, 7 insertions, 2 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index b6bba2ac3e0..f2b1707e5a1 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,4 +1,9 @@ 2016-12-20 Andrew Waterman <andrew@sifive.com> + + * elfnn-riscv.c (bfd_riscv_get_max_alignment): Return bfd_vma + instead of unsigned int. + +2016-12-20 Andrew Waterman <andrew@sifive.com> Kuan-Lin Chen <kuanlinchentw@gmail.com> * reloc.c (BFD_RELOC_RISCV_TPREL_I): New relocation. diff --git a/bfd/elfnn-riscv.c b/bfd/elfnn-riscv.c index 51a2a10e03a..a9b0ac6be09 100644 --- a/bfd/elfnn-riscv.c +++ b/bfd/elfnn-riscv.c @@ -2759,7 +2759,7 @@ _bfd_riscv_relax_call (bfd *abfd, asection *sec, asection *sym_sec, /* Traverse all output sections and return the max alignment. */ -static unsigned int +static bfd_vma _bfd_riscv_get_max_alignment (asection *sec) { unsigned int max_alignment_power = 0; @@ -2771,7 +2771,7 @@ _bfd_riscv_get_max_alignment (asection *sec) max_alignment_power = o->alignment_power; } - return 1 << max_alignment_power; + return (bfd_vma) 1 << max_alignment_power; } /* Relax non-PIC global variable references. */ |