microsoft/Terminal

Missing operator delete/delete[] for some allocated objects

Open

#875 aberto em 17 de mai. de 2019

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-CodeHealthHelp WantedIssue-BugProduct-Conhost

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

Description

Hi.

Looks like there are few places in the Terminal code where I can suspect a memory leaks due to missing operator delete or operator delete[]. These allocation sites can be grouped into following:

Here is my gist with quite a simple grep foroperator new calls - https://gist.github.com/dreamer-dead/c43248466fc4d445378b4dfba80d8d25

Is this an intentional behaviour or a bug that should be fixed?

Guia do colaborador

Direção de pesquisa
Investigue cada local de alocação listado no gist (por exemplo, ConptyConnection.cpp linha 191, TermControl.cpp linha 281) e adicione as chamadas operator delete/delete[] ausentes. Considere usar RAII ou ponteiros inteligentes para evitar vazamentos.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Memory Management
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65