diff options
| author | Thomas Heller <theller@ctypes.org> | 2006-07-10 11:11:10 (GMT) |
|---|---|---|
| committer | Thomas Heller <theller@ctypes.org> | 2006-07-10 11:11:10 (GMT) |
| commit | 7644262aa5ef62a0e033ac1158b7cc7b41ffdd1a (patch) | |
| tree | 4c45284216ecd8a6476f3a5d4e17b1782b8c3915 | |
| parent | 7b1da513fd5620e3046047af28a6cf38e753e6f5 (diff) | |
| download | cpython-7644262aa5ef62a0e033ac1158b7cc7b41ffdd1a.zip cpython-7644262aa5ef62a0e033ac1158b7cc7b41ffdd1a.tar.gz cpython-7644262aa5ef62a0e033ac1158b7cc7b41ffdd1a.tar.bz2 | |
Assigning None to pointer type structure fields possible overwrote
wrong fields.
| -rw-r--r-- | Lib/ctypes/test/test_structures.py | 10 | ||||
| -rw-r--r-- | Misc/NEWS | 3 | ||||
| -rw-r--r-- | Modules/_ctypes/_ctypes.c | 2 |
3 files changed, 14 insertions, 1 deletions
diff --git a/Lib/ctypes/test/test_structures.py b/Lib/ctypes/test/test_structures.py index bb56a61..8a4531d 100644 --- a/Lib/ctypes/test/test_structures.py +++ b/Lib/ctypes/test/test_structures.py @@ -371,5 +371,15 @@ class PointerMemberTestCase(unittest.TestCase): items = [s.array[i] for i in range(3)] self.failUnlessEqual(items, [1, 2, 3]) + def test_none_to_pointer_fields(self): + class S(Structure): + _fields_ = [("x", c_int), + ("p", POINTER(c_int))] + + s = S() + s.x = 12345678 + s.p = None + self.failUnlessEqual(s.x, 12345678) + if __name__ == '__main__': unittest.main() @@ -39,6 +39,9 @@ Core and builtins Library ------- +- Assigning None to pointer type fields in ctypes structures possible + overwrote the wrong fields, this is fixed now. + - Fixed a segfault in _ctypes when ctypes.wintypes were imported on non-Windows platforms. diff --git a/Modules/_ctypes/_ctypes.c b/Modules/_ctypes/_ctypes.c index f786ead..b332932 100644 --- a/Modules/_ctypes/_ctypes.c +++ b/Modules/_ctypes/_ctypes.c @@ -2187,7 +2187,7 @@ _CData_set(CDataObject *dst, PyObject *type, SETFUNC setfunc, PyObject *value, Py_DECREF(ob); return result; } else if (value == Py_None && PointerTypeObject_Check(type)) { - *(void **)dst->b_ptr = NULL; + *(void **)ptr = NULL; Py_INCREF(Py_None); return Py_None; } else { |