From: Thomas Wouters <thomas@python.org>
Date: Wed, 1 Mar 2006 01:01:55 +0000 (+0000)
Subject: Fix DBEnv's set_tx_timestamp wrapper to be slightly more correct on
X-Git-Tag: v2.5a0~457
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9d63ccae909122dc5a6e4dc793ad3c3885cc5bdc;p=python

Fix DBEnv's set_tx_timestamp wrapper to be slightly more correct on
non-32bit platforms. Will still only allow 32 bits in a timestamp on Win64,
but at least it won't crash, and it'll work right on platforms where longs
are big enough to contain time_t's.

(A better-working, although conceptually less-right fix would have been to
use Py_ssize_t here, but Martin and Tim won't let me.)
---

diff --git a/Modules/_bsddb.c b/Modules/_bsddb.c
index c769a11f19..8f8eab24c3 100644
--- a/Modules/_bsddb.c
+++ b/Modules/_bsddb.c
@@ -4190,13 +4190,14 @@ static PyObject*
 DBEnv_set_tx_timestamp(DBEnvObject* self, PyObject* args)
 {
     int err;
-    time_t stamp;
+    long stamp;
+    time_t timestamp;
 
-    if (!PyArg_ParseTuple(args, "i:set_tx_timestamp", &stamp))
+    if (!PyArg_ParseTuple(args, "l:set_tx_timestamp", &stamp))
         return NULL;
     CHECK_ENV_NOT_CLOSED(self);
-
-    err = self->db_env->set_tx_timestamp(self->db_env, &stamp);
+    timestamp = (time_t)stamp;
+    err = self->db_env->set_tx_timestamp(self->db_env, &timestamp);
     RETURN_IF_ERR();
     RETURN_NONE();
 }