On error we return NULL, not out_buf. So make sure to not leak it. Signed-off-by: Mark Wielaard <[email protected]> --- libelf/ChangeLog | 5 +++++ libelf/elf_compress.c | 1 + 2 files changed, 6 insertions(+)
diff --git a/libelf/ChangeLog b/libelf/ChangeLog index 350e4eb..668687d 100644 --- a/libelf/ChangeLog +++ b/libelf/ChangeLog @@ -1,3 +1,8 @@ +2016-04-14 Mark Wielaard <[email protected]> + + * elf_compress.c (__libelf_compress): Free out_buf if deflateInit + fails. + 2016-02-13 Mark Wielaard <[email protected]> * elf32_updatefile.c (updatemmap): Free scns when out of memory. diff --git a/libelf/elf_compress.c b/libelf/elf_compress.c index 4c7c35e..10574ea 100644 --- a/libelf/elf_compress.c +++ b/libelf/elf_compress.c @@ -117,6 +117,7 @@ __libelf_compress (Elf_Scn *scn, size_t hsize, int ei_data, int zrc = deflateInit (&z, Z_BEST_COMPRESSION); if (zrc != Z_OK) { + free (out_buf); __libelf_seterrno (ELF_E_COMPRESS_ERROR); return NULL; } -- 2.5.5
