From 2217022b7d1142738656d891e00b3d2d9179b796 Mon Sep 17 00:00:00 2001 From: Thomas Klausner Date: Mon, 14 Aug 2017 10:55:44 +0200 Subject: [PATCH] Fix double free(). Found by Brian 'geeknik' Carpenter using AFL. --- THANKS | 1 + lib/zip_dirent.c | 3 --- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/THANKS b/THANKS index be0cca9..a80ee1d 100644 --- a/THANKS +++ b/THANKS @@ -12,6 +12,7 @@ BALATON Zoltan Benjamin Gilbert Boaz Stolk Bogdan +Brian 'geeknik' Carpenter Chris Nehren Coverity Dane Springmeyer diff --git a/lib/zip_dirent.c b/lib/zip_dirent.c index a369900..e5a7cc9 100644 --- a/lib/zip_dirent.c +++ b/lib/zip_dirent.c @@ -579,9 +579,6 @@ _zip_dirent_read(zip_dirent_t *zde, zip_source_t *src, zip_buffer_t *buffer, boo } if (!_zip_dirent_process_winzip_aes(zde, error)) { - if (!from_buffer) { - _zip_buffer_free(buffer); - } return -1; }