[patch] allocate name of anonymous objfiles properly

David Carlton carlton@bactrian.org
Thu May 1 00:58:00 GMT 2003


In my earlier patch to make sure that anonymous objfiles have a name,
I neglected to ensure that the name is allocated properly.  This can
cause problems if an objfile is unloaded and loaded again, because in
that case somebody will try to free the name, which you can't do if
the name is a constant string.  Oops.

This patch fixes that; I've tested it with GCC 3.2, DWARF 2,
i686-pc-linux-gnu.  It seems obvious; I'll commit it on Friday unless
somebody objects.

David Carlton
carlton@bactrian.org

2003-04-30  David Carlton  <carlton@bactrian.org>

	* objfiles.c (allocate_objfile): For anonymous objfiles, allocate
	the name with mstrsave.

Index: objfiles.c
===================================================================
RCS file: /cvs/src/src/gdb/objfiles.c,v
retrieving revision 1.29
diff -u -p -r1.29 objfiles.c
--- objfiles.c	25 Feb 2003 21:36:18 -0000	1.29
+++ objfiles.c	1 May 2003 00:41:31 -0000
@@ -325,7 +325,7 @@ allocate_objfile (bfd *abfd, int flags)
     }
   else
     {
-      objfile->name = "<<anonymous objfile>>";
+      objfile->name = mstrsave (objfile->md, "<<anonymous objfile>>");
     }
 
   /* Initialize the section indexes for this objfile, so that we can



More information about the Gdb-patches mailing list