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.1077 < prev    next >
Encoding:
Internet Message Format  |  2013-05-29  |  9.8 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.1077
  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.1077
  11. Problem:    Python: Allocating dict the wrong way, causing a crash.
  12. Solution:   Use py_dict_alloc(). Fix some exception problems. (ZyX)
  13. Files:        src/if_py_both.h
  14.  
  15.  
  16. *** ../vim-7.3.1076/src/if_py_both.h    2013-05-30 19:01:20.000000000 +0200
  17. --- src/if_py_both.h    2013-05-30 21:53:00.000000000 +0200
  18. ***************
  19. *** 26,56 ****
  20.   
  21.   #define PyErr_SetVim(str) PyErr_SetString(VimError, str)
  22.   
  23.   #define INVALID_BUFFER_VALUE ((buf_T *)(-1))
  24.   #define INVALID_WINDOW_VALUE ((win_T *)(-1))
  25.   #define INVALID_TABPAGE_VALUE ((tabpage_T *)(-1))
  26.   
  27. - #define DICTKEY_DECL \
  28. -     PyObject    *dictkey_todecref = NULL;
  29. - #define DICTKEY_GET(err, decref) \
  30. -     if (!(key = StringToChars(keyObject, &dictkey_todecref))) \
  31. -     { \
  32. -     if (decref) \
  33. -     { \
  34. -         Py_DECREF(keyObject); \
  35. -     } \
  36. -     return err; \
  37. -     } \
  38. -     if (decref && !dictkey_todecref) \
  39. -     dictkey_todecref = keyObject; \
  40. -     if (*key == NUL) \
  41. -     { \
  42. -     PyErr_SetString(PyExc_ValueError, _("empty keys are not allowed")); \
  43. -     return err; \
  44. -     }
  45. - #define DICTKEY_UNREF \
  46. -     Py_XDECREF(dictkey_todecref);
  47. - 
  48.   typedef void (*rangeinitializer)(void *);
  49.   typedef void (*runner)(const char *, void *
  50.   #ifdef PY_CAN_RECURSE
  51. --- 26,38 ----
  52.   
  53.   #define PyErr_SetVim(str) PyErr_SetString(VimError, str)
  54.   
  55. + #define RAISE_NO_EMPTY_KEYS PyErr_SetString(PyExc_ValueError, \
  56. +                         _("empty keys are not allowed"))
  57. + 
  58.   #define INVALID_BUFFER_VALUE ((buf_T *)(-1))
  59.   #define INVALID_WINDOW_VALUE ((win_T *)(-1))
  60.   #define INVALID_TABPAGE_VALUE ((tabpage_T *)(-1))
  61.   
  62.   typedef void (*rangeinitializer)(void *);
  63.   typedef void (*runner)(const char *, void *
  64.   #ifdef PY_CAN_RECURSE
  65. ***************
  66. *** 1016,1023 ****
  67.       dictitem_T    *di;
  68.       dict_T    *dict = self->dict;
  69.       hashitem_T    *hi;
  70. ! 
  71. !     DICTKEY_DECL
  72.   
  73.       if (flags & DICT_FLAG_HAS_DEFAULT)
  74.       {
  75. --- 998,1004 ----
  76.       dictitem_T    *di;
  77.       dict_T    *dict = self->dict;
  78.       hashitem_T    *hi;
  79. !     PyObject    *todecref;
  80.   
  81.       if (flags & DICT_FLAG_HAS_DEFAULT)
  82.       {
  83. ***************
  84. *** 1030,1040 ****
  85.       if (flags & DICT_FLAG_RETURN_BOOL)
  86.       defObject = Py_False;
  87.   
  88. !     DICTKEY_GET(NULL, 0)
  89.   
  90.       hi = hash_find(&dict->dv_hashtab, key);
  91.   
  92. !     DICTKEY_UNREF
  93.   
  94.       if (HASHITEM_EMPTY(hi))
  95.       {
  96. --- 1011,1028 ----
  97.       if (flags & DICT_FLAG_RETURN_BOOL)
  98.       defObject = Py_False;
  99.   
  100. !     if (!(key = StringToChars(keyObject, &todecref)))
  101. !     return NULL;
  102. ! 
  103. !     if (*key == NUL)
  104. !     {
  105. !     RAISE_NO_EMPTY_KEYS;
  106. !     return NULL;
  107. !     }
  108.   
  109.       hi = hash_find(&dict->dv_hashtab, key);
  110.   
  111. !     Py_XDECREF(todecref);
  112.   
  113.       if (HASHITEM_EMPTY(hi))
  114.       {
  115. ***************
  116. *** 1173,1179 ****
  117.       typval_T    tv;
  118.       dict_T    *dict = self->dict;
  119.       dictitem_T    *di;
  120. !     DICTKEY_DECL
  121.   
  122.       if (dict->dv_lock)
  123.       {
  124. --- 1161,1167 ----
  125.       typval_T    tv;
  126.       dict_T    *dict = self->dict;
  127.       dictitem_T    *di;
  128. !     PyObject    *todecref;
  129.   
  130.       if (dict->dv_lock)
  131.       {
  132. ***************
  133. *** 1181,1187 ****
  134.       return -1;
  135.       }
  136.   
  137. !     DICTKEY_GET(-1, 0)
  138.   
  139.       di = dict_find(dict, key, -1);
  140.   
  141. --- 1169,1181 ----
  142.       return -1;
  143.       }
  144.   
  145. !     if (!(key = StringToChars(keyObject, &todecref)))
  146. !     return -1;
  147. !     if (*key == NUL)
  148. !     {
  149. !     RAISE_NO_EMPTY_KEYS;
  150. !     return -1;
  151. !     }
  152.   
  153.       di = dict_find(dict, key, -1);
  154.   
  155. ***************
  156. *** 1191,1197 ****
  157.   
  158.       if (di == NULL)
  159.       {
  160. !         DICTKEY_UNREF
  161.           PyErr_SetObject(PyExc_KeyError, keyObject);
  162.           return -1;
  163.       }
  164. --- 1185,1191 ----
  165.   
  166.       if (di == NULL)
  167.       {
  168. !         Py_XDECREF(todecref);
  169.           PyErr_SetObject(PyExc_KeyError, keyObject);
  170.           return -1;
  171.       }
  172. ***************
  173. *** 1208,1213 ****
  174. --- 1202,1208 ----
  175.       {
  176.       if (!(di = dictitem_alloc(key)))
  177.       {
  178. +         Py_XDECREF(todecref);
  179.           PyErr_NoMemory();
  180.           return -1;
  181.       }
  182. ***************
  183. *** 1216,1222 ****
  184.   
  185.       if (dict_add(dict, di) == FAIL)
  186.       {
  187. !         DICTKEY_UNREF
  188.           vim_free(di);
  189.           dictitem_free(di);
  190.           PyErr_SetVim(_("failed to add key to dictionary"));
  191. --- 1211,1217 ----
  192.   
  193.       if (dict_add(dict, di) == FAIL)
  194.       {
  195. !         Py_XDECREF(todecref);
  196.           vim_free(di);
  197.           dictitem_free(di);
  198.           PyErr_SetVim(_("failed to add key to dictionary"));
  199. ***************
  200. *** 1226,1232 ****
  201.       else
  202.       clear_tv(&di->di_tv);
  203.   
  204. !     DICTKEY_UNREF
  205.   
  206.       copy_tv(&tv, &di->di_tv);
  207.       clear_tv(&tv);
  208. --- 1221,1227 ----
  209.       else
  210.       clear_tv(&di->di_tv);
  211.   
  212. !     Py_XDECREF(todecref);
  213.   
  214.       copy_tv(&tv, &di->di_tv);
  215.       clear_tv(&tv);
  216. ***************
  217. *** 2202,2218 ****
  218.       int        flags;
  219.       long    numval;
  220.       char_u    *stringval;
  221. !     DICTKEY_DECL
  222.   
  223.       if (self->Check(self->from))
  224.       return NULL;
  225.   
  226. !     DICTKEY_GET(NULL, 0)
  227.   
  228.       flags = get_option_value_strict(key, &numval, &stringval,
  229.                       self->opt_type, self->from);
  230.   
  231. !     DICTKEY_UNREF
  232.   
  233.       if (flags == 0)
  234.       {
  235. --- 2197,2219 ----
  236.       int        flags;
  237.       long    numval;
  238.       char_u    *stringval;
  239. !     PyObject    *todecref;
  240.   
  241.       if (self->Check(self->from))
  242.       return NULL;
  243.   
  244. !     if (!(key = StringToChars(keyObject, &todecref)))
  245. !     return NULL;
  246. !     if (*key == NUL)
  247. !     {
  248. !     RAISE_NO_EMPTY_KEYS;
  249. !     return NULL;
  250. !     }
  251.   
  252.       flags = get_option_value_strict(key, &numval, &stringval,
  253.                       self->opt_type, self->from);
  254.   
  255. !     Py_XDECREF(todecref);
  256.   
  257.       if (flags == 0)
  258.       {
  259. ***************
  260. *** 2329,2340 ****
  261.       int        flags;
  262.       int        opt_flags;
  263.       int        r = 0;
  264. !     DICTKEY_DECL
  265.   
  266.       if (self->Check(self->from))
  267.       return -1;
  268.   
  269. !     DICTKEY_GET(-1, 0)
  270.   
  271.       flags = get_option_value_strict(key, NULL, NULL,
  272.                       self->opt_type, self->from);
  273. --- 2330,2347 ----
  274.       int        flags;
  275.       int        opt_flags;
  276.       int        r = 0;
  277. !     PyObject    *todecref;
  278.   
  279.       if (self->Check(self->from))
  280.       return -1;
  281.   
  282. !     if (!(key = StringToChars(keyObject, &todecref)))
  283. !     return -1;
  284. !     if (*key == NUL)
  285. !     {
  286. !     RAISE_NO_EMPTY_KEYS;
  287. !     return -1;
  288. !     }
  289.   
  290.       flags = get_option_value_strict(key, NULL, NULL,
  291.                       self->opt_type, self->from);
  292. ***************
  293. *** 2342,2348 ****
  294.       if (flags == 0)
  295.       {
  296.       PyErr_SetObject(PyExc_KeyError, keyObject);
  297. !     DICTKEY_UNREF
  298.       return -1;
  299.       }
  300.   
  301. --- 2349,2355 ----
  302.       if (flags == 0)
  303.       {
  304.       PyErr_SetObject(PyExc_KeyError, keyObject);
  305. !     Py_XDECREF(todecref);
  306.       return -1;
  307.       }
  308.   
  309. ***************
  310. *** 2352,2371 ****
  311.       {
  312.           PyErr_SetString(PyExc_ValueError,
  313.               _("unable to unset global option"));
  314. !         DICTKEY_UNREF
  315.           return -1;
  316.       }
  317.       else if (!(flags & SOPT_GLOBAL))
  318.       {
  319.           PyErr_SetString(PyExc_ValueError, _("unable to unset option "
  320.                           "without global value"));
  321. !         DICTKEY_UNREF
  322.           return -1;
  323.       }
  324.       else
  325.       {
  326.           unset_global_local_option(key, self->from);
  327. !         DICTKEY_UNREF
  328.           return 0;
  329.       }
  330.       }
  331. --- 2359,2378 ----
  332.       {
  333.           PyErr_SetString(PyExc_ValueError,
  334.               _("unable to unset global option"));
  335. !         Py_XDECREF(todecref);
  336.           return -1;
  337.       }
  338.       else if (!(flags & SOPT_GLOBAL))
  339.       {
  340.           PyErr_SetString(PyExc_ValueError, _("unable to unset option "
  341.                           "without global value"));
  342. !         Py_XDECREF(todecref);
  343.           return -1;
  344.       }
  345.       else
  346.       {
  347.           unset_global_local_option(key, self->from);
  348. !         Py_XDECREF(todecref);
  349.           return 0;
  350.       }
  351.       }
  352. ***************
  353. *** 2396,2402 ****
  354.       else
  355.       {
  356.           PyErr_SetString(PyExc_TypeError, _("object must be integer"));
  357. !         DICTKEY_UNREF
  358.           return -1;
  359.       }
  360.   
  361. --- 2403,2409 ----
  362.       else
  363.       {
  364.           PyErr_SetString(PyExc_TypeError, _("object must be integer"));
  365. !         Py_XDECREF(todecref);
  366.           return -1;
  367.       }
  368.   
  369. ***************
  370. *** 2418,2424 ****
  371.           r = -1;
  372.       }
  373.   
  374. !     DICTKEY_UNREF
  375.   
  376.       return r;
  377.   }
  378. --- 2425,2431 ----
  379.           r = -1;
  380.       }
  381.   
  382. !     Py_XDECREF(todecref);
  383.   
  384.       return r;
  385.   }
  386. ***************
  387. *** 4528,4534 ****
  388.       PyObject    *valObject;
  389.       Py_ssize_t    iter = 0;
  390.   
  391. !     if (!(dict = dict_alloc()))
  392.       return -1;
  393.   
  394.       tv->v_type = VAR_DICT;
  395. --- 4535,4541 ----
  396.       PyObject    *valObject;
  397.       Py_ssize_t    iter = 0;
  398.   
  399. !     if (!(dict = py_dict_alloc()))
  400.       return -1;
  401.   
  402.       tv->v_type = VAR_DICT;
  403. ***************
  404. *** 4553,4558 ****
  405. --- 4560,4566 ----
  406.       {
  407.           dict_unref(dict);
  408.           Py_XDECREF(todecref);
  409. +         RAISE_NO_EMPTY_KEYS;
  410.           return -1;
  411.       }
  412.   
  413. ***************
  414. *** 4600,4606 ****
  415.       PyObject    *keyObject;
  416.       PyObject    *valObject;
  417.   
  418. !     if (!(dict = dict_alloc()))
  419.       return -1;
  420.   
  421.       tv->v_type = VAR_DICT;
  422. --- 4608,4614 ----
  423.       PyObject    *keyObject;
  424.       PyObject    *valObject;
  425.   
  426. !     if (!(dict = py_dict_alloc()))
  427.       return -1;
  428.   
  429.       tv->v_type = VAR_DICT;
  430. ***************
  431. *** 4637,4642 ****
  432. --- 4645,4651 ----
  433.           Py_DECREF(iterator);
  434.           Py_XDECREF(todecref);
  435.           dict_unref(dict);
  436. +         RAISE_NO_EMPTY_KEYS;
  437.           return -1;
  438.       }
  439.   
  440. *** ../vim-7.3.1076/src/version.c    2013-05-30 21:42:09.000000000 +0200
  441. --- src/version.c    2013-05-30 21:49:50.000000000 +0200
  442. ***************
  443. *** 730,731 ****
  444. --- 730,733 ----
  445.   {   /* Add new patch number below this line */
  446. + /**/
  447. +     1077,
  448.   /**/
  449.  
  450. -- 
  451. The History of every major Galactic Civilization tends to pass through
  452. three distinct and recognizable phases, those of Survival, Inquiry and
  453. Sophistication, otherwise known as the How, Why and Where phases.
  454. For instance, the first phase is characterized by the question 'How can
  455. we eat?' the second by the question 'Why do we eat?' and the third by
  456. the question 'Where shall we have lunch?'
  457.         -- Douglas Adams, "The Hitchhiker's Guide to the Galaxy"
  458.  
  459.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  460. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  461. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  462.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  463.