`_pyio.BufferedReader.readinto()` can raise `ValueError` after partially filling the destination
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Python
- Estrelas
- 77.2k
- Forks
- 36k
- Métricas de merge de PRs
- Métricas de PR pendentes
Descrição
Bug report
Bug description:
The pure Python implementation of BufferedReader.readinto() can raise ValueError for a valid writable buffer after partially modifying it.
import io
import _pyio
for module in (io, _pyio):
reader = module.BufferedReader(module.BytesIO(b"abcd"), buffer_size=2)
assert reader.read(1) == b"a"
destination = bytearray(2)
try:
result = reader.readinto(destination)
except ValueError as error:
result = f"{type(error).__name__}: {error}"
print(module.__name__, result, destination, reader.read())
Current output on main:
io 2 bytearray(b'bc') b'd'
_pyio ValueError: memoryview assignment: lvalue and rvalue have different structures bytearray(b'b\x00') b'cd'
_pyio should match io: return 2, fill the destination with b"bc", and leave b"d" unread. readinto() first partially fills the destination with data from its internal buffer, then refills that buffer with more data than fits in the remaining space. It then raises ValueError. Because the destination has already been modified and the stream has advanced when the exception is raised, the caller receives no byte count and cannot safely retry on a non-seekable stream.
_pyio.BufferedRandom.readinto() and _pyio.BufferedRWPair.readinto() are also affected because they use the same reader path.
I am working on a PR with a regression test shared by the C and pure Python implementations.
CPython versions tested on:
CPython main branch
Operating systems tested on:
Linux
Linked PRs
- gh-155075
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece no ponto de entrada _pyio.BufferedReader.readinto() e execute o reproducer da issue, comparando-o com io.BufferedReader.readinto(). Está concluído quando o caminho de Python puro retorna 2, preenche o destino com b"bc", deixa b"d" não lido e cobre o comportamento de regressão compartilhado descrito para BufferedRandom e BufferedRWPair.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- backend
- Tipo de issue
- Bug
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Claramente especificada
- Facilidade para iniciantes
- 25/100