|
 |
2adbafb |
To: vim_dev@googlegroups.com
|
|
|
2adbafb |
Subject: Patch 7.4.063
|
|
|
2adbafb |
Fcc: outbox
|
|
|
2adbafb |
From: Bram Moolenaar <Bram@moolenaar.net>
|
|
|
2adbafb |
Mime-Version: 1.0
|
|
|
2adbafb |
Content-Type: text/plain; charset=UTF-8
|
|
|
2adbafb |
Content-Transfer-Encoding: 8bit
|
|
|
2adbafb |
------------
|
|
|
2adbafb |
|
|
|
2adbafb |
Patch 7.4.063
|
|
|
2adbafb |
Problem: Crash when using invalid key in Python dictionary.
|
|
|
2adbafb |
Solution: Check for object to be NULL. Add tests. (ZyX)
|
|
|
2adbafb |
Files: src/if_py_both.h, src/testdir/test86.in, src/testdir/test86.ok,
|
|
|
2adbafb |
src/testdir/test87.in, src/testdir/test87.ok
|
|
|
2adbafb |
|
|
|
2adbafb |
|
|
|
2adbafb |
*** ../vim-7.4.062/src/if_py_both.h 2013-07-24 17:09:19.000000000 +0200
|
|
|
2adbafb |
--- src/if_py_both.h 2013-11-04 00:27:40.000000000 +0100
|
|
|
2adbafb |
***************
|
|
|
2adbafb |
*** 1624,1629 ****
|
|
|
2adbafb |
--- 1624,1632 ----
|
|
|
2adbafb |
PyObject *rObj = _DictionaryItem(self, keyObject, DICT_FLAG_RETURN_BOOL);
|
|
|
2adbafb |
int ret;
|
|
|
2adbafb |
|
|
|
2adbafb |
+ if (rObj == NULL)
|
|
|
2adbafb |
+ return -1;
|
|
|
2adbafb |
+
|
|
|
2adbafb |
ret = (rObj == Py_True);
|
|
|
2adbafb |
|
|
|
2adbafb |
Py_DECREF(rObj);
|
|
|
2adbafb |
*** ../vim-7.4.062/src/testdir/test86.in 2013-07-13 14:00:31.000000000 +0200
|
|
|
2adbafb |
--- src/testdir/test86.in 2013-11-04 00:27:11.000000000 +0100
|
|
|
2adbafb |
***************
|
|
|
2adbafb |
*** 1088,1093 ****
|
|
|
2adbafb |
--- 1088,1096 ----
|
|
|
2adbafb |
stringtochars_test('d.get(%s)')
|
|
|
2adbafb |
ee('d.pop("a")')
|
|
|
2adbafb |
ee('dl.pop("a")')
|
|
|
2adbafb |
+ cb.append(">> DictionaryContains")
|
|
|
2adbafb |
+ ee('"" in d')
|
|
|
2adbafb |
+ ee('0 in d')
|
|
|
2adbafb |
cb.append(">> DictionaryIterNext")
|
|
|
2adbafb |
ee('for i in ned: ned["a"] = 1')
|
|
|
2adbafb |
del i
|
|
|
2adbafb |
*** ../vim-7.4.062/src/testdir/test86.ok 2013-06-23 16:38:39.000000000 +0200
|
|
|
2adbafb |
--- src/testdir/test86.ok 2013-11-04 00:27:11.000000000 +0100
|
|
|
2adbafb |
***************
|
|
|
2adbafb |
*** 516,521 ****
|
|
|
2adbafb |
--- 516,524 ----
|
|
|
2adbafb |
<<< Finished
|
|
|
2adbafb |
d.pop("a"):KeyError:('a',)
|
|
|
2adbafb |
dl.pop("a"):error:('dictionary is locked',)
|
|
|
2adbafb |
+ >> DictionaryContains
|
|
|
2adbafb |
+ "" in d:ValueError:('empty keys are not allowed',)
|
|
|
2adbafb |
+ 0 in d:TypeError:('expected str() or unicode() instance, but got int',)
|
|
|
2adbafb |
>> DictionaryIterNext
|
|
|
2adbafb |
for i in ned: ned["a"] = 1:RuntimeError:('hashtab changed during iteration',)
|
|
|
2adbafb |
>> DictionaryAssItem
|
|
|
2adbafb |
*** ../vim-7.4.062/src/testdir/test87.in 2013-07-06 13:41:30.000000000 +0200
|
|
|
2adbafb |
--- src/testdir/test87.in 2013-11-04 00:27:11.000000000 +0100
|
|
|
2adbafb |
***************
|
|
|
2adbafb |
*** 1039,1044 ****
|
|
|
2adbafb |
--- 1039,1047 ----
|
|
|
2adbafb |
stringtochars_test('d.get(%s)')
|
|
|
2adbafb |
ee('d.pop("a")')
|
|
|
2adbafb |
ee('dl.pop("a")')
|
|
|
2adbafb |
+ cb.append(">> DictionaryContains")
|
|
|
2adbafb |
+ ee('"" in d')
|
|
|
2adbafb |
+ ee('0 in d')
|
|
|
2adbafb |
cb.append(">> DictionaryIterNext")
|
|
|
2adbafb |
ee('for i in ned: ned["a"] = 1')
|
|
|
2adbafb |
del i
|
|
|
2adbafb |
*** ../vim-7.4.062/src/testdir/test87.ok 2013-06-23 16:38:39.000000000 +0200
|
|
|
2adbafb |
--- src/testdir/test87.ok 2013-11-04 00:27:11.000000000 +0100
|
|
|
2adbafb |
***************
|
|
|
2adbafb |
*** 505,510 ****
|
|
|
2adbafb |
--- 505,513 ----
|
|
|
2adbafb |
<<< Finished
|
|
|
2adbafb |
d.pop("a"):(<class 'KeyError'>, KeyError('a',))
|
|
|
2adbafb |
dl.pop("a"):(<class 'vim.error'>, error('dictionary is locked',))
|
|
|
2adbafb |
+ >> DictionaryContains
|
|
|
2adbafb |
+ "" in d:(<class 'ValueError'>, ValueError('empty keys are not allowed',))
|
|
|
2adbafb |
+ 0 in d:(<class 'TypeError'>, TypeError('expected bytes() or str() instance, but got int',))
|
|
|
2adbafb |
>> DictionaryIterNext
|
|
|
2adbafb |
for i in ned: ned["a"] = 1:(<class 'RuntimeError'>, RuntimeError('hashtab changed during iteration',))
|
|
|
2adbafb |
>> DictionaryAssItem
|
|
|
2adbafb |
*** ../vim-7.4.062/src/version.c 2013-11-03 20:26:27.000000000 +0100
|
|
|
2adbafb |
--- src/version.c 2013-11-04 00:26:39.000000000 +0100
|
|
|
2adbafb |
***************
|
|
|
2adbafb |
*** 740,741 ****
|
|
|
2adbafb |
--- 740,743 ----
|
|
|
2adbafb |
{ /* Add new patch number below this line */
|
|
|
2adbafb |
+ /**/
|
|
|
2adbafb |
+ 63,
|
|
|
2adbafb |
/**/
|
|
|
2adbafb |
|
|
|
2adbafb |
--
|
|
|
2adbafb |
A parent can be arrested if his child cannot hold back a burp during a church
|
|
|
2adbafb |
service.
|
|
|
2adbafb |
[real standing law in Nebraska, United States of America]
|
|
|
2adbafb |
|
|
|
2adbafb |
/// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\
|
|
|
2adbafb |
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
|
|
|
2adbafb |
\\\ an exciting new programming language -- http://www.Zimbu.org ///
|
|
|
2adbafb |
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
|