Plug a possible descriptor leak

2012-02-14 14:26:58 -04:00
parent bd4e97b857
commit ff52506b24
1 changed files with 3 additions and 1 deletions
--- a/apps/document_signatures/models.py
+++ b/apps/document_signatures/models.py
@@ -28,7 +28,9 @@ class DocumentVersionSignature(models.Model):

    def save(self, *args, **kwargs):
        if not self.pk:
-            self.has_embedded_signature = gpg.has_embedded_signature(self.document_version.open(raw=True))
+            descriptor = self.document_version.open(raw=True)
+            self.has_embedded_signature = gpg.has_embedded_signature(descriptor)
+            descriptor.close()
        super(DocumentVersionSignature, self).save(*args, **kwargs)

    class Meta: