Skip to content

[3.14] gh-132631: Fix "I/O operation on closed file" when parsing JSON Lines file (GH-132632)#148921

Merged
hugovk merged 1 commit intopython:3.14from
miss-islington:backport-42d645a-3.14
Apr 23, 2026
Merged

[3.14] gh-132631: Fix "I/O operation on closed file" when parsing JSON Lines file (GH-132632)#148921
hugovk merged 1 commit intopython:3.14from
miss-islington:backport-42d645a-3.14

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Apr 23, 2026

(cherry picked from commit 42d645a)

Co-authored-by: Hugo van Kemenade 1324225+hugovk@users.noreply.github.com
Co-authored-by: Brian Schubert brianm.schubert@gmail.com

… Lines file (pythonGH-132632)

(cherry picked from commit 42d645a)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Co-authored-by: Brian Schubert <brianm.schubert@gmail.com>
@hugovk
Copy link
Copy Markdown
Member

hugovk commented Apr 23, 2026

@StanFromIreland The fuzzer is failing, what do we need to do?

BAD BUILD: /tmp/not-out/tmpp_w4bm86/fuzzer-binascii seems to have either startup crash or exit:
sysctl: setting key "vm.mmap_rnd_bits", ignoring: Read-only file system
/tmp/not-out/tmpp_w4bm86/fuzzer-binascii -- -rss_limit_mb=2560 -timeout=25 -seed=1337 -runs=4 < /dev/null
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'fuzzeddataprovider'

@StanFromIreland
Copy link
Copy Markdown
Member

Pinging me was a good start ;-)

I know the reason straight away.

@StanFromIreland
Copy link
Copy Markdown
Member

I wrote python/library-fuzzers#51 to revert the addition of the broken fuzzer while we wait on the OSS-Fuzz team to merge my fix.

@hugovk
Copy link
Copy Markdown
Member

hugovk commented Apr 23, 2026

Thank you!

@hugovk hugovk merged commit 31ba91a into python:3.14 Apr 23, 2026
94 of 95 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants