[Lldb-commits] [lldb] r174267 - Change CommandObjectTargetSymbolsAdd to require that a target exists;

Jason Molenda jmolenda at apple.com
Fri Feb 1 22:00:36 PST 2013

Author: jmolenda
Date: Sat Feb  2 00:00:36 2013
New Revision: 174267

URL: http://llvm.org/viewvc/llvm-project?rev=174267&view=rev
Change CommandObjectTargetSymbolsAdd to require that a target exists;
fixes crash of the form

% lldb
(lldb) target symbols add /tmp/symbols.dSYM
(lldb) Killed: 9



Modified: lldb/trunk/source/Commands/CommandObjectTarget.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Commands/CommandObjectTarget.cpp?rev=174267&r1=174266&r2=174267&view=diff
--- lldb/trunk/source/Commands/CommandObjectTarget.cpp (original)
+++ lldb/trunk/source/Commands/CommandObjectTarget.cpp Sat Feb  2 00:00:36 2013
@@ -4204,7 +4204,7 @@ public:
         CommandObjectParsed (interpreter,
                              "target symbols add",
                              "Add a debug symbol file to one of the target's current modules by specifying a path to a debug symbols file, or using the options to specify a module to download symbols for.",
-                             "target symbols add [<symfile>]"),
+                             "target symbols add [<symfile>]", eFlagRequiresTarget),
         m_option_group (interpreter),
         m_file_option (LLDB_OPT_SET_1, false, "shlib", 's', CommandCompletions::eModuleCompletion, eArgTypeShlibName, "Fullpath or basename for module to find debug symbols for."),
         m_current_frame_option (LLDB_OPT_SET_2, false, "frame", 'F', "Locate the debug symbols the currently selected frame.", false, true)

More information about the lldb-commits mailing list