commit f092c7c1d7bfcd07dfe621b0b21ab4ece2434c57
author Benjamin Peterson <benjamin@python.org> Wed Jul 07 22:46:00 2010 +0000
committer Benjamin Peterson <benjamin@python.org> Wed Jul 07 22:46:00 2010 +0000
tree b1cfb4e3de9ce76738cc465af0cf83978d4f9908
parent ddd46ceeb0b37cd639fa45cfe847fce668806d7a

Merged revisions 82628,82630 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r82628 | benjamin.peterson | 2010-07-07 13:44:05 -0500 (Wed, 07 Jul 2010) | 1 line

  this needn't be in the loop
........
  r82630 | benjamin.peterson | 2010-07-07 13:54:59 -0500 (Wed, 07 Jul 2010) | 1 line

  don't ignore exceptions from PyObject_IsTrue
........

Modules/_struct.c

diff --git a/Modules/_struct.c b/Modules/_struct.c
index f629817..ba8a8ed 100644
--- a/Modules/_struct.c
+++ b/Modules/_struct.c
@@ -580,9 +580,13 @@
 static int
 np_bool(char *p, PyObject *v, const formatdef *f)
 {
-    BOOL_TYPE y;
+    int y;
+    BOOL_TYPE x;
     y = PyObject_IsTrue(v);
-    memcpy(p, (char *)&y, sizeof y);
+    if (y < 0)
+        return -1;
+    x = y;
+    memcpy(p, (char *)&x, sizeof x);
     return 0;
 }
 
@@ -854,6 +858,8 @@
 {
     char y;
     y = PyObject_IsTrue(v);
+    if (y < 0)
+        return -1;
     memcpy(p, (char *)&y, sizeof y);
     return 0;
 }