[RFA] varobj.c memory leaks

[Keith Seitz <keiths@cygnus.com>]

Hi,

I was looking at c++ problems in varobj when I noticed that we had a few
memory leaks.

*_name_of_child allocates memory for the return result. The result must be
xfreed when no longer needed.

I've tested this on gdb.mi/mi-var-display.exp, gdb.mi/mi-watch.exp,
gdb.gdbtk/c_variable.exp, and gdb.gdbtk/cpp_variable.exp, and it has no
regressions.

Keith

ChangeLog
2001-11-20  Keith Seitz  <keiths@redhat.com>

	* varobj.c (c_value_of_child): Release memory for "name" when
	finshed using it.
	(c_type_of_child): Likewise.
	(cplus_value_of_child): Isolate the use of name_of_child to
	one case that needs it.
	Release memory for "name" when finished using it.

Patch
Index: varobj.c
===================================================================
RCS file: /cvs/src/src/gdb/varobj.c,v
retrieving revision 1.24
diff -u -p -r1.24 varobj.c
--- varobj.c	2001/11/19 19:44:04	1.24
+++ varobj.c	2001/11/20 17:03:27
@@ -1977,6 +1977,7 @@ c_value_of_child (struct varobj *parent,
   if (value != NULL)
     release_value (value);

+  xfree (name);
   return value;
 }

@@ -2019,6 +2020,7 @@ c_type_of_child (struct varobj *parent,
       break;
     }

+  xfree (name);
   return type;
 }

@@ -2281,7 +2283,6 @@ cplus_value_of_child (struct varobj *par
 {
   struct type *type;
   struct value *value;
-  char *name;

   if (CPLUS_FAKE_CHILD (parent))
     type = get_type_deref (parent->parent);
@@ -2289,19 +2290,22 @@ cplus_value_of_child (struct varobj *par
     type = get_type_deref (parent);

   value = NULL;
-  name = name_of_child (parent, index);

   if (((TYPE_CODE (type)) == TYPE_CODE_STRUCT) ||
       ((TYPE_CODE (type)) == TYPE_CODE_UNION))
     {
       if (CPLUS_FAKE_CHILD (parent))
 	{
+	  char *name;
 	  struct value *temp = parent->parent->value;

+	  name = name_of_child (parent, index);
 	  gdb_value_struct_elt (NULL, &value, &temp, NULL, name, NULL,
 				"cplus_structure");
 	  if (value != NULL)
 	    release_value (value);
+
+	  xfree (name);
 	}
       else if (index >= TYPE_N_BASECLASSES (type))
 	{