]> git.ipfire.org Git - thirdparty/Python/cpython.git/commit
closes bpo-39510: Fix use-after-free in BufferedReader.readinto() (GH-18295)
authorPhilipp Gesang <phg@phi-gamma.net>
Tue, 4 Feb 2020 21:25:16 +0000 (22:25 +0100)
committerGitHub <noreply@github.com>
Tue, 4 Feb 2020 21:25:16 +0000 (13:25 -0800)
commitcb1c0746f277052e45a60d6c436a765e34722821
tree6f53c2b8a883c3371b55e39821aefa0cd7470c5a
parent9538bc9185e934bee2bd5ae2cda2b2e92a61906d
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>
Lib/test/test_io.py
Misc/NEWS.d/next/Core and Builtins/2020-02-04-10-27-41.bpo-39510.PMIh-f.rst [new file with mode: 0644]
Modules/_io/bufferedio.c