From: Mikael Morin <mikael@gcc.gnu.org>
Date: Fri, 24 Feb 2023 21:11:17 +0000 (+0100)
Subject: fortran: Plug leak of associated_dummy memory. [PR108923]
X-Git-Tag: basepoints/gcc-14~870
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=24c9edfa73632276d7698c103f35833f29804d98;p=thirdparty%2Fgcc.git

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.
---

diff --git a/gcc/fortran/expr.cc b/gcc/fortran/expr.cc
index c295721b9d6d..4662328bf315 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;