Issue #24583: Fix crash when set is mutated while being updated.
diff --git a/Lib/test/test_set.py b/Lib/test/test_set.py
index 9f96f87..54de508 100644
--- a/Lib/test/test_set.py
+++ b/Lib/test/test_set.py
@@ -1742,6 +1742,19 @@
s.update(range(100))
list(si)
+ def test_merge_and_mutate(self):
+ class X:
+ def __hash__(self):
+ return hash(0)
+ def __eq__(self, o):
+ other.clear()
+ return False
+
+ other = set()
+ other = {X() for i in range(10)}
+ s = {0}
+ s.update(other)
+
# Application tests (based on David Eppstein's graph recipes ====================================
def powerset(U):
diff --git a/Misc/NEWS b/Misc/NEWS
index bb7e04d..e1d78ad 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -12,6 +12,8 @@
- Issue #24569: Make PEP 448 dictionary evaluation more consistent.
+- Issue #24583: Fix crash when set is mutated while being updated.
+
- Issue #24407: Fix crash when dict is mutated while being updated.
Library
diff --git a/Objects/setobject.c b/Objects/setobject.c
index d962c1e..704d7e2 100644
--- a/Objects/setobject.c
+++ b/Objects/setobject.c
@@ -600,7 +600,8 @@
}
/* We can't assure there are no duplicates, so do normal insertions */
- for (i = 0; i <= other->mask; i++, other_entry++) {
+ for (i = 0; i <= other->mask; i++) {
+ other_entry = &other->table[i];
key = other_entry->key;
if (key != NULL && key != dummy) {
Py_INCREF(key);