diff options
| author | Jeremy Hylton <jeremy@alum.mit.edu> | 2001-11-28 20:24:33 (GMT) |
|---|---|---|
| committer | Jeremy Hylton <jeremy@alum.mit.edu> | 2001-11-28 20:24:33 (GMT) |
| commit | 05bd787c6cd55f29d43465de621778221e0fc46e (patch) | |
| tree | 48cda9656ae44839b52a7b8b8d3979ef92bbef5c /Python | |
| parent | a30eacf45815c14b480972c5097677fac83ed4c0 (diff) | |
| download | cpython-05bd787c6cd55f29d43465de621778221e0fc46e.zip cpython-05bd787c6cd55f29d43465de621778221e0fc46e.tar.gz cpython-05bd787c6cd55f29d43465de621778221e0fc46e.tar.bz2 | |
Use PyOS_snprintf instead of sprintf.
Just being sure. The old code looks like it was safe, but there's no
harm in double-checking.
Diffstat (limited to 'Python')
| -rw-r--r-- | Python/exceptions.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/Python/exceptions.c b/Python/exceptions.c index 4d17529..99002bf 100644 --- a/Python/exceptions.c +++ b/Python/exceptions.c @@ -810,21 +810,21 @@ SyntaxError__str__(PyObject *self, PyObject *args) if (have_filename) bufsize += PyString_GET_SIZE(filename); - buffer = PyMem_Malloc(bufsize); + buffer = PyMem_MALLOC(bufsize); if (buffer != NULL) { if (have_filename && have_lineno) - sprintf(buffer, "%s (%s, line %ld)", - PyString_AS_STRING(str), - my_basename(PyString_AS_STRING(filename)), - PyInt_AsLong(lineno)); + PyOS_snprintf(buffer, bufsize, "%s (%s, line %ld)", + PyString_AS_STRING(str), + my_basename(PyString_AS_STRING(filename)), + PyInt_AsLong(lineno)); else if (have_filename) - sprintf(buffer, "%s (%s)", - PyString_AS_STRING(str), - my_basename(PyString_AS_STRING(filename))); + PyOS_snprintf(buffer, bufsize, "%s (%s)", + PyString_AS_STRING(str), + my_basename(PyString_AS_STRING(filename))); else if (have_lineno) - sprintf(buffer, "%s (line %ld)", - PyString_AS_STRING(str), - PyInt_AsLong(lineno)); + PyOS_snprintf(buffer, bufsize, "%s (line %ld)", + PyString_AS_STRING(str), + PyInt_AsLong(lineno)); result = PyString_FromString(buffer); PyMem_FREE(buffer); |