fortran: Plug leak of associated_dummy memory. [PR108923]
authorMikael Morin <mikael@gcc.gnu.org>
Fri, 24 Feb 2023 21:11:17 +0000 (22:11 +0100)
committerMikael Morin <mikael@gcc.gnu.org>
Fri, 24 Feb 2023 21:14:20 +0000 (22:14 +0100)
This fixes a memory leak by accompanying the release of
gfc_actual_arglist elements' memory with a release of the
associated_dummy field memory (if allocated).
Actual argument copy is adjusted as well so that each copy can free
its field independently.

PR fortran/108923

gcc/fortran/ChangeLog:

* expr.cc (gfc_free_actual_arglist): Free associated_dummy
memory.
(gfc_copy_actual_arglist): Make a copy of the associated_dummy
field if it is set in the original element.

gcc/fortran/expr.cc

index c295721b9d6d531d918161f8c42b53dc18a928d7..4662328bf3156989c6b008d0d913588cc3e3ce4e 100644 (file)
@@ -545,6 +545,7 @@ gfc_free_actual_arglist (gfc_actual_arglist *a1)
       a2 = a1->next;
       if (a1->expr)
        gfc_free_expr (a1->expr);
+      free (a1->associated_dummy);
       free (a1);
       a1 = a2;
     }
@@ -565,6 +566,12 @@ gfc_copy_actual_arglist (gfc_actual_arglist *p)
       new_arg = gfc_get_actual_arglist ();
       *new_arg = *p;
 
+      if (p->associated_dummy != NULL)
+       {
+         new_arg->associated_dummy = gfc_get_dummy_arg ();
+         *new_arg->associated_dummy = *p->associated_dummy;
+       }
+
       new_arg->expr = gfc_copy_expr (p->expr);
       new_arg->next = NULL;