python / python/cpython

Refactor `codegen_deferred_annotations_body` to ensure `mangled` is decref'd

Aberta
#148,059 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

interpreter-core type-bug
Linguagem predominante
Python
Estrelas
77.2k
Forks
35.9k
Métricas de merge de PRs
Métricas de PR pendentes

Descrição

Bug report

Bug description:

Follow-up item from this PR: https://github.com/python/cpython/pull/146443#pullrequestreview-4033013919

During error cases we may fail to decref mangled while using these ADDOP* and VISIT macros. As a result, we can extract the for loop's body into a subfunction that we check the return of and decref mangled if needed.

        // NOTE: ref of mangled can be leaked on ADDOP* and VISIT macros due to early returns
        // fixing would require an overhaul of these macros
CPython versions tested on:

CPython main branch

Operating systems tested on:

Windows

Linked PRs
  • gh-148060

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece localizando codegen_deferred_annotations_body e inspecionando o loop for que usa as macros ADDOP* e VISIT; em seguida, revise os PR 146443 e PR 148060 vinculados para obter contexto. O trabalho estará concluído quando o caminho de erro do loop não deixar mais objetos corrompidos não liberados, preservando o comportamento existente de geração de código.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
c
Domínio
compilers
Tipo de issue
Bug
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.