fix(consensus): free allocated CStrings (#6054)

jina-ai · Sep 26, 2023 · 195d376 · 195d376
1 parent 7d7918d
commit 195d376
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/jina/serve/consensus/run.go b/jina/serve/consensus/run.go
@@ -374,7 +374,9 @@ func add_voter(self *C.PyObject, args *C.PyObject) *C.PyObject {
         err := AddVoter(C.GoString(target), C.GoString(raftId), C.GoString(voterAddress))
         if err != nil {
             logger.Error("Error received calling AddVoter", "error", err)
-            C.raise_exception(C.CString("Error from AddVoter"))
+            cerr := C.CString("Error from AddVoter")
+            defer C.free(unsafe.Pointer(cerr))
+            C.raise_exception(cerr)
             return nil
         }
     }
@@ -455,6 +457,8 @@ func get_configuration(self *C.PyObject, args *C.PyObject) *C.PyObject {
         return pyStr
 
     }
-    C.raise_exception(C.CString("Error from get_configuration, wrong parameters passed"))
+    cerr := C.CString("Error from get_configuration, wrong parameters passed")
+    defer C.free(unsafe.Pointer(cerr))
+    C.raise_exception(cerr)
     return nil
 }