[mojo-edk] Fix lifetime management of rewritten Windows HANDLEs

mojo/edk/system/node_controller.cc

 // Copyright 2016 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/node_controller.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/bind.h"
@@ skipping 893 matching lines @@
 
   // The parent should always know which process this came from.
   DCHECK(from_process != base::kNullProcessHandle);
 
 #if defined(OS_WIN)
   // Rewrite the handles to this (the parent) process. If the message is
   // destined for another child process, the handles will be rewritten to that
   // process before going out (see NodeChannel::WriteChannelMessage).
   //
   // TODO: We could avoid double-duplication.
+  //
+  // Note that we explicitly mark the handles as being owned by the sending
+  // process before rewriting them, in order to accommodate RewriteHandles'
+  // internal sanity checks.
+  for (size_t i = 0; i < message->num_handles(); ++i)
+    message->handles()[i].owning_process = from_process;
   if (!Channel::Message::RewriteHandles(from_process,
                                         base::GetCurrentProcessHandle(),
                                         message->handles(),
                                         message->num_handles())) {
     DLOG(ERROR) << "Failed to relay one or more handles.";
   }
 #else
   MachPortRelay* relay = GetMachPortRelay();
   if (!relay) {
     LOG(ERROR) << "Receiving Mach ports without a port relay from "
@@ skipping 71 matching lines @@
     shutdown_callback_flag_.Set(false);
   }
 
   DCHECK(!callback.is_null());
 
   callback.Run();
 }
 
 }  // namespace edk
 }  // namespace mojo