codegen: Don't leak array memory after it was implicitly copied

Found by -fsanitize=address

diff --git a/codegen/valaccodebasemodule.vala b/codegen/valaccodebasemodule.vala
index 61e831ff9df8a998d404b4a3b6e5aa8180dbc1bb..cb45509428bf79056cbae2940666897ff85bc7c3 100644 (file)
--- a/codegen/valaccodebasemodule.vala
+++ b/codegen/valaccodebasemodule.vala
@@ -6154,6 +6154,11 @@ public abstract class Vala.CCodeBaseModule : CodeGenerator {
                                Report.error (node.source_reference, "missing class prerequisite for interface `%s', add GLib.Object to interface declaration if unsure", target_type.type_symbol.get_full_name ());
                                return result;
                        }
+                       // need to free old array after copying it
+                       if (array_needs_copy && requires_destroy (type)) {
+                               result.value_type = type.copy ();
+                               ccode.add_expression (destroy_value (result));
+                       }
                        result = copy;
 
                        // implicit array copying is deprecated, but allow it for internal codegen usage