diff options
| author | Philipp Gesang <phg@phi-gamma.net> | 2020-02-04 21:25:16 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-02-04 21:25:16 (GMT) |
| commit | cb1c0746f277052e45a60d6c436a765e34722821 (patch) | |
| tree | 6f53c2b8a883c3371b55e39821aefa0cd7470c5a /Misc | |
| parent | 9538bc9185e934bee2bd5ae2cda2b2e92a61906d (diff) | |
| download | cpython-cb1c0746f277052e45a60d6c436a765e34722821.zip cpython-cb1c0746f277052e45a60d6c436a765e34722821.tar.gz cpython-cb1c0746f277052e45a60d6c436a765e34722821.tar.bz2 | |
closes bpo-39510: Fix use-after-free in BufferedReader.readinto() (GH-18295)
When called on a closed object, readinto() segfaults on account
of a write to a freed buffer:
==220553== Process terminating with default action of signal 11 (SIGSEGV): dumping core
==220553== Access not within mapped region at address 0x2A
==220553== at 0x48408A0: memmove (vg_replace_strmem.c:1272)
==220553== by 0x58DB0C: _buffered_readinto_generic (bufferedio.c:972)
==220553== by 0x58DCBA: _io__Buffered_readinto_impl (bufferedio.c:1053)
==220553== by 0x58DCBA: _io__Buffered_readinto (bufferedio.c.h:253)
Reproducer:
reader = open ("/dev/zero", "rb")
_void = reader.read (42)
reader.close ()
reader.readinto (bytearray (42)) ### BANG!
The problem exists since 2012 when commit dc469454ec added code
to free the read buffer on close().
Signed-off-by: Philipp Gesang <philipp.gesang@intra2net.com>
Diffstat (limited to 'Misc')
| -rw-r--r-- | Misc/NEWS.d/next/Core and Builtins/2020-02-04-10-27-41.bpo-39510.PMIh-f.rst | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Misc/NEWS.d/next/Core and Builtins/2020-02-04-10-27-41.bpo-39510.PMIh-f.rst b/Misc/NEWS.d/next/Core and Builtins/2020-02-04-10-27-41.bpo-39510.PMIh-f.rst new file mode 100644 index 0000000..9a38e4a --- /dev/null +++ b/Misc/NEWS.d/next/Core and Builtins/2020-02-04-10-27-41.bpo-39510.PMIh-f.rst @@ -0,0 +1 @@ +Fix segfault in ``readinto()`` method on closed BufferedReader. |