[PATCH] fix Bug 15180 Agent style dprintf does not respect conditions

[Hui Zhu <teawater@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 441 bytes --]

Hi,

This is because in linux_wait_1, it didn't add any check just call
run_breakpoint_commands (event_child->stop_pc);

So I add a check call run_breakpoint_commands only when condition is true.

Because I don't know howto handle the output in gdbserver.  So I
didn't add test for it.

Thanks,
Hui

2013-04-23  Hui Zhu  <hui@codesourcery.com>

	PR gdb/15180

	* linux-low.c (linux_wait_1): Add a check before call
	run_breakpoint_commands.

[-- Attachment #2: dprintf-gdbserver-cond.txt --]
[-- Type: text/plain, Size: 1369 bytes --]

--- a/gdb/gdbserver/linux-low.c
+++ b/gdb/gdbserver/linux-low.c
@@ -2313,6 +2313,7 @@ linux_wait_1 (ptid_t ptid,
   int maybe_internal_trap;
   int report_to_gdb;
   int trace_event;
+  int condition_true;
 
   /* Translate generic target options into linux options.  */
   options = __WALL;
@@ -2615,6 +2616,9 @@ Check if we're already there.\n",
       goto retry;
     }
 
+  condition_true = gdb_breakpoint_here (event_child->stop_pc)
+		   && gdb_condition_true_at_breakpoint (event_child->stop_pc);
+
   /* If GDB wanted this thread to single step, we always want to
      report the SIGTRAP, and let GDB handle it.  Watchpoints should
      always be reported.  So should signals we can't explain.  A
@@ -2628,11 +2632,11 @@ Check if we're already there.\n",
 		   || event_child->stopped_by_watchpoint
 		   || (!step_over_finished
 		       && !bp_explains_trap && !trace_event)
-		   || (gdb_breakpoint_here (event_child->stop_pc)
-		       && gdb_condition_true_at_breakpoint (event_child->stop_pc)
+		   || (condition_true
 		       && gdb_no_commands_at_breakpoint (event_child->stop_pc)));
 
-  run_breakpoint_commands (event_child->stop_pc);
+  if (condition_true)
+    run_breakpoint_commands (event_child->stop_pc);
 
   /* We found no reason GDB would want us to stop.  We either hit one
      of our own breakpoints, or finished an internal step GDB