Issue #532631: Add paranoid check to avoid potential buffer overflow

on systems with sizeof(int) > 4.
This commit is contained in:
Mark Dickinson 2009-03-29 16:17:16 +00:00
parent 2e648ecc7d
commit 174e909842
1 changed files with 10 additions and 1 deletions

View File

@ -4344,6 +4344,15 @@ formatfloat(char *buf, size_t buflen, int flags,
}
if (prec < 0)
prec = 6;
/* make sure that the decimal representation of precision really does
need at most 10 digits: platforms with sizeof(int) == 8 exist! */
if (prec > 0x7fffffffL) {
PyErr_SetString(PyExc_OverflowError,
"outrageously large precision "
"for formatted float");
return -1;
}
if (type == 'f' && fabs(x) >= 1e50)
type = 'g';
/* Worst case length calc to ensure no buffer overrun:
@ -4372,7 +4381,7 @@ formatfloat(char *buf, size_t buflen, int flags,
PyOS_snprintf(fmt, sizeof(fmt), "%%%s.%d%c",
(flags&F_ALT) ? "#" : "",
prec, type);
PyOS_ascii_formatd(buf, buflen, fmt, x);
PyOS_ascii_formatd(buf, buflen, fmt, x);
return (int)strlen(buf);
}