diff options
| author | Alan Modra <amodra@gmail.com> | 2017-10-05 17:32:18 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2017-10-05 17:43:34 +1030 |
| commit | 9c0f3d3f2017829ffd908c9893b85094985c3b58 (patch) | |
| tree | 5923acc679c5289d96b9e10c2feec203eb808c57 /binutils/ChangeLog | |
| parent | c91933e9e33654c7b8973cb449b93f5152afcbcb (diff) | |
| download | fsf-binutils-gdb-9c0f3d3f2017829ffd908c9893b85094985c3b58.zip fsf-binutils-gdb-9c0f3d3f2017829ffd908c9893b85094985c3b58.tar.gz fsf-binutils-gdb-9c0f3d3f2017829ffd908c9893b85094985c3b58.tar.bz2 | |
PR22239 - invalid memory read in display_debug_frames
Pointer comparisons have traps for the unwary. After adding a large
unknown value to "start", the test "start < end" depends on where
"start" is originally in memory.
PR 22239
* dwarf.c (read_cie): Don't compare "start" and "end" pointers
after adding a possibly wild length to "start", compare the length
to the difference of the pointers instead. Remove now redundant
"negative" length test.
Diffstat (limited to 'binutils/ChangeLog')
| -rw-r--r-- | binutils/ChangeLog | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 490085c..21b4350 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,11 @@ +2017-10-05 Alan Modra <amodra@gmail.com> + + PR 22239 + * dwarf.c (read_cie): Don't compare "start" and "end" pointers + after adding a possibly wild length to "start", compare the length + to the difference of the pointers instead. Remove now redundant + "negative" length test. + 2017-10-05 Tristan Gingold <tgingold@free.fr> * MAINTAINERS: Update email address. Redirect release maintainer |