DynamoRIO/drmemory

Add size= and offset= qualifiers to leak and possible leak suppressions

Open

#759 aberto em 28 de nov. de 2014

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C (289 forks)auto 404
Component-LeakCheckComponent-SuppressionMigratedPriority-MediumType-Featurehelp wanted

Métricas do repositório

Stars
 (2.727 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

From rnk@google.com on January 19, 2012 16:01:18

This is to support issue #741 . For leak suppressions where we don't have good symbols, as in issue #18 , it can help make the suppression more precise without resorting to something brittle like mod+offs.

The size= qualifier is fairly obvious in its simplest form. We could support a syntax for alternate sizes, such as size=18|24|32, and perhaps ranges (size=32<=x<=48)?.

The offset qualifier is useful for possible leaks where we have something like an SLIST_ENTRY embedded in a struct, as occurred in issue #18 ole32!EventEntry::CreatePoolEntry.

Original issue: http://code.google.com/p/drmemory/issues/detail?id=759

Guia do colaborador

Direção de pesquisa
Examine a lógica de análise de supressão existente no código fonte do DrMemory (provavelmente em drmemory/suppress.c ou similar). Entenda como os qualificadores atuais, como 'fun=', são analisados e correspondidos. Em seguida, projete e implemente a análise para os qualificadores 'size=' e 'offset=', suportando intervalos opcionais e tamanhos alternativos. Adicione testes para verificar a nova funcionalidade.
Pilha de tecnologia
c
Domain
tooling
Tipo Issue
Funcionalidade
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.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
CGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40