From: Daniel Jacobowitz <drow@false.org>
Date: Sun, 22 Feb 2004 19:26:43 +0000 (+0000)
Subject: 	* dwarf2read.c (partial_die_full_name): Always return NULL or
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9be5860a1812276984e7a71aab6018440d00cf10;p=thirdparty%2Fbinutils-gdb.git

	* dwarf2read.c (partial_die_full_name): Always return NULL or
	malloc'd memory.  Don't try to free real_parent->full_name.
---

diff --git a/gdb/ChangeLog.intercu b/gdb/ChangeLog.intercu
index f8f04129b2c..2782fab9444 100644
--- a/gdb/ChangeLog.intercu
+++ b/gdb/ChangeLog.intercu
@@ -1,3 +1,8 @@
+2004-02-22  Daniel Jacobowitz  <drow@mvista.com>
+
+	* dwarf2read.c (partial_die_full_name): Always return NULL or
+	malloc'd memory.  Don't try to free real_parent->full_name.
+
 2004-02-22  Daniel Jacobowitz  <drow@mvista.com>
 
 	* dwarf2read.c (struct dwarf2_cu): Add read_in_chain and per_cu.
diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c
index b2cffbaf0d9..7b36f678a30 100644
--- a/gdb/dwarf2read.c
+++ b/gdb/dwarf2read.c
@@ -1688,13 +1688,13 @@ static char *
 partial_die_full_name (struct partial_die_info *pdi,
 		       struct dwarf2_cu *cu)
 {
-  char *parent_name, *full_name;
+  char *parent_name = NULL, *full_name;
   struct partial_die_info *real_pdi, *real_parent;
   struct dwarf2_cu *spec_cu;
   int free_parent_name = 0;
 
-  if (pdi->full_name_set)
-    return pdi->full_name;
+  /* We shouldn't even have been called in this case.  */
+  gdb_assert (!pdi->full_name_set);
 
   /* Note: this code could probably be micro-optimized.  We may be
      able to avoid redoing the hash table lookup, and we might be able
@@ -1710,7 +1710,7 @@ partial_die_full_name (struct partial_die_info *pdi,
      CU or later in this CU.  It's correct, but somewhat inefficient.  */
 
   if (real_pdi->full_name_set)
-    return real_pdi->full_name;
+    return xstrdup (real_pdi->full_name);
 
   real_parent = real_pdi->die_parent;
   if (real_parent == NULL)
@@ -1724,12 +1724,14 @@ partial_die_full_name (struct partial_die_info *pdi,
       fixup_partial_die (real_parent, spec_cu);
       parent_name = partial_die_full_name (real_parent, spec_cu);
       /* Could cache the full name, too.  */
-      free_parent_name = 1;
+      if (parent_name != NULL)
+	free_parent_name = 1;
     }
 
   /* End hack zone.  */
 
-  parent_name = real_parent->full_name;
+  if (parent_name == NULL)
+    parent_name = real_parent->full_name;
   if (parent_name == NULL)
     parent_name = real_parent->name;