Potential Memory Leak
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 2
- Forks
- 2
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
In src/constants.c, if the PyNumber_And call fails, the routine returns without disposing of allocated objects.
Enclosed is a patch that fixes this, while regularizing the handling of cleanup on errors in an easier-to-understand, more structured fashion, avoiding gotos. Even with explicitly spelling out PyErr_Occurred() calls everywhere, the code is still slightly shorter.
But I think a C extension library requires too much work for this kind of thing. A more complete binding for Fontconfig can be done in pure Python with much less code.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Lesen von src/constants.c und vergleiche dessen PyNumber_And-Fehlerpfad mit dem beigefügten Patch. Bestätige, dass zugewiesene Objekte bei diesem Fehler freigegeben werden und dass die Fehlerbereinigungspfade konsistent behandelt werden; abgeschlossen ist die Routine, wenn sie auf diesen Fehlerpfaden leckfrei ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- c, python
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 42/100