aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMikael Morin <mikael@gcc.gnu.org>2023-02-24 22:11:17 +0100
committerMikael Morin <mikael@gcc.gnu.org>2023-02-24 22:14:20 +0100
commit24c9edfa73632276d7698c103f35833f29804d98 (patch)
tree03eb074b71262dfcdfc7b703bf9ae7f4c2fc2b4e
parent45f406c4f62e516b58dcda20b5a7aa43ff0aa0f3 (diff)
downloadgcc-24c9edfa73632276d7698c103f35833f29804d98.zip
gcc-24c9edfa73632276d7698c103f35833f29804d98.tar.gz
gcc-24c9edfa73632276d7698c103f35833f29804d98.tar.bz2
fortran: Plug leak of associated_dummy memory. [PR108923]
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.
-rw-r--r--gcc/fortran/expr.cc7
1 files changed, 7 insertions, 0 deletions
diff --git a/gcc/fortran/expr.cc b/gcc/fortran/expr.cc
index c295721..4662328 100644
--- a/gcc/fortran/expr.cc
+++ b/gcc/fortran/expr.cc
@@ -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;