about summary refs log tree commit diff stats
path: root/cpp/037call_reply
diff options
context:
space:
mode:
authorKartik K. Agaram <vc@akkartik.com>2015-04-24 21:05:28 -0700
committerKartik K. Agaram <vc@akkartik.com>2015-04-24 21:05:28 -0700
commit9b035724e7277c8761a999d6a6dafd06382b776f (patch)
tree1f16157f935e2ada0bf2ae3f9c1c70b922c6fbe3 /cpp/037call_reply
parent5eb49929b0ccc3b1660e9d7d26ba5adec363fb68 (diff)
downloadmu-9b035724e7277c8761a999d6a6dafd06382b776f.tar.gz
1180 - finally dump that 'pc' reference
In the process we uncovered yet another out-of-bounds access, in the
implementation of 'reply'. Another sign from the gods that large-scope
pointers/references are a bad idea.
Diffstat (limited to 'cpp/037call_reply')
-rw-r--r--cpp/037call_reply3
1 files changed, 1 insertions, 2 deletions
diff --git a/cpp/037call_reply b/cpp/037call_reply
index d48a17de..9e62c1da 100644
--- a/cpp/037call_reply
+++ b/cpp/037call_reply
@@ -33,8 +33,7 @@ case REPLY: {
     trace("run") << "result " << i << " is " << to_string(callee_results[i]);
     write_memory(caller_instruction.products[i], callee_results[i]);
   }
-  ++current_step_index();  // caller
-  break;
+  break;  // instruction loop will increment caller's step_index
 }
 
 //: Products can include containers and exclusive containers, addresses and arrays.