mirror of
https://github.com/python/cpython.git
synced 2025-07-19 17:25:54 +00:00
Merged revisions 75145 via svnmerge from
svn+ssh://pythondev@www.python.org/python/branches/py3k ................ r75145 | mark.dickinson | 2009-09-29 20:21:35 +0100 (Tue, 29 Sep 2009) | 10 lines Merged revisions 75141 via svnmerge from svn+ssh://pythondev@svn.python.org/python/trunk ........ r75141 | mark.dickinson | 2009-09-29 20:01:06 +0100 (Tue, 29 Sep 2009) | 3 lines Issue #7019: Unmarshalling of bad long data could produce unnormalized PyLongs. Raise ValueError instead. ........ ................
This commit is contained in:
parent
595ad32f59
commit
2bbeb0eacd
3 changed files with 21 additions and 3 deletions
|
@ -553,7 +553,7 @@ static PyObject *
|
|||
r_PyLong(RFILE *p)
|
||||
{
|
||||
PyLongObject *ob;
|
||||
int size, i, j, md;
|
||||
int size, i, j, md, shorts_in_top_digit;
|
||||
long n;
|
||||
digit d;
|
||||
|
||||
|
@ -566,7 +566,8 @@ r_PyLong(RFILE *p)
|
|||
return NULL;
|
||||
}
|
||||
|
||||
size = 1 + (ABS(n)-1) / PyLong_MARSHAL_RATIO;
|
||||
size = 1 + (ABS(n) - 1) / PyLong_MARSHAL_RATIO;
|
||||
shorts_in_top_digit = 1 + (ABS(n) - 1) % PyLong_MARSHAL_RATIO;
|
||||
ob = _PyLong_New(size);
|
||||
if (ob == NULL)
|
||||
return NULL;
|
||||
|
@ -583,12 +584,21 @@ r_PyLong(RFILE *p)
|
|||
ob->ob_digit[i] = d;
|
||||
}
|
||||
d = 0;
|
||||
for (j=0; j < (ABS(n)-1)%PyLong_MARSHAL_RATIO + 1; j++) {
|
||||
for (j=0; j < shorts_in_top_digit; j++) {
|
||||
md = r_short(p);
|
||||
if (md < 0 || md > PyLong_MARSHAL_BASE)
|
||||
goto bad_digit;
|
||||
/* topmost marshal digit should be nonzero */
|
||||
if (md == 0 && j == shorts_in_top_digit - 1) {
|
||||
Py_DECREF(ob);
|
||||
PyErr_SetString(PyExc_ValueError,
|
||||
"bad marshal data (unnormalized long data)");
|
||||
return NULL;
|
||||
}
|
||||
d += (digit)md << j*PyLong_MARSHAL_SHIFT;
|
||||
}
|
||||
/* top digit should be nonzero, else the resulting PyLong won't be
|
||||
normalized */
|
||||
ob->ob_digit[size-1] = d;
|
||||
return (PyObject *)ob;
|
||||
bad_digit:
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue