8000 gh-92930: _pickle.c: Acquire strong references before calling save() by sweeneyde · Pull Request #92931 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-92930: _pickle.c: Acquire strong references before calling save() #92931

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Jun 11, 2022
Merged

gh-92930: _pickle.c: Acquire strong references before calling save() #92931

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
9b70755
Acquire strong references after PyDict_Next
sweeneyde May 18, 2022
7af44c4
simplify test
sweeneyde May 18, 2022
802a220
📜🤖 Added by blurb_it.
blurb-it[bot] May 18, 2022
a2a8e35
INCREF before save() in batch_list_exact
sweeneyde Jun 10, 2022
58ad6a2
Merge branch 'main' into picklecrasher
sweeneyde Jun 10, 2022
e48b31d
Add INCREF/DECREF for save_set
sweeneyde Jun 10, 2022
72b3dd5
Merge branch 'picklecrasher' of https://github.com/sweeneyde/cpython …
sweeneyde Jun 10, 2022
7b8f697
More tests
sweeneyde Jun 10, 2022
7133c2c
Check non-singleton set
sweeneyde Jun 10, 2022
a939d16
Update Misc/NEWS.d/next/Core and Builtins/2022-05-18-18-34-45.gh-issu…
sweeneyde Jun 10, 2022
370c44e
clean up test case, better name
sweeneyde Jun 11, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
simplify test
  • Loading branch information
@sweeneyde
sweeneyde committed May 18, 2022
commit 7af44c4fac14d6d7ed487b21d856ddafb66acbb9
26 changes: 8 additions & 18 deletions Lib/test/pickletester.py
8000
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3041,35 +3041,25 @@ def test_evil_class_mutating_dict(self):
global Bad
class Bad:
def __eq__(self, other):
if not ENABLED:
return False
return getrandbits(4) == 0
return ENABLED
def __hash__(self):
return getrandbits(1)
return 42
def __reduce__(self):
break_things()
return (Bad, (), ())
def __setstate__(self, *args):
break_things()
def __del__(self):
break_things()
def __getattr__(self):
break_things()

def break_things():
if ENABLED and getrandbits(6) == 0:
collection.clear()
if getrandbits(6) == 0:
collection.clear()
return (Bad, ())

for proto in protocols:
for _ in range(20):
ENABLED = False
collection = {Bad(): Bad() for _ in range(50)}
collection = {Bad(): Bad() for _ in range(20)}
for bad in collection:
bad.bad = bad
bad.collection = collection
ENABLED = True
try:
self.loads(self.dumps(collection, proto))
data = self.dumps(collection, proto)
self.loads(data)
except RuntimeError as e:
expected = "changed size during iteration"
self.assertIn(expected, str(e))
Expand Down
0