Fixed handling large Oid valuesfix-961

Oid is defined as unsigned 32. On some Python implementations (probably
the ones where maxint = 2 ** 31) this can cause int overflow for large
values (see #961). On my 64 box it doesn't seem the case.

Oid handling was sloppy here and there (messages, casts...): trying to
use uint everywhere, and added a couple of helper macros to treat Oid
consistently.

Close #961.

1 files changed, 10 insertions, 0 deletions

diff --git a/psycopg/python.h b/psycopg/python.h
index 521c568..f45aeb5 100644
--- a/psycopg/python.h
+++ b/psycopg/python.h
@@ -91,6 +91,11 @@ typedef unsigned long Py_uhash_t;
 
 #define INIT_MODULE(m) init ## m
 
+/* fix #961, but don't change all types to longs. Sure someone will complain. */
+#define PyLong_FromOid(x) (((x) & 0x80000000) ? \
+    PyLong_FromUnsignedLong((unsigned long)(x)) : \
+    PyInt_FromLong((x)))
+
 #endif  /* PY_2 */
 
 #if PY_3
@@ -133,6 +138,11 @@ typedef unsigned long Py_uhash_t;
 
 #define INIT_MODULE(m) PyInit_ ## m
 
+#define PyLong_FromOid(x) (PyLong_FromUnsignedLong((unsigned long)(x)))
+
 #endif  /* PY_3 */
 
+/* expose Oid attributes in Python C objects */
+#define T_OID T_UINT
+
 #endif /* !defined(PSYCOPG_PYTHON_H) */