Fix leak of auto_obstack objfile_per_bfd_storage->storage_obstack;

[deliverable/binutils-gdb.git] / gdb / demangle.c

diff --git a/gdb/demangle.c b/gdb/demangle.c
index f5f93e5daef4750aa029b3b99ceeae6984b707c1..882356992eda60c2c7103c12407bf21836e56514 100644 (file)
--- a/gdb/demangle.c
+++ b/gdb/demangle.c
@@ -1,6 +1,6 @@
 /* Basic C++ demangling support for GDB.
 
-   Copyright (C) 1991-2016 Free Software Foundation, Inc.
+   Copyright (C) 1991-2017 Free Software Foundation, Inc.
 
    Written by Fred Fish at Cygnus Support.
 
@@ -161,41 +161,33 @@ is_cplus_marker (int c)
 static void
 demangle_command (char *args, int from_tty)
 {
-  char *demangled, *name, *lang_name = NULL;
-  char *arg_buf, *arg_start;
+  char *demangled;
+  const char *name;
+  const char *arg_start;
   int processing_args = 1;
   const struct language_defn *lang;
-  struct cleanup *cleanups;
 
-  arg_buf = xstrdup (args != NULL ? args : "");
-  cleanups = make_cleanup (xfree, arg_buf);
-  arg_start = arg_buf;
+  std::string arg_buf = args != NULL ? args : "";
+  arg_start = arg_buf.c_str ();
 
+  gdb::unique_xmalloc_ptr<char> lang_name;
   while (processing_args
         && *arg_start == '-')
     {
-      char *p = skip_to_space (arg_start);
+      const char *p = skip_to_space_const (arg_start);
 
       if (strncmp (arg_start, "-l", p - arg_start) == 0)
-       {
-         char *lang_name_end;
-
-         lang_name = skip_spaces (p);
-         lang_name_end = skip_to_space (lang_name);
-         lang_name = savestring (lang_name, lang_name_end - lang_name);
-         make_cleanup (xfree, lang_name);
-         p = lang_name_end;
-       }
+       lang_name.reset (extract_arg_const (&p));
       else if (strncmp (arg_start, "--", p - arg_start) == 0)
        processing_args = 0;
       else
        {
-         *p = '\0';
+         gdb::unique_xmalloc_ptr<char> option (extract_arg_const (&p));
          error (_("Unrecognized option '%s' to demangle command.  "
-                  "Try \"help demangle\"."), arg_start);
+                  "Try \"help demangle\"."), option.get ());
        }
 
-      arg_start = skip_spaces (p);
+      arg_start = skip_spaces_const (p);
     }
 
   name = arg_start;
@@ -207,9 +199,9 @@ demangle_command (char *args, int from_tty)
     {
       enum language lang_enum;
 
-      lang_enum = language_enum (lang_name);
+      lang_enum = language_enum (lang_name.get ());
       if (lang_enum == language_unknown)
-       error (_("Unknown language \"%s\""), lang_name);
+       error (_("Unknown language \"%s\""), lang_name.get ());
       lang = language_def (lang_enum);
     }
   else
@@ -223,8 +215,6 @@ demangle_command (char *args, int from_tty)
     }
   else
     error (_("Can't demangle \"%s\""), name);
-
-  do_cleanups (cleanups);
 }
 
 extern initialize_file_ftype _initialize_demangler; /* -Wmissing-prototypes */