Added missing type check to prevent coercion of decimal to float

(#68).

src/NumberVar.c

@@ -198,6 +198,14 @@ static int NumberVar_SetValueDecimal(udt_Variable *var, uint32_t pos,
 static int NumberVar_SetValueFloat(udt_Variable *var, uint32_t pos,
         dpiData *data, PyObject *value)
 {
+    if (!PyFloat_Check(value) &&
+#if PY_MAJOR_VERSION < 3
+            !PyInt_Check(value) &&
+#endif
+            !PyLong_Check(value)) {
+        PyErr_SetString(PyExc_TypeError, "expecting float");
+        return -1;
+    }
     data->value.asDouble = PyFloat_AsDouble(value);
     if (PyErr_Occurred())
         return -1;

test/NumberVar.py

@@ -18,6 +18,12 @@
 
 class TestNumberVar(BaseTestCase):
 
+    def outputTypeHandlerDecimal(self, cursor, name, defaultType, size,
+            precision, scale):
+        if defaultType == cx_Oracle.NUMBER:
+            return cursor.var(str, 255, outconverter = decimal.Decimal,
+                    arraysize = cursor.arraysize)
+
     def setUp(self):
       BaseTestCase.setUp(self)
       self.rawData = []
@@ -103,6 +109,17 @@ def testBindIntegerAfterString(self):
                 value = 3)
         self.assertEqual(self.cursor.fetchall(), [self.dataByKey[3]])
 
+    def testBindDecimalAfterNumber(self):
+        "test binding in a decimal value after setting input sizes to a number"
+        cursor = self.connection.cursor()
+        value = decimal.Decimal("319438950232418390.273596")
+        cursor.setinputsizes(value = cx_Oracle.NUMBER)
+        cursor.outputtypehandler = self.outputTypeHandlerDecimal
+        cursor.execute("select :value from dual",
+                value = value)
+        outValue, = cursor.fetchone()
+        self.assertEqual(outValue, value)
+
     def testBindNull(self):
         "test binding in a null"
         self.cursor.execute("""