Introduce a gdb_ref_ptr specialization for struct value
[deliverable/binutils-gdb.git] / gdb / main.c
index e55c07b4c2a93499345fccbbcf24c541eddae693..189266f90e0e520073c68035c09b0851afd3f32e 100644 (file)
@@ -1,6 +1,6 @@
 /* Top level stuff for GDB, the GNU debugger.
 
-   Copyright (C) 1986-2016 Free Software Foundation, Inc.
+   Copyright (C) 1986-2018 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -45,6 +45,8 @@
 #include "event-top.h"
 #include "infrun.h"
 #include "signals-state-save-restore.h"
+#include <vector>
+#include "common/pathstuff.h"
 
 /* The selected interpreter.  This will be used as a set command
    variable, so it should always be malloc'ed - since
@@ -119,7 +121,7 @@ set_gdb_data_directory (const char *new_datadir)
     warning (_("%s is not a directory."), new_datadir);
 
   xfree (gdb_datadir);
-  gdb_datadir = gdb_realpath (new_datadir);
+  gdb_datadir = gdb_realpath (new_datadir).release ();
 
   /* gdb_realpath won't return an absolute path if the path doesn't exist,
      but we still want to record an absolute path here.  If the user entered
@@ -127,10 +129,10 @@ set_gdb_data_directory (const char *new_datadir)
      isn't canonical, but that's ok.  */
   if (!IS_ABSOLUTE_PATH (gdb_datadir))
     {
-      char *abs_datadir = gdb_abspath (gdb_datadir);
+      gdb::unique_xmalloc_ptr<char> abs_datadir = gdb_abspath (gdb_datadir);
 
       xfree (gdb_datadir);
-      gdb_datadir = abs_datadir;
+      gdb_datadir = abs_datadir.release ();
     }
 }
 
@@ -223,7 +225,7 @@ get_init_files (const char **system_gdbinit,
            {
              /* Append the part of SYSTEM_GDBINIT that follows GDB_DATADIR
                 to gdb_datadir.  */
-             char *tmp_sys_gdbinit = xstrdup (SYSTEM_GDBINIT + datadir_len);
+             char *tmp_sys_gdbinit = xstrdup (&SYSTEM_GDBINIT[datadir_len]);
              char *p;
 
              for (p = tmp_sys_gdbinit; IS_DIR_SEPARATOR (*p); ++p)
@@ -304,11 +306,15 @@ setup_alternate_signal_stack (void)
 #endif
 }
 
-/* Call command_loop.  If it happens to return, pass that through as a
-   non-zero return status.  */
+/* Call command_loop.  */
 
-static int
-captured_command_loop (void *data)
+/* Prevent inlining this function for the benefit of GDB's selftests
+   in the testsuite.  Those tests want to run GDB under GDB and stop
+   here.  */
+static void captured_command_loop () __attribute__((noinline));
+
+static void
+captured_command_loop ()
 {
   struct ui *ui = current_ui;
 
@@ -332,15 +338,12 @@ captured_command_loop (void *data)
      check to detect bad FUNCs code.  */
   do_cleanups (all_cleanups ());
   /* If the command_loop returned, normally (rather than threw an
-     error) we try to quit.  If the quit is aborted, catch_errors()
-     which called this catch the signal and restart the command
-     loop.  */
+     error) we try to quit.  If the quit is aborted, our caller
+     catches the signal and restarts the command loop.  */
   quit_command (NULL, ui->instream == ui->stdin_stream);
-  return 1;
 }
 
-/* Handle command errors thrown from within
-   catch_command_errors/catch_command_errors_const.  */
+/* Handle command errors thrown from within catch_command_errors.  */
 
 static int
 handle_command_errors (struct gdb_exception e)
@@ -358,15 +361,16 @@ handle_command_errors (struct gdb_exception e)
   return 1;
 }
 
-/* Type of the command callback passed to catch_command_errors.  */
+/* Type of the command callback passed to the const
+   catch_command_errors.  */
 
-typedef void (catch_command_errors_ftype) (char *, int);
+typedef void (catch_command_errors_const_ftype) (const char *, int);
 
 /* Wrap calls to commands run before the event loop is started.  */
 
 static int
-catch_command_errors (catch_command_errors_ftype *command,
-                     char *arg, int from_tty)
+catch_command_errors (catch_command_errors_const_ftype command,
+                     const char *arg, int from_tty)
 {
   TRY
     {
@@ -385,31 +389,31 @@ catch_command_errors (catch_command_errors_ftype *command,
   return 1;
 }
 
-/* Type of the command callback passed to catch_command_errors_const.  */
+/* Adapter for symbol_file_add_main that translates 'from_tty' to a
+   symfile_add_flags.  */
 
-typedef void (catch_command_errors_const_ftype) (const char *, int);
+static void
+symbol_file_add_main_adapter (const char *arg, int from_tty)
+{
+  symfile_add_flags add_flags = 0;
 
-/* Like catch_command_errors, but works with const command and args.  */
+  if (from_tty)
+    add_flags |= SYMFILE_VERBOSE;
 
-static int
-catch_command_errors_const (catch_command_errors_const_ftype *command,
-                           const char *arg, int from_tty)
-{
-  TRY
-    {
-      int was_sync = current_ui->prompt_state == PROMPT_BLOCKED;
+  symbol_file_add_main (arg, add_flags);
+}
 
-      command (arg, from_tty);
+/* Perform validation of the '--readnow' and '--readnever' flags.  */
 
-      maybe_wait_sync_command_done (was_sync);
-    }
-  CATCH (e, RETURN_MASK_ALL)
+static void
+validate_readnow_readnever ()
+{
+  if (readnever_symbol_files && readnow_symbol_files)
     {
-      return handle_command_errors (e);
+      error (_("%s: '--readnow' and '--readnever' cannot be "
+              "specified simultaneously"),
+            gdb_program_name);
     }
-  END_CATCH
-
-  return 1;
 }
 
 /* Type of this option.  */
@@ -429,17 +433,19 @@ enum cmdarg_kind
 };
 
 /* Arguments of --command option and its counterpart.  */
-typedef struct cmdarg {
+struct cmdarg
+{
+  cmdarg (cmdarg_kind type_, char *string_)
+    : type (type_), string (string_)
+  {}
+
   /* Type of this option.  */
   enum cmdarg_kind type;
 
   /* Value of this option - filename or the GDB command itself.  String memory
      is not owned by this structure despite it is 'const'.  */
   char *string;
-} cmdarg_s;
-
-/* Define type VEC (cmdarg_s).  */
-DEF_VEC_O (cmdarg_s);
+};
 
 static void
 captured_main_1 (struct captured_main_args *context)
@@ -467,15 +473,10 @@ captured_main_1 (struct captured_main_args *context)
   static int print_configuration;
 
   /* Pointers to all arguments of --command option.  */
-  VEC (cmdarg_s) *cmdarg_vec = NULL;
-  struct cmdarg *cmdarg_p;
+  std::vector<struct cmdarg> cmdarg_vec;
 
-  /* Indices of all arguments of --directory option.  */
-  char **dirarg;
-  /* Allocated size.  */
-  int dirsize;
-  /* Number of elements used.  */
-  int ndir;
+  /* All arguments of --directory option.  */
+  std::vector<char *> dirarg;
 
   /* gdb init files.  */
   const char *system_gdbinit;
@@ -486,8 +487,6 @@ captured_main_1 (struct captured_main_args *context)
   int save_auto_load;
   struct objfile *objfile;
 
-  struct cleanup *chain;
-
 #ifdef HAVE_SBRK
   /* Set this before constructing scoped_command_stats.  */
   lim_at_start = (char *) sbrk (0);
@@ -508,12 +507,6 @@ captured_main_1 (struct captured_main_args *context)
 
   bfd_init ();
   notice_open_fds ();
-  save_original_signals_state ();
-
-  chain = make_cleanup (VEC_cleanup (cmdarg_s), &cmdarg_vec);
-  dirsize = 1;
-  dirarg = (char **) xmalloc (dirsize * sizeof (*dirarg));
-  ndir = 0;
 
   saved_command_line = (char *) xstrdup ("");
 
@@ -523,7 +516,7 @@ captured_main_1 (struct captured_main_args *context)
   setvbuf (stderr, NULL, _IONBF, BUFSIZ);
 #endif
 
-  main_ui = new_ui (stdin, stdout, stderr);
+  main_ui = new ui (stdin, stdout, stderr);
   current_ui = main_ui;
 
   gdb_stdtargerr = gdb_stderr; /* for moment */
@@ -538,13 +531,14 @@ captured_main_1 (struct captured_main_args *context)
 #endif
 
   /* Prefix warning messages with the command name.  */
-  warning_pre_print = xstrprintf ("%s: warning: ", gdb_program_name);
+  gdb::unique_xmalloc_ptr<char> tmp_warn_preprint
+    (xstrprintf ("%s: warning: ", gdb_program_name));
+  warning_pre_print = tmp_warn_preprint.get ();
 
-  if (! getcwd (gdb_dirbuf, sizeof (gdb_dirbuf)))
+  current_directory = getcwd (NULL, 0);
+  if (current_directory == NULL)
     perror_warning_with_name (_("error finding working directory"));
 
-  current_directory = gdb_dirbuf;
-
   /* Set the sysroot path.  */
   gdb_sysroot = relocate_gdb_directory (TARGET_SYSTEM_ROOT,
                                        TARGET_SYSTEM_ROOT_RELOCATABLE);
@@ -598,14 +592,17 @@ captured_main_1 (struct captured_main_args *context)
       OPT_NOWINDOWS,
       OPT_WINDOWS,
       OPT_IX,
-      OPT_IEX
+      OPT_IEX,
+      OPT_READNOW,
+      OPT_READNEVER
     };
     static struct option long_options[] =
     {
       {"tui", no_argument, 0, OPT_TUI},
       {"dbx", no_argument, &dbx_commands, 1},
-      {"readnow", no_argument, &readnow_symbol_files, 1},
-      {"r", no_argument, &readnow_symbol_files, 1},
+      {"readnow", no_argument, NULL, OPT_READNOW},
+      {"readnever", no_argument, NULL, OPT_READNEVER},
+      {"r", no_argument, NULL, OPT_READNOW},
       {"quiet", no_argument, &quiet, 1},
       {"q", no_argument, &quiet, 1},
       {"silent", no_argument, &quiet, 1},
@@ -743,36 +740,20 @@ captured_main_1 (struct captured_main_args *context)
            pidarg = optarg;
            break;
          case 'x':
-           {
-             struct cmdarg cmdarg = { CMDARG_FILE, optarg };
-
-             VEC_safe_push (cmdarg_s, cmdarg_vec, &cmdarg);
-           }
+           cmdarg_vec.emplace_back (CMDARG_FILE, optarg);
            break;
          case 'X':
-           {
-             struct cmdarg cmdarg = { CMDARG_COMMAND, optarg };
-
-             VEC_safe_push (cmdarg_s, cmdarg_vec, &cmdarg);
-           }
+           cmdarg_vec.emplace_back (CMDARG_COMMAND, optarg);
            break;
          case OPT_IX:
-           {
-             struct cmdarg cmdarg = { CMDARG_INIT_FILE, optarg };
-
-             VEC_safe_push (cmdarg_s, cmdarg_vec, &cmdarg);
-           }
+           cmdarg_vec.emplace_back (CMDARG_INIT_FILE, optarg);
            break;
          case OPT_IEX:
-           {
-             struct cmdarg cmdarg = { CMDARG_INIT_COMMAND, optarg };
-
-             VEC_safe_push (cmdarg_s, cmdarg_vec, &cmdarg);
-           }
+           cmdarg_vec.emplace_back (CMDARG_INIT_COMMAND, optarg);
            break;
          case 'B':
            batch_flag = batch_silent = 1;
-           gdb_stdout = ui_file_new();
+           gdb_stdout = new null_file ();
            break;
          case 'D':
            if (optarg[0] == '\0')
@@ -809,13 +790,7 @@ captured_main_1 (struct captured_main_args *context)
            interpreter_p = xstrdup (optarg);
            break;
          case 'd':
-           dirarg[ndir++] = optarg;
-           if (ndir >= dirsize)
-             {
-               dirsize *= 2;
-               dirarg = (char **) xrealloc ((char *) dirarg,
-                                            dirsize * sizeof (*dirarg));
-             }
+           dirarg.push_back (optarg);
            break;
          case 't':
            ttyarg = optarg;
@@ -850,6 +825,20 @@ captured_main_1 (struct captured_main_args *context)
            }
            break;
 
+         case OPT_READNOW:
+           {
+             readnow_symbol_files = 1;
+             validate_readnow_readnever ();
+           }
+           break;
+
+         case OPT_READNEVER:
+           {
+             readnever_symbol_files = 1;
+             validate_readnow_readnever ();
+           }
+           break;
+
          case '?':
            error (_("Use `%s --help' for a complete list of options."),
                   gdb_program_name);
@@ -860,6 +849,8 @@ captured_main_1 (struct captured_main_args *context)
       quiet = 1;
   }
 
+  save_original_signals_state (quiet);
+
   /* Try to set up an alternate signal stack for SIGSEGV handlers.  */
   setup_alternate_signal_stack ();
 
@@ -987,7 +978,7 @@ captured_main_1 (struct captured_main_args *context)
     }
 
   /* Set off error and warning messages with a blank line.  */
-  xfree (warning_pre_print);
+  tmp_warn_preprint.reset ();
   warning_pre_print = _("\nwarning: ");
 
   /* Read and execute the system-wide gdbinit file, if it exists.
@@ -995,7 +986,7 @@ captured_main_1 (struct captured_main_args *context)
      processed; it sets global parameters, which are independent of
      what file you are debugging or what directory you are in.  */
   if (system_gdbinit && !inhibit_gdbinit)
-    catch_command_errors_const (source_script, system_gdbinit, 0);
+    catch_command_errors (source_script, system_gdbinit, 0);
 
   /* Read and execute $HOME/.gdbinit file, if it exists.  This is done
      *before* all the command line arguments are processed; it sets
@@ -1003,20 +994,24 @@ captured_main_1 (struct captured_main_args *context)
      debugging or what directory you are in.  */
 
   if (home_gdbinit && !inhibit_gdbinit && !inhibit_home_gdbinit)
-    catch_command_errors_const (source_script, home_gdbinit, 0);
+    catch_command_errors (source_script, home_gdbinit, 0);
 
   /* Process '-ix' and '-iex' options early.  */
-  for (i = 0; VEC_iterate (cmdarg_s, cmdarg_vec, i, cmdarg_p); i++)
-    switch (cmdarg_p->type)
+  for (i = 0; i < cmdarg_vec.size (); i++)
     {
-      case CMDARG_INIT_FILE:
-        catch_command_errors_const (source_script, cmdarg_p->string,
-                                   !batch_flag);
-       break;
-      case CMDARG_INIT_COMMAND:
-        catch_command_errors (execute_command, cmdarg_p->string,
-                             !batch_flag);
-       break;
+      const struct cmdarg &cmdarg_p = cmdarg_vec[i];
+
+      switch (cmdarg_p.type)
+       {
+       case CMDARG_INIT_FILE:
+         catch_command_errors (source_script, cmdarg_p.string,
+                               !batch_flag);
+         break;
+       case CMDARG_INIT_COMMAND:
+         catch_command_errors (execute_command, cmdarg_p.string,
+                               !batch_flag);
+         break;
+       }
     }
 
   /* Now perform all the actions indicated by the arguments.  */
@@ -1025,9 +1020,8 @@ captured_main_1 (struct captured_main_args *context)
       catch_command_errors (cd_command, cdarg, 0);
     }
 
-  for (i = 0; i < ndir; i++)
+  for (i = 0; i < dirarg.size (); i++)
     catch_command_errors (directory_switch, dirarg[i], 0);
-  xfree (dirarg);
 
   /* Skip auto-loading section-specified scripts until we've sourced
      local_gdbinit (which is often used to augment the source search
@@ -1042,19 +1036,19 @@ captured_main_1 (struct captured_main_args *context)
       /* The exec file and the symbol-file are the same.  If we can't
          open it, better only print one error message.
          catch_command_errors returns non-zero on success!  */
-      if (catch_command_errors_const (exec_file_attach, execarg,
-                                     !batch_flag))
-       catch_command_errors_const (symbol_file_add_main, symarg,
-                                   !batch_flag);
+      if (catch_command_errors (exec_file_attach, execarg,
+                               !batch_flag))
+       catch_command_errors (symbol_file_add_main_adapter, symarg,
+                             !batch_flag);
     }
   else
     {
       if (execarg != NULL)
-       catch_command_errors_const (exec_file_attach, execarg,
-                                   !batch_flag);
+       catch_command_errors (exec_file_attach, execarg,
+                             !batch_flag);
       if (symarg != NULL)
-       catch_command_errors_const (symbol_file_add_main, symarg,
-                                   !batch_flag);
+       catch_command_errors (symbol_file_add_main_adapter, symarg,
+                             !batch_flag);
     }
 
   if (corearg && pidarg)
@@ -1093,7 +1087,8 @@ captured_main_1 (struct captured_main_args *context)
      the same as the $HOME/.gdbinit file (it should exist, also).  */
   if (local_gdbinit)
     {
-      auto_load_local_gdbinit_pathname = gdb_realpath (local_gdbinit);
+      auto_load_local_gdbinit_pathname
+       = gdb_realpath (local_gdbinit).release ();
 
       if (!inhibit_gdbinit && auto_load_local_gdbinit
          && file_is_auto_load_safe (local_gdbinit,
@@ -1103,7 +1098,7 @@ captured_main_1 (struct captured_main_args *context)
        {
          auto_load_local_gdbinit_loaded = 1;
 
-         catch_command_errors_const (source_script, local_gdbinit, 0);
+         catch_command_errors (source_script, local_gdbinit, 0);
        }
     }
 
@@ -1116,17 +1111,21 @@ captured_main_1 (struct captured_main_args *context)
     load_auto_scripts_for_objfile (objfile);
 
   /* Process '-x' and '-ex' options.  */
-  for (i = 0; VEC_iterate (cmdarg_s, cmdarg_vec, i, cmdarg_p); i++)
-    switch (cmdarg_p->type)
+  for (i = 0; i < cmdarg_vec.size (); i++)
     {
-      case CMDARG_FILE:
-        catch_command_errors_const (source_script, cmdarg_p->string,
-                                   !batch_flag);
-       break;
-      case CMDARG_COMMAND:
-        catch_command_errors (execute_command, cmdarg_p->string,
-                             !batch_flag);
-       break;
+      const struct cmdarg &cmdarg_p = cmdarg_vec[i];
+
+      switch (cmdarg_p.type)
+       {
+       case CMDARG_FILE:
+         catch_command_errors (source_script, cmdarg_p.string,
+                               !batch_flag);
+         break;
+       case CMDARG_COMMAND:
+         catch_command_errors (execute_command, cmdarg_p.string,
+                               !batch_flag);
+         break;
+       }
     }
 
   /* Read in the old history after all the command files have been
@@ -1138,8 +1137,6 @@ captured_main_1 (struct captured_main_args *context)
       /* We have hit the end of the batch file.  */
       quit_force (NULL, 0);
     }
-
-  do_cleanups (chain);
 }
 
 static void
@@ -1155,7 +1152,15 @@ captured_main (void *data)
      change - SET_TOP_LEVEL() - has been eliminated.  */
   while (1)
     {
-      catch_errors (captured_command_loop, 0, "", RETURN_MASK_ALL);
+      TRY
+       {
+         captured_command_loop ();
+       }
+      CATCH (ex, RETURN_MASK_ALL)
+       {
+         exception_print (gdb_stderr, ex);
+       }
+      END_CATCH
     }
   /* No exit -- exit is through quit_command.  */
 }
@@ -1210,6 +1215,7 @@ Selection of debuggee and its files:\n\n\
   --se=FILE          Use FILE as symbol file and executable file.\n\
   --symbols=SYMFILE  Read symbols from SYMFILE.\n\
   --readnow          Fully read symbol files on first access.\n\
+  --readnever        Do not read symbol files.\n\
   --write            Set writing into executable and core files.\n\n\
 "), stream);
   fputs_unfiltered (_("\
This page took 0.03186 seconds and 4 git commands to generate.