diff options
| author | Alan Modra <amodra@gmail.com> | 2019-08-29 09:04:08 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2019-08-29 09:04:35 +0930 |
| commit | a85eba51f619d3e8c813a12871172f23689b28d6 (patch) | |
| tree | 2e1ecfbaf00869124544c05344e2d07d201a4618 /binutils/ChangeLog | |
| parent | 83cf0d04dcdc9a4e7f105a517390e4c1af444340 (diff) | |
| download | gdb-a85eba51f619d3e8c813a12871172f23689b28d6.zip gdb-a85eba51f619d3e8c813a12871172f23689b28d6.tar.gz gdb-a85eba51f619d3e8c813a12871172f23689b28d6.tar.bz2 | |
Tidy check_uvalue
I don't see a need to calculate "ptr = start + uvalue" then compare
"ptr" with "start" and "end". Given "start <= end" on entry, the
"uvalue" comparison with "max_uvalue" ought to be sufficient to ensure
"start + uvalue" is bounded by "start" and "end" regardless of the
size of pointers and the unsigned dwarf_vma integer type.
* dwarf.c (check_uvalue): Remove unnecessary pointer checks.
Diffstat (limited to 'binutils/ChangeLog')
| -rw-r--r-- | binutils/ChangeLog | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 1148e9d..98e660d 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,7 @@ +2019-08-29 Alan Modra <amodra@gmail.com> + + * dwarf.c (check_uvalue): Remove unnecessary pointer checks. + 2019-08-28 Niklas Gürtler <profclonk@gmail.com> PR 24942 |