Fix memory leak for multiple constructors

daanzu · Sep 4, 2024 · f7ffb83 · f7ffb83
1 parent 1b2657e
commit f7ffb83
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/cbits/pywebrtcvad.c b/cbits/pywebrtcvad.c
@@ -22,8 +22,7 @@ static PyObject* vad_create(PyObject *self, PyObject *args)
   if (WebRtcVad_Create(&handle)) {
     return NULL;
   }
-  vadptr = PyCapsule_New(handle, "WebRtcVadPtr", vad_free);
-  return Py_BuildValue("O", vadptr);
+  return PyCapsule_New(handle, "WebRtcVadPtr", vad_free);
 }
 
 static PyObject* vad_init(PyObject *self, PyObject *vadptr)

diff --git a/test_webrtcvad.py b/test_webrtcvad.py
@@ -114,7 +114,6 @@ def test_leak_linear_usage(self):
                         find_voice = True
                 self.assertTrue(find_voice)
 
-    @unittest.expectedFailure
     def test_leak_constructor(self):
         nrepeats = 100000
         with self._profile_memory(0.1):