Avoid duplicate QThreadEvents packets

[binutils-gdb.git] / gdbserver / ax.cc

diff --git a/gdbserver/ax.cc b/gdbserver/ax.cc
index 38ebfbbd7506eb32ec3091f420c8cac4c23200a1..abc2d1f47ceec8c3cb63f225cd777aa786d25b2a 100644 (file)
--- a/gdbserver/ax.cc
+++ b/gdbserver/ax.cc
@@ -985,7 +985,7 @@ gdb_eval_agent_expr (struct eval_agent_expr_context *ctx,
     }
 
   /* Cache the stack top in its own variable. Much of the time we can
-     operate on this variable, rather than dinking with the stack. It
+     operate on this variable, rather than syncing with the stack. It
      needs to be copied to the stack when sp changes.  */
   top = 0;
 
@@ -1112,22 +1112,26 @@ gdb_eval_agent_expr (struct eval_agent_expr_context *ctx,
          break;
 
        case gdb_agent_op_ref8:
-         agent_mem_read (ctx, cnv.u8.bytes, (CORE_ADDR) top, 1);
+         if (agent_mem_read (ctx, cnv.u8.bytes, (CORE_ADDR) top, 1) != 0)
+           return expr_eval_invalid_memory_access;
          top = cnv.u8.val;
          break;
 
        case gdb_agent_op_ref16:
-         agent_mem_read (ctx, cnv.u16.bytes, (CORE_ADDR) top, 2);
+         if (agent_mem_read (ctx, cnv.u16.bytes, (CORE_ADDR) top, 2) != 0)
+           return expr_eval_invalid_memory_access;
          top = cnv.u16.val;
          break;
 
        case gdb_agent_op_ref32:
-         agent_mem_read (ctx, cnv.u32.bytes, (CORE_ADDR) top, 4);
+         if (agent_mem_read (ctx, cnv.u32.bytes, (CORE_ADDR) top, 4) != 0)
+           return expr_eval_invalid_memory_access;
          top = cnv.u32.val;
          break;
 
        case gdb_agent_op_ref64:
-         agent_mem_read (ctx, cnv.u64.bytes, (CORE_ADDR) top, 8);
+         if (agent_mem_read (ctx, cnv.u64.bytes, (CORE_ADDR) top, 8) != 0)
+           return expr_eval_invalid_memory_access;
          top = cnv.u64.val;
          break;