commit 2ef5453be5e06566f60edcba7700503259c29b47
author Tom Tromey <tromey@adacore.com> Wed Feb 19 11:06:33 2020 -0700
committer Tom Tromey <tromey@adacore.com> Wed Feb 19 11:17:35 2020 -0700
tree 058188a6487a735d4e4956ca955dbc221d0d5951
parent 9f1528a1bdb541510c56d48bbd77b50b617cb952

Use obstack_strdup in ada-lang.c

I happened across a spot that was still using obstack_alloc and
strcpy, rather than obstack_strdup.  This patch makes the obvious fix.

gdb/ChangeLog
2020-02-19  Tom Tromey  <tromey@adacore.com>

	* ada-lang.c (cache_symbol): Use obstack_strdup.

gdb/ChangeLog

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4a2f813..7825311 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2020-02-19  Tom Tromey  <tromey@adacore.com>
+
+	* ada-lang.c (cache_symbol): Use obstack_strdup.
+
 2020-02-19  Andrew Burgess  <andrew.burgess@embecosm.com>
 
 	* configure: Regenerate.

gdb/ada-lang.c

diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c
index c5b5fdf..316eaf5 100644
--- a/gdb/ada-lang.c
+++ b/gdb/ada-lang.c
@@ -4710,7 +4710,6 @@
   struct ada_symbol_cache *sym_cache
     = ada_get_symbol_cache (current_program_space);
   int h;
-  char *copy;
   struct cache_entry *e;
 
   /* Symbols for builtin types don't have a block.
@@ -4733,9 +4732,7 @@
   e = XOBNEW (&sym_cache->cache_space, cache_entry);
   e->next = sym_cache->root[h];
   sym_cache->root[h] = e;
-  e->name = copy
-    = (char *) obstack_alloc (&sym_cache->cache_space, strlen (name) + 1);
-  strcpy (copy, name);
+  e->name = obstack_strdup (&sym_cache->cache_space, name);
   e->sym = sym;
   e->domain = domain;
   e->block = block;