DynamoRIO/drmemory

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

Open

#759 geöffnet am 28. Nov. 2014

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C (289 Forks)auto 404
Component-LeakCheckComponent-SuppressionMigratedPriority-MediumType-Featurehelp wanted

Repository-Metriken

Stars
 (2.727 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die vorhandene Parsing Logik für Unterdrückungen im Quellcode von DrMemory (wahrscheinlich in drmemory/suppress.c oder ähnlich). Verstehe, wie aktuelle Qualifizierer wie 'fun=' geparst und abgeglichen werden. Entwerfe und implementiere dann das Parsing für die Qualifizierer 'size=' und 'offset=', mit optionalen Bereichen und alternativen Größen. Füge Tests hinzu, um die neue Funktionalität zu überprüfen.
Tech Stack
c
Domain
tooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
CGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40