summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJiyong Min <jiyong.min@samsung.com>2017-04-28 18:18:07 +0900
committerJiyong Min <jiyong.min@samsung.com>2017-04-28 18:30:31 +0900
commit25f96fe50baab58244927291119730fa129fc0b0 (patch)
tree4b0c407685c572281a6e99b35d0f5e432c2865e7
parent8236083c901e4740d31e916bccfecf37522f82f2 (diff)
downloadgiflib-25f96fe50baab58244927291119730fa129fc0b0.tar.gz
giflib-25f96fe50baab58244927291119730fa129fc0b0.tar.bz2
giflib-25f96fe50baab58244927291119730fa129fc0b0.zip
Remove unnecessary duplicate EGifClose(). (Fixes SF bug #83) Multiple use-after-free and double-free vulnerabilities in gifcolor.c in GIFLIB 5.1.2 have unspecified impact and attack vectors. https://nvd.nist.gov/vuln/detail/CVE-2016-3177 Change-Id: I24e077f2a3fbd6554d2557c4bdd94e06e3e08661 Signed-off-by: Jiyong Min <jiyong.min@samsung.com>
-rw-r--r--util/gifcolor.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/util/gifcolor.c b/util/gifcolor.c
index 1700b07..b3e586b 100644
--- a/util/gifcolor.c
+++ b/util/gifcolor.c
@@ -123,9 +123,6 @@ int main(int argc, char **argv)
if (EGifCloseFile(GifFile, &ErrorCode) == GIF_ERROR)
{
PrintGifError(ErrorCode);
- if (GifFile != NULL) {
- EGifCloseFile(GifFile, NULL);
- }
exit(EXIT_FAILURE);
}