[Lldb-commits] [lldb] bae10a6 - Fix a bug with "process continue -b" when no breakpoints are

[Jim Ingham via lldb-commits]

Author: Jim Ingham
Date: 2022-06-22T12:18:07-07:00
New Revision: bae10a6bbb1ed6ba8185884e3239fe3556602dab

URL: https://github.com/llvm/llvm-project/commit/bae10a6bbb1ed6ba8185884e3239fe3556602dab
DIFF: https://github.com/llvm/llvm-project/commit/bae10a6bbb1ed6ba8185884e3239fe3556602dab.diff

LOG: Fix a bug with "process continue -b" when no breakpoints are
passed.  I was passing the empty list of breakponts to the
VerifyBreakpointList routine, but that treats empty as "choose
the default breakpoint" which we don't want here.

Added: 
    

Modified: 
    lldb/source/Commands/CommandObjectProcess.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/source/Commands/CommandObjectProcess.cpp b/lldb/source/Commands/CommandObjectProcess.cpp
index def0e00af087..2f5f649636ae 100644
--- a/lldb/source/Commands/CommandObjectProcess.cpp
+++ b/lldb/source/Commands/CommandObjectProcess.cpp
@@ -595,9 +595,12 @@ class CommandObjectProcessContinue : public CommandObjectParsed {
       
       Target *target = m_exe_ctx.GetTargetPtr();
       BreakpointIDList run_to_bkpt_ids;
-      CommandObjectMultiwordBreakpoint::VerifyBreakpointOrLocationIDs(
-          m_options.m_run_to_bkpt_args, target, result, &run_to_bkpt_ids, 
-          BreakpointName::Permissions::disablePerm);
+      // Don't pass an empty run_to_breakpoint list, as Verify will look for the
+      // default breakpoint.
+      if (m_options.m_run_to_bkpt_args.GetArgumentCount() > 0)
+        CommandObjectMultiwordBreakpoint::VerifyBreakpointOrLocationIDs(
+            m_options.m_run_to_bkpt_args, target, result, &run_to_bkpt_ids, 
+            BreakpointName::Permissions::disablePerm);
       if (!result.Succeeded()) {
         return false;
       }