Fix memory leak in FfiConverterString

Credits to @dignifiedquire Signed-off-by: Kristupas Antanavicius <[email protected]>
NordSecurity · Sep 22, 2023 · 3cb2fbb · 3cb2fbb
1 parent 95b8d9d
commit 3cb2fbb
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/bindgen/templates/StringHelper.go b/bindgen/templates/StringHelper.go
@@ -9,7 +9,10 @@ type {{ ffi_converter_name }} struct{}
 var {{ ffi_converter_name }}INSTANCE = {{ ffi_converter_name }}{}
 
 func ({{ ffi_converter_name }}) lift(cRustBuf C.RustBuffer) string {
-	reader := fromCRustBuffer(cRustBuf).asReader()
+	rustBuf := fromCRustBuffer(cRustBuf)
+	defer rustBuf.free()
+
+	reader := rustBuf.asReader()
 	b, err := io.ReadAll(reader)
 	if err != nil {
 		panic(fmt.Errorf("reading reader: %w", err))