home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 7.3 / 7.3.1069 < prev    next >
Encoding:
Internet Message Format  |  2013-05-29  |  4.5 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.1069
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=UTF-8
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 7.3.1069
  11. Problem:    Python: memory leaks.
  12. Solution:   Python patch 28: Purge out DICTKEY_CHECK_EMPTY macros. (ZyX)
  13. Files:        src/if_py_both.h
  14.  
  15.  
  16. *** ../vim-7.3.1068/src/if_py_both.h    2013-05-30 13:37:23.000000000 +0200
  17. --- src/if_py_both.h    2013-05-30 14:50:11.000000000 +0200
  18. ***************
  19. *** 32,46 ****
  20.   
  21.   #define DICTKEY_DECL \
  22.       PyObject    *dictkey_todecref = NULL;
  23. - #define DICTKEY_CHECK_EMPTY(err) \
  24. -     if (*key == NUL) \
  25. -     { \
  26. -     PyErr_SetString(PyExc_ValueError, _("empty keys are not allowed")); \
  27. -     return err; \
  28. -     }
  29. - #define DICTKEY_SET_KEY (key = StringToChars(keyObject, &dictkey_todecref))
  30.   #define DICTKEY_GET(err, decref) \
  31. !     if (!DICTKEY_SET_KEY) \
  32.       { \
  33.       if (decref) \
  34.       { \
  35. --- 32,39 ----
  36.   
  37.   #define DICTKEY_DECL \
  38.       PyObject    *dictkey_todecref = NULL;
  39.   #define DICTKEY_GET(err, decref) \
  40. !     if (!(key = StringToChars(keyObject, &dictkey_todecref))) \
  41.       { \
  42.       if (decref) \
  43.       { \
  44. ***************
  45. *** 50,56 ****
  46.       } \
  47.       if (decref && !dictkey_todecref) \
  48.       dictkey_todecref = keyObject; \
  49. !     DICTKEY_CHECK_EMPTY(err)
  50.   #define DICTKEY_UNREF \
  51.       Py_XDECREF(dictkey_todecref);
  52.   
  53. --- 43,53 ----
  54.       } \
  55.       if (decref && !dictkey_todecref) \
  56.       dictkey_todecref = keyObject; \
  57. !     if (*key == NUL) \
  58. !     { \
  59. !     PyErr_SetString(PyExc_ValueError, _("empty keys are not allowed")); \
  60. !     return err; \
  61. !     }
  62.   #define DICTKEY_UNREF \
  63.       Py_XDECREF(dictkey_todecref);
  64.   
  65. ***************
  66. *** 4551,4557 ****
  67.   
  68.       while (PyDict_Next(obj, &iter, &keyObject, &valObject))
  69.       {
  70. !     DICTKEY_DECL
  71.   
  72.       if (keyObject == NULL || valObject == NULL)
  73.       {
  74. --- 4548,4554 ----
  75.   
  76.       while (PyDict_Next(obj, &iter, &keyObject, &valObject))
  77.       {
  78. !     PyObject    *todecref = NULL;
  79.   
  80.       if (keyObject == NULL || valObject == NULL)
  81.       {
  82. ***************
  83. *** 4559,4574 ****
  84.           return -1;
  85.       }
  86.   
  87. !     if (!DICTKEY_SET_KEY)
  88.       {
  89.           dict_unref(dict);
  90.           return -1;
  91.       }
  92. -     DICTKEY_CHECK_EMPTY(-1)
  93.   
  94.       di = dictitem_alloc(key);
  95.   
  96. !     DICTKEY_UNREF
  97.   
  98.       if (di == NULL)
  99.       {
  100. --- 4556,4576 ----
  101.           return -1;
  102.       }
  103.   
  104. !     if (!(key = StringToChars(keyObject, &todecref)))
  105. !     {
  106. !         dict_unref(dict);
  107. !         return -1;
  108. !     }
  109. !     if (*key == NUL)
  110.       {
  111.           dict_unref(dict);
  112. +         Py_XDECREF(todecref);
  113.           return -1;
  114.       }
  115.   
  116.       di = dictitem_alloc(key);
  117.   
  118. !     Py_XDECREF(todecref);
  119.   
  120.       if (di == NULL)
  121.       {
  122. ***************
  123. *** 4632,4662 ****
  124.   
  125.       while ((keyObject = PyIter_Next(iterator)))
  126.       {
  127. !     DICTKEY_DECL
  128.   
  129. !     if (!DICTKEY_SET_KEY)
  130.       {
  131.           Py_DECREF(iterator);
  132.           dict_unref(dict);
  133. -         DICTKEY_UNREF
  134.           return -1;
  135.       }
  136. !     DICTKEY_CHECK_EMPTY(-1)
  137.   
  138.       if (!(valObject = PyObject_GetItem(obj, keyObject)))
  139.       {
  140.           Py_DECREF(keyObject);
  141.           Py_DECREF(iterator);
  142.           dict_unref(dict);
  143. -         DICTKEY_UNREF
  144.           return -1;
  145.       }
  146.   
  147.       di = dictitem_alloc(key);
  148.   
  149. -     DICTKEY_UNREF
  150. - 
  151.       Py_DECREF(keyObject);
  152.   
  153.       if (di == NULL)
  154.       {
  155. --- 4634,4670 ----
  156.   
  157.       while ((keyObject = PyIter_Next(iterator)))
  158.       {
  159. !     PyObject    *todecref;
  160.   
  161. !     if (!(key = StringToChars(keyObject, &todecref)))
  162.       {
  163. +         Py_DECREF(keyObject);
  164.           Py_DECREF(iterator);
  165.           dict_unref(dict);
  166.           return -1;
  167.       }
  168. !     if (*key == NUL)
  169. !     {
  170. !         Py_DECREF(keyObject);
  171. !         Py_DECREF(iterator);
  172. !         Py_XDECREF(todecref);
  173. !         dict_unref(dict);
  174. !         return -1;
  175. !     }
  176.   
  177.       if (!(valObject = PyObject_GetItem(obj, keyObject)))
  178.       {
  179.           Py_DECREF(keyObject);
  180.           Py_DECREF(iterator);
  181. +         Py_XDECREF(todecref);
  182.           dict_unref(dict);
  183.           return -1;
  184.       }
  185.   
  186.       di = dictitem_alloc(key);
  187.   
  188.       Py_DECREF(keyObject);
  189. +     Py_XDECREF(todecref);
  190.   
  191.       if (di == NULL)
  192.       {
  193. *** ../vim-7.3.1068/src/version.c    2013-05-30 13:37:23.000000000 +0200
  194. --- src/version.c    2013-05-30 13:38:46.000000000 +0200
  195. ***************
  196. *** 730,731 ****
  197. --- 730,733 ----
  198.   {   /* Add new patch number below this line */
  199. + /**/
  200. +     1069,
  201.   /**/
  202.  
  203. -- 
  204. How To Keep A Healthy Level Of Insanity:
  205. 11. Specify that your drive-through order is "to go".
  206.  
  207.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  208. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  209. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  210.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  211.