Doing EV validation before accessing it.

Harshasa · Mar 18, 2015 · d678059 · d678059
1 parent 6c398b3
commit d678059
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/cpp/common/completion_queue.cc b/src/cpp/common/completion_queue.cc
@@ -95,8 +95,8 @@ bool CompletionQueue::Pluck(CompletionQueueTag* tag) {
 void CompletionQueue::TryPluck(CompletionQueueTag* tag) {
   std::unique_ptr<grpc_event, EventDeleter> ev;
 
-  ev.reset(grpc_completion_queue_pluck(cq_, tag, gpr_inf_past));
   if (!ev) return;
+  ev.reset(grpc_completion_queue_pluck(cq_, tag, gpr_inf_past));
   bool ok = ev->data.op_complete == GRPC_OP_OK;
   void* ignored = tag;
   // the tag must be swallowed if using TryPluck