Commit edb21680 authored by Neal Norwitz's avatar Neal Norwitz

This code is actually not used unless WITHOUT_COMPLEX is defined.

However, there was no error checking that PyFloat_FromDouble returned
a valid pointer.  I believe this change is correct as it seemed
to follow other code in the area.

Klocwork # 292.
parent 43bd4db9
...@@ -213,6 +213,10 @@ w_object(PyObject *v, WFILE *p) ...@@ -213,6 +213,10 @@ w_object(PyObject *v, WFILE *p)
w_byte(TYPE_COMPLEX, p); w_byte(TYPE_COMPLEX, p);
temp = (PyFloatObject*)PyFloat_FromDouble( temp = (PyFloatObject*)PyFloat_FromDouble(
PyComplex_RealAsDouble(v)); PyComplex_RealAsDouble(v));
if (!temp) {
p->error = 1;
return;
}
PyFloat_AsReprString(buf, temp); PyFloat_AsReprString(buf, temp);
Py_DECREF(temp); Py_DECREF(temp);
n = strlen(buf); n = strlen(buf);
...@@ -220,6 +224,10 @@ w_object(PyObject *v, WFILE *p) ...@@ -220,6 +224,10 @@ w_object(PyObject *v, WFILE *p)
w_string(buf, (int)n, p); w_string(buf, (int)n, p);
temp = (PyFloatObject*)PyFloat_FromDouble( temp = (PyFloatObject*)PyFloat_FromDouble(
PyComplex_ImagAsDouble(v)); PyComplex_ImagAsDouble(v));
if (!temp) {
p->error = 1;
return;
}
PyFloat_AsReprString(buf, temp); PyFloat_AsReprString(buf, temp);
Py_DECREF(temp); Py_DECREF(temp);
n = strlen(buf); n = strlen(buf);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment