commit abb3351785f0e526a8f5170a5ad08ae9d4638a0a
author Brian Curtin <brian.curtin@gmail.com> Tue Aug 17 20:08:40 2010 +0000
committer Brian Curtin <brian.curtin@gmail.com> Tue Aug 17 20:08:40 2010 +0000
tree 1d44d9e38b4af5337f1030d0e5666817a11b38c6
parent 19467d27ff14ebe31978438078ed5a661ffd29fb

Properly downcast from size_t/Py_ssize_t in a few places.

PC/winreg.c

diff --git a/PC/winreg.c b/PC/winreg.c
index eacb4c2..390a9ce 100644
--- a/PC/winreg.c
+++ b/PC/winreg.c
@@ -765,8 +765,9 @@
             else {
                 if (!PyUnicode_Check(value))
                     return FALSE;
-
-                *retDataSize = 2 + PyUnicode_GET_DATA_SIZE(value);
+                *retDataSize = Py_SAFE_DOWNCAST(
+                                   2 + PyUnicode_GET_DATA_SIZE(value),
+                                   size_t, DWORD);
             }
             *retDataBuf = (BYTE *)PyMem_NEW(DWORD, *retDataSize);
             if (*retDataBuf==NULL){
@@ -798,7 +799,8 @@
                     t = PyList_GET_ITEM(value, j);
                     if (!PyUnicode_Check(t))
                         return FALSE;
-                    size += 2 + PyUnicode_GET_DATA_SIZE(t);
+                    size += Py_SAFE_DOWNCAST(2 + PyUnicode_GET_DATA_SIZE(t),
+                                             size_t, DWORD);
                 }
 
                 *retDataSize = size + 2;
@@ -848,7 +850,7 @@
                     PyErr_NoMemory();
                     return FALSE;
                 }
-                *retDataSize = view.len;
+                *retDataSize = Py_SAFE_DOWNCAST(view.len, Py_ssize_t, DWORD);
                 memcpy(*retDataBuf, view.buf, view.len);
                 PyBuffer_Release(&view);
             }