DynamoRIO/drmemory

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

Open

#759 ouverte le 28 nov. 2014

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C (289 forks)auto 404
Component-LeakCheckComponent-SuppressionMigratedPriority-MediumType-Featurehelp wanted

Métriques du dépôt

Stars
 (2 727 stars)
Métriques de merge PR
 (Métriques PR en attente)

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

Guide contributeur

Direction de recherche
Examinez la logique d'analyse de suppression existante dans le code source de DrMemory (probablement dans drmemory/suppress.c ou similaire). Comprenez comment les qualificateurs actuels comme 'fun=' sont analysés et mis en correspondance. Ensuite, concevez et implémentez l'analyse pour les qualificateurs 'size=' et 'offset=', en prenant en charge les plages optionnelles et les tailles alternatives. Ajoutez des tests pour vérifier la nouvelle fonctionnalité.
Stack technique
c
Domaine
tooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40