When the entire op fails, the recv_message is always discarded.

10 years ago · 467d7bd414
parent 0b7c4d29d4
commit 467d7bd414
2 changed files with 2 additions and 2 deletions
--- a/src/cpp/common/call.cc
+++ b/src/cpp/common/call.cc
@ -256,7 +256,7 @@ void CallOpBuffer::FinalizeResult(void** tag, bool* status) {
  if (recv_message_) {
    if (recv_message_buf_) {
      got_message = true;
-      *status &= DeserializeProto(recv_message_buf_, recv_message_);
+      *status = *status && DeserializeProto(recv_message_buf_, recv_message_);
      grpc_byte_buffer_destroy(recv_message_buf_);
      recv_message_buf_ = nullptr;
    } else {
--- a/src/cpp/server/server.cc
+++ b/src/cpp/server/server.cc
@ -314,7 +314,7 @@ class Server::AsyncRequest final : public CompletionQueueTag {
    *tag = tag_;
    if (*status && request_) {
      if (payload_) {
-        *status &= DeserializeProto(payload_, request_);
+        *status = *status && DeserializeProto(payload_, request_);
      } else {
        *status = false;
      }