ChildPortHandshake: allow receive rights to be received

util/mach/mach_message_server_test.cc

 // Copyright 2014 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 198 matching lines @@
     struct ReceiveLargeRequestMessage : public LargeRequestMessage {
       mach_msg_trailer_t trailer;
     };
 
     const ReceiveRequestMessage* request =
         reinterpret_cast<const ReceiveRequestMessage*>(in);
     const mach_msg_bits_t expect_msgh_bits =
         MACH_MSGH_BITS(MACH_MSG_TYPE_MOVE_SEND, MACH_MSG_TYPE_MOVE_SEND) |
         (options_.client_send_complex ? MACH_MSGH_BITS_COMPLEX : 0);
     EXPECT_EQ(expect_msgh_bits, request->header.msgh_bits);
-    EXPECT_EQ(options_.client_send_large ? sizeof(LargeRequestMessage) :
-                                           sizeof(RequestMessage),
+    EXPECT_EQ(options_.client_send_large ? sizeof(LargeRequestMessage)
+                                         : sizeof(RequestMessage),
               request->header.msgh_size);
     if (options_.client_reply_port_type == Options::kReplyPortNormal) {
       EXPECT_EQ(RemotePort(), request->header.msgh_remote_port);
     }
     EXPECT_EQ(LocalPort(), request->header.msgh_local_port);
     EXPECT_EQ(kRequestMessageID, request->header.msgh_id);
     if (options_.client_send_complex) {
       EXPECT_EQ(1u, request->body.msgh_descriptor_count);
       EXPECT_NE(kMachPortNull, request->port_descriptor.name);
       parent_complex_message_port_ = request->port_descriptor.name;
@@ skipping 39 matching lines @@
     reply->Head.msgh_id = kReplyMessageID;
     reply->NDR = NDR_record;
     reply->RetCode = options_.server_mig_retcode;
     reply->number = replies_++;
 
     return true;
   }
 
   std::set<mach_msg_id_t> MachMessageServerRequestIDs() override {
     const mach_msg_id_t request_ids[] = {kRequestMessageID};
-    return std::set<mach_msg_id_t>(
-        &request_ids[0], &request_ids[arraysize(request_ids)]);
+    return std::set<mach_msg_id_t>(&request_ids[0],
+                                   &request_ids[arraysize(request_ids)]);
   }
 
   mach_msg_size_t MachMessageServerRequestSize() override {
     return sizeof(RequestMessage);
   }
 
   mach_msg_size_t MachMessageServerReplySize() override {
     return sizeof(ReplyMessage);
   }
 
@@ skipping 69 matching lines @@
 
       if (!options_.expect_server_destroyed_complex) {
         // MachMessageServer should not have destroyed the resources sent in the
         // complex request message.
         kr = mach_port_type(
             mach_task_self(), parent_complex_message_port_, &type);
         EXPECT_EQ(KERN_SUCCESS, kr) << MachErrorMessage(kr, "mach_port_type");
         EXPECT_EQ(MACH_PORT_TYPE_SEND, type);
 
         // Destroy the resources here.
-        kr = mach_port_deallocate(
-            mach_task_self(), parent_complex_message_port_);
+        kr = mach_port_deallocate(mach_task_self(),
+                                  parent_complex_message_port_);
         EXPECT_EQ(KERN_SUCCESS, kr)
             << MachErrorMessage(kr, "mach_port_deallocate");
       }
 
       // The kernel won’t have reused the same name for another Mach port in
       // this task so soon. It’s possible that something else in this task could
       // have reused the name, but it’s unlikely for that to have happened in
       // this test environment.
-      kr = mach_port_type(
-          mach_task_self(), parent_complex_message_port_, &type);
+      kr =
+          mach_port_type(mach_task_self(), parent_complex_message_port_, &type);
       EXPECT_EQ(KERN_INVALID_NAME, kr)
           << MachErrorMessage(kr, "mach_port_type");
     }
 
     if (options_.child_wait_for_parent_pipe_late) {
       // Let the child know it’s safe to exit.
       char c = '\0';
       CheckedWriteFile(WritePipeHandle(), &c, 1);
     }
   }
@@ skipping 452 matching lines @@
   options.expect_server_transaction_count = 0;
   options.client_send_large = true;
   options.client_expect_reply = false;
   TestMachMessageServer test_mach_message_server(options);
   test_mach_message_server.Test();
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace crashpad