[mojo-edk] Fix uninitialized reads in EDK dispatchers

mojo/edk/system/data_pipe_producer_dispatcher.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "mojo/edk/system/data_pipe_producer_dispatcher.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <utility>
@@ skipping 478 matching lines @@
   UpdateSignalsStateNoLock();
 }
 
 void DataPipeProducerDispatcher::UpdateSignalsStateNoLock() {
   lock_.AssertAcquired();
 
   bool was_peer_closed = peer_closed_;
   size_t previous_capacity = available_capacity_;
 
   ports::PortStatus port_status;
-  if (node_controller_->node()->GetStatus(control_port_, &port_status) !=
-          ports::OK ||
-      !port_status.receiving_messages) {
+  int rv = node_controller_->node()->GetStatus(control_port_, &port_status);
+  if (rv != ports::OK || !port_status.receiving_messages) {
     DVLOG(1) << "Data pipe producer " << pipe_id_ << " is aware of peer closure"
              << " [control_port=" << control_port_.name() << "]";
-
     peer_closed_ = true;
-  }
-
-  if (port_status.has_messages && !in_transit_) {
+  } else if (rv == ports::OK && port_status.has_messages && !in_transit_) {
     ports::ScopedMessage message;
     do {
       int rv = node_controller_->node()->GetMessageIf(control_port_, nullptr,
                                                       &message);
       if (rv != ports::OK)
         peer_closed_ = true;
       if (message) {
         PortsMessage* ports_message = static_cast<PortsMessage*>(message.get());
         const DataPipeControlMessage* m =
             static_cast<const DataPipeControlMessage*>(
@@ skipping 21 matching lines @@
   }
 
   if (peer_closed_ != was_peer_closed ||
       available_capacity_ != previous_capacity) {
     awakable_list_.AwakeForStateChange(GetHandleSignalsStateNoLock());
   }
 }
 
 }  // namespace edk
 }  // namespace mojo