diff options
author | Mikael Morin <mikael@gcc.gnu.org> | 2023-02-24 22:11:17 +0100 |
---|---|---|
committer | Mikael Morin <mikael@gcc.gnu.org> | 2023-02-24 22:14:20 +0100 |
commit | 24c9edfa73632276d7698c103f35833f29804d98 (patch) | |
tree | 03eb074b71262dfcdfc7b703bf9ae7f4c2fc2b4e | |
parent | 45f406c4f62e516b58dcda20b5a7aa43ff0aa0f3 (diff) | |
download | gcc-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.cc | 7 |
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; |