diff options
| author | Mark Wielaard <mark@klomp.org> | 2023-02-22 23:34:00 +0100 |
|---|---|---|
| committer | Mark Wielaard <mark@klomp.org> | 2023-02-28 12:35:15 +0100 |
| commit | e24d8a4a3ea106608bb3e8d33c4639cf71d0f08d (patch) | |
| tree | 47de840bd6ec8ec60bb2d828f2756d83d6354d04 /libdw/dwarf_getscopes.c | |
| parent | 4961f9ae2f11795022166698aa15a15f48ec8c5b (diff) | |
| download | elfutils-e24d8a4a3ea106608bb3e8d33c4639cf71d0f08d.tar.gz | |
libdw: Fix dwarf_getscopes memory leak on error
When there is an error in dwarf_getscopes after the initial scopes
have been allocated, e.g. when looking for the inlined scopes, then
the scopes would leak. Fix this by explicitly free the scopes on error.
https://sourceware.org/bugzilla/show_bug.cgi?id=29434
Signed-off-by: Mark Wielaard <mark@klomp.org>
Diffstat (limited to 'libdw/dwarf_getscopes.c')
| -rw-r--r-- | libdw/dwarf_getscopes.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libdw/dwarf_getscopes.c b/libdw/dwarf_getscopes.c index 833f1ac5..ce073b13 100644 --- a/libdw/dwarf_getscopes.c +++ b/libdw/dwarf_getscopes.c @@ -101,7 +101,7 @@ origin_match (unsigned int depth, struct Dwarf_Die_Chain *die, void *arg) Dwarf_Die *scopes = realloc (a->scopes, nscopes * sizeof scopes[0]); if (scopes == NULL) { - free (a->scopes); + /* a->scopes will be freed by dwarf_getscopes on error. */ __libdw_seterrno (DWARF_E_NOMEM); return -1; } @@ -200,6 +200,8 @@ dwarf_getscopes (Dwarf_Die *cudie, Dwarf_Addr pc, Dwarf_Die **scopes) if (result > 0) *scopes = a.scopes; + else if (result < 0) + free (a.scopes); return result; } |