diff options
author | Benjamin Peterson <benjamin@python.org> | 2015-07-05 00:59:24 (GMT) |
---|---|---|
committer | Benjamin Peterson <benjamin@python.org> | 2015-07-05 00:59:24 (GMT) |
commit | e54d5321cc527d0c97a95b90b2754f6494e7b3e1 (patch) | |
tree | 6798c8787e5648f43ceb868269b56469e1aa8010 /Objects | |
parent | fdbeb2b4b67e1e44c96127a06cf1bdf878f4f7ca (diff) | |
parent | 2a48a6eb335e496220aa75f6daae5b56e231b8e4 (diff) | |
download | cpython-e54d5321cc527d0c97a95b90b2754f6494e7b3e1.zip cpython-e54d5321cc527d0c97a95b90b2754f6494e7b3e1.tar.gz cpython-e54d5321cc527d0c97a95b90b2754f6494e7b3e1.tar.bz2 |
merge 3.4 (#24407)
Diffstat (limited to 'Objects')
-rw-r--r-- | Objects/dictobject.c | 26 |
1 files changed, 19 insertions, 7 deletions
diff --git a/Objects/dictobject.c b/Objects/dictobject.c index 3791d5b..5a7de9e 100644 --- a/Objects/dictobject.c +++ b/Objects/dictobject.c @@ -2039,20 +2039,32 @@ PyDict_Merge(PyObject *a, PyObject *b, int override) if (dictresize(mp, (mp->ma_used + other->ma_used)*2) != 0) return -1; for (i = 0, n = DK_SIZE(other->ma_keys); i < n; i++) { - PyObject *value; + PyObject *key, *value; + Py_hash_t hash; entry = &other->ma_keys->dk_entries[i]; + key = entry->me_key; + hash = entry->me_hash; if (other->ma_values) value = other->ma_values[i]; else value = entry->me_value; - if (value != NULL && - (override || - PyDict_GetItem(a, entry->me_key) == NULL)) { - if (insertdict(mp, entry->me_key, - entry->me_hash, - value) != 0) + if (value != NULL) { + int err = 0; + Py_INCREF(key); + Py_INCREF(value); + if (override || PyDict_GetItem(a, key) == NULL) + err = insertdict(mp, key, hash, value); + Py_DECREF(value); + Py_DECREF(key); + if (err != 0) + return -1; + + if (n != DK_SIZE(other->ma_keys)) { + PyErr_SetString(PyExc_RuntimeError, + "dict mutated during update"); return -1; + } } } } |