Potential Memory Leak
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 2
- Forks
- 2
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par lire src/constants.c et comparer son chemin d’échec de PyNumber_And avec le correctif joint. Vérifiez que les objets alloués sont libérés lors de cet échec et que les chemins de nettoyage des erreurs sont traités de manière cohérente ; la routine est considérée comme terminée lorsqu’elle est exempte de fuites sur ces chemins d’erreur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- c, python
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 42/100