From d751040b1a4e35fd3b01fc919cd8f9374ed714fd Mon Sep 17 00:00:00 2001 From: Berker Peksag Date: Wed, 14 Sep 2016 08:37:28 +0300 Subject: [PATCH] Issue #26171: Prevent buffer overflow in get_data Backport of 01ddd608b85c. --- Misc/NEWS | 3 +++ Modules/zipimport.c | 5 +++++ 2 files changed, 8 insertions(+) diff --git a/Misc/NEWS b/Misc/NEWS index a38d8beeb1d812..731cd0f1ed075f 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -10,6 +10,9 @@ What's New in Python 3.3.7? Core and Builtins ----------------- +- Issue #26171: Fix possible integer overflow and heap corruption in + zipimporter.get_data(). + - Issue #25709: Fixed problem with in-place string concatenation and utf-8 cache. - Issue #24407: Fix crash when dict is mutated while being updated. diff --git a/Modules/zipimport.c b/Modules/zipimport.c index 2feb2a827c8b67..dad699e7e93024 100644 --- a/Modules/zipimport.c +++ b/Modules/zipimport.c @@ -1089,6 +1089,11 @@ get_data(PyObject *archive, PyObject *toc_entry) PyMarshal_ReadShortFromFile(fp); /* local header size */ file_offset += l; /* Start of file data */ + if (data_size > LONG_MAX - 1) { + fclose(fp); + PyErr_NoMemory(); + return NULL; + } bytes_size = compress == 0 ? data_size : data_size + 1; if (bytes_size == 0) bytes_size++;