- CVE-2010-1634: fix various integer overflow checks in the audioop module (patch 113) - CVE-2010-2089: further checks within the audioop module (patch 114) - CVE-2008-5983: the new PySys_SetArgvEx entry point from r81399 (patch 115)
211 lines
8.1 KiB
Diff
211 lines
8.1 KiB
Diff
diff -up Python-2.6.2/Modules/audioop.c.CVE-2010-1634 Python-2.6.2/Modules/audioop.c
|
|
--- Python-2.6.2/Modules/audioop.c.CVE-2010-1634 2008-07-07 13:02:59.000000000 -0400
|
|
+++ Python-2.6.2/Modules/audioop.c 2010-06-04 11:02:45.743200233 -0400
|
|
@@ -829,7 +829,7 @@ static PyObject *
|
|
audioop_tostereo(PyObject *self, PyObject *args)
|
|
{
|
|
signed char *cp, *ncp;
|
|
- int len, new_len, size, val1, val2, val = 0;
|
|
+ int len, size, val1, val2, val = 0;
|
|
double fac1, fac2, fval, maxval;
|
|
PyObject *rv;
|
|
int i;
|
|
@@ -846,14 +846,13 @@ audioop_tostereo(PyObject *self, PyObjec
|
|
return 0;
|
|
}
|
|
|
|
- new_len = len*2;
|
|
- if (new_len < 0) {
|
|
+ if (len > INT_MAX/2) {
|
|
PyErr_SetString(PyExc_MemoryError,
|
|
"not enough memory for output buffer");
|
|
return 0;
|
|
}
|
|
|
|
- rv = PyString_FromStringAndSize(NULL, new_len);
|
|
+ rv = PyString_FromStringAndSize(NULL, len*2);
|
|
if ( rv == 0 )
|
|
return 0;
|
|
ncp = (signed char *)PyString_AsString(rv);
|
|
@@ -1016,7 +1015,7 @@ audioop_lin2lin(PyObject *self, PyObject
|
|
{
|
|
signed char *cp;
|
|
unsigned char *ncp;
|
|
- int len, new_len, size, size2, val = 0;
|
|
+ int len, size, size2, val = 0;
|
|
PyObject *rv;
|
|
int i, j;
|
|
|
|
@@ -1030,13 +1029,12 @@ audioop_lin2lin(PyObject *self, PyObject
|
|
return 0;
|
|
}
|
|
|
|
- new_len = (len/size)*size2;
|
|
- if (new_len < 0) {
|
|
+ if (len/size > INT_MAX/size2) {
|
|
PyErr_SetString(PyExc_MemoryError,
|
|
"not enough memory for output buffer");
|
|
return 0;
|
|
}
|
|
- rv = PyString_FromStringAndSize(NULL, new_len);
|
|
+ rv = PyString_FromStringAndSize(NULL, (len/size)*size2);
|
|
if ( rv == 0 )
|
|
return 0;
|
|
ncp = (unsigned char *)PyString_AsString(rv);
|
|
@@ -1072,7 +1070,6 @@ audioop_ratecv(PyObject *self, PyObject
|
|
int chan, d, *prev_i, *cur_i, cur_o;
|
|
PyObject *state, *samps, *str, *rv = NULL;
|
|
int bytes_per_frame;
|
|
- size_t alloc_size;
|
|
|
|
weightA = 1;
|
|
weightB = 0;
|
|
@@ -1115,14 +1112,13 @@ audioop_ratecv(PyObject *self, PyObject
|
|
inrate /= d;
|
|
outrate /= d;
|
|
|
|
- alloc_size = sizeof(int) * (unsigned)nchannels;
|
|
- if (alloc_size < nchannels) {
|
|
+ if ((size_t)nchannels > PY_SIZE_MAX/sizeof(int)) {
|
|
PyErr_SetString(PyExc_MemoryError,
|
|
"not enough memory for output buffer");
|
|
return 0;
|
|
}
|
|
- prev_i = (int *) malloc(alloc_size);
|
|
- cur_i = (int *) malloc(alloc_size);
|
|
+ prev_i = (int *) malloc(nchannels * sizeof(int));
|
|
+ cur_i = (int *) malloc(nchannels * sizeof(int));
|
|
if (prev_i == NULL || cur_i == NULL) {
|
|
(void) PyErr_NoMemory();
|
|
goto exit;
|
|
@@ -1159,25 +1155,16 @@ audioop_ratecv(PyObject *self, PyObject
|
|
ceiling(len*outrate/inrate) output frames, and each frame
|
|
requires bytes_per_frame bytes. Computing this
|
|
without spurious overflow is the challenge; we can
|
|
- settle for a reasonable upper bound, though. */
|
|
- int ceiling; /* the number of output frames */
|
|
- int nbytes; /* the number of output bytes needed */
|
|
- int q = len / inrate;
|
|
- /* Now len = q * inrate + r exactly (with r = len % inrate),
|
|
- and this is less than q * inrate + inrate = (q+1)*inrate.
|
|
- So a reasonable upper bound on len*outrate/inrate is
|
|
- ((q+1)*inrate)*outrate/inrate =
|
|
- (q+1)*outrate.
|
|
- */
|
|
- ceiling = (q+1) * outrate;
|
|
- nbytes = ceiling * bytes_per_frame;
|
|
- /* See whether anything overflowed; if not, get the space. */
|
|
- if (q+1 < 0 ||
|
|
- ceiling / outrate != q+1 ||
|
|
- nbytes / bytes_per_frame != ceiling)
|
|
+ settle for a reasonable upper bound, though, in this
|
|
+ case ceiling(len/inrate) * outrate. */
|
|
+
|
|
+ /* compute ceiling(len/inrate) without overflow */
|
|
+ int q = len > 0 ? 1 + (len - 1) / inrate : 0;
|
|
+ if (outrate > INT_MAX / q / bytes_per_frame)
|
|
str = NULL;
|
|
else
|
|
- str = PyString_FromStringAndSize(NULL, nbytes);
|
|
+ str = PyString_FromStringAndSize(NULL,
|
|
+ q * outrate * bytes_per_frame);
|
|
|
|
if (str == NULL) {
|
|
PyErr_SetString(PyExc_MemoryError,
|
|
@@ -1296,7 +1283,7 @@ audioop_ulaw2lin(PyObject *self, PyObjec
|
|
unsigned char *cp;
|
|
unsigned char cval;
|
|
signed char *ncp;
|
|
- int len, new_len, size, val;
|
|
+ int len, size, val;
|
|
PyObject *rv;
|
|
int i;
|
|
|
|
@@ -1309,18 +1296,17 @@ audioop_ulaw2lin(PyObject *self, PyObjec
|
|
return 0;
|
|
}
|
|
|
|
- new_len = len*size;
|
|
- if (new_len < 0) {
|
|
+ if (len > INT_MAX/size) {
|
|
PyErr_SetString(PyExc_MemoryError,
|
|
"not enough memory for output buffer");
|
|
return 0;
|
|
}
|
|
- rv = PyString_FromStringAndSize(NULL, new_len);
|
|
+ rv = PyString_FromStringAndSize(NULL, len*size);
|
|
if ( rv == 0 )
|
|
return 0;
|
|
ncp = (signed char *)PyString_AsString(rv);
|
|
|
|
- for ( i=0; i < new_len; i += size ) {
|
|
+ for ( i=0; i < len*size; i += size ) {
|
|
cval = *cp++;
|
|
val = st_ulaw2linear16(cval);
|
|
|
|
@@ -1370,7 +1356,7 @@ audioop_alaw2lin(PyObject *self, PyObjec
|
|
unsigned char *cp;
|
|
unsigned char cval;
|
|
signed char *ncp;
|
|
- int len, new_len, size, val;
|
|
+ int len, size, val;
|
|
PyObject *rv;
|
|
int i;
|
|
|
|
@@ -1383,18 +1369,17 @@ audioop_alaw2lin(PyObject *self, PyObjec
|
|
return 0;
|
|
}
|
|
|
|
- new_len = len*size;
|
|
- if (new_len < 0) {
|
|
+ if (len > INT_MAX/size) {
|
|
PyErr_SetString(PyExc_MemoryError,
|
|
"not enough memory for output buffer");
|
|
return 0;
|
|
}
|
|
- rv = PyString_FromStringAndSize(NULL, new_len);
|
|
+ rv = PyString_FromStringAndSize(NULL, len*size);
|
|
if ( rv == 0 )
|
|
return 0;
|
|
ncp = (signed char *)PyString_AsString(rv);
|
|
|
|
- for ( i=0; i < new_len; i += size ) {
|
|
+ for ( i=0; i < len*size; i += size ) {
|
|
cval = *cp++;
|
|
val = st_alaw2linear16(cval);
|
|
|
|
@@ -1519,7 +1504,7 @@ audioop_adpcm2lin(PyObject *self, PyObje
|
|
{
|
|
signed char *cp;
|
|
signed char *ncp;
|
|
- int len, new_len, size, valpred, step, delta, index, sign, vpdiff;
|
|
+ int len, size, valpred, step, delta, index, sign, vpdiff;
|
|
PyObject *rv, *str, *state;
|
|
int i, inputbuffer = 0, bufferstep;
|
|
|
|
@@ -1541,13 +1526,12 @@ audioop_adpcm2lin(PyObject *self, PyObje
|
|
} else if ( !PyArg_ParseTuple(state, "ii", &valpred, &index) )
|
|
return 0;
|
|
|
|
- new_len = len*size*2;
|
|
- if (new_len < 0) {
|
|
+ if (len > (INT_MAX/2)/size) {
|
|
PyErr_SetString(PyExc_MemoryError,
|
|
"not enough memory for output buffer");
|
|
return 0;
|
|
}
|
|
- str = PyString_FromStringAndSize(NULL, new_len);
|
|
+ str = PyString_FromStringAndSize(NULL, len*size*2);
|
|
if ( str == 0 )
|
|
return 0;
|
|
ncp = (signed char *)PyString_AsString(str);
|
|
@@ -1555,7 +1539,7 @@ audioop_adpcm2lin(PyObject *self, PyObje
|
|
step = stepsizeTable[index];
|
|
bufferstep = 0;
|
|
|
|
- for ( i=0; i < new_len; i += size ) {
|
|
+ for ( i=0; i < len*size*2; i += size ) {
|
|
/* Step 1 - get the delta value and compute next index */
|
|
if ( bufferstep ) {
|
|
delta = inputbuffer & 0xf;
|