commit 8404749e4b65f23a7d84c3940ea061eee0b4b344
author Hirokazu Yamamoto <ocean-city@m2.ccsnet.ne.jp> Tue Mar 03 07:49:01 2009 +0000
committer Hirokazu Yamamoto <ocean-city@m2.ccsnet.ne.jp> Tue Mar 03 07:49:01 2009 +0000
tree d8082d486912a2af4485d4b47dceee81bc6a3f02
parent 45ed72ddaf16c97237f2ad522959d6ffd8b40e2e

Fixed memory leak on failure. This is related to issue5403 but won't crash on py3k.

Modules/sha1module.c

diff --git a/Modules/sha1module.c b/Modules/sha1module.c
index a7f6ad2..3e630eb 100644
--- a/Modules/sha1module.c
+++ b/Modules/sha1module.c
@@ -502,18 +502,23 @@
     if (data_obj)
         GET_BUFFER_VIEW_OR_ERROUT(data_obj, &buf);
 
-    if ((new = newSHA1object()) == NULL)
+    if ((new = newSHA1object()) == NULL) {
+        if (data_obj)
+            PyBuffer_Release(&buf);
         return NULL;
+    }
 
     sha1_init(&new->hash_state);
 
     if (PyErr_Occurred()) {
         Py_DECREF(new);
+        if (data_obj)
+            PyBuffer_Release(&buf);
         return NULL;
     }
     if (data_obj) {
         sha1_process(&new->hash_state, buf.buf, buf.len);
-	PyBuffer_Release(&buf);
+        PyBuffer_Release(&buf);
     }
 
     return (PyObject *)new;