diff options
author | Alan Modra <amodra@gmail.com> | 2020-03-27 10:30:56 +1030 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2020-03-27 10:37:18 +1030 |
commit | 80e2a3b66ed9055fc26a1e5e26c6158b1c87111d (patch) | |
tree | 5abc92bf82b05a19d765b1d4ff906a1e427fc362 /binutils/ChangeLog | |
parent | 89ff19d45e9acfff9c77bf570631df6b99b62619 (diff) | |
download | gdb-80e2a3b66ed9055fc26a1e5e26c6158b1c87111d.zip gdb-80e2a3b66ed9055fc26a1e5e26c6158b1c87111d.tar.gz gdb-80e2a3b66ed9055fc26a1e5e26c6158b1c87111d.tar.bz2 |
Re: readelf looping in process_archive
This patch fixes a leak of qualified_name caused by 4c83662712 and a
double free introduced by fd486f32d1. Not breaking out of the loop
results in an error: "failed to seek to next archive header". That's
slightly better than silently preventing the possibility of endless
loops.
* readelf.c (process_archive): Don't double free qualified_name.
Don't break out of loop with "negative" archive_file_size, just
set file offset to max.
Diffstat (limited to 'binutils/ChangeLog')
-rw-r--r-- | binutils/ChangeLog | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 2f551f1..efc2e9e 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,9 @@ +2020-03-27 Alan Modra <amodra@gmail.com> + + * readelf.c (process_archive): Don't double free qualified_name. + Don't break out of loop with "negative" archive_file_size, just + set file offset to max. + 2020-03-25 Alan Modra <amodra@gmail.com> * readelf.c (process_archive): Prevent endless loop. |