Don’t trigger EXC_CORPSE_NOTIFY on OS X 10.11

tools/mac/catch_exception_tool.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 61 matching lines @@
       ConstThreadState old_state,
       mach_msg_type_number_t old_state_count,
       thread_state_t new_state,
       mach_msg_type_number_t* new_state_count,
       const mach_msg_trailer_t* trailer,
       bool* destroy_complex_request) override {
     *destroy_complex_request = true;
     ++*exceptions_handled_;
 
     fprintf(options_.file,
-            "%s: behavior %s, ",
+            "%s: behavior %s",
             me_.c_str(),
             ExceptionBehaviorToString(
                 behavior, kUseFullName | kUnknownIsNumeric | kUseOr).c_str());
 
     kern_return_t kr;
     if (ExceptionBehaviorHasIdentity(behavior)) {
-      pid_t pid;
-      kr = pid_for_task(task, &pid);
-      if (kr != KERN_SUCCESS) {
-        MACH_LOG(ERROR, kr) << "pid_for_task";
-        return KERN_FAILURE;
+      // It’s not possible to call pid_for_task() once EXC_CORPSE_NOTIFY has
+      // been generated. It is possible to obtain the process ID by mapping the
+      // corpse kcdata area from the task’s address space at code[0] (size
+      // code[1]) and locating TASK_CRASHINFO_PID within that area. This area
+      // also includes TASK_CRASHINFO_CRASHED_THREADID which could be used
+      // instead of thread_info() below, and TASK_CRASHINFO_EXCEPTION_CODES
+      // which could be used to recover the exception codes passed to the
+      // EXC_CRASH handler. None of this is currently done because corpses are a
+      // new 10.11-only feature. See 10.11 <corpses/task_corpse.h> and
+      // <kern/kern_cdata.h>.
+      if (exception != EXC_CORPSE_NOTIFY) {
+        pid_t pid;
+        kr = pid_for_task(task, &pid);
+        if (kr != KERN_SUCCESS) {
+          fprintf(options_.file, "\n");
+          fflush(options_.file);
+          MACH_LOG(ERROR, kr) << "pid_for_task";
+          return KERN_FAILURE;
+        }
+        fprintf(options_.file, ", pid %d", pid);
       }
-      fprintf(options_.file, "pid %d, ", pid);
 
       thread_identifier_info identifier_info;
       mach_msg_type_number_t count = THREAD_IDENTIFIER_INFO_COUNT;
       kr = thread_info(thread,
                        THREAD_IDENTIFIER_INFO,
                        reinterpret_cast<thread_info_t>(&identifier_info),
                        &count);
       if (kr != KERN_SUCCESS) {
+        fprintf(options_.file, "\n");
+        fflush(options_.file);
         MACH_LOG(ERROR, kr) << "thread_info";
         return KERN_FAILURE;
       }
-      fprintf(options_.file, "thread %lld, ", identifier_info.thread_id);
+      fprintf(options_.file, ", thread %lld", identifier_info.thread_id);
     }
 
     fprintf(
         options_.file,
-        "exception %s, codes[%d] ",
+        ", exception %s, codes[%d]",
         ExceptionToString(exception, kUseFullName | kUnknownIsNumeric).c_str(),
         code_count);
 
     for (size_t index = 0; index < code_count; ++index) {
       fprintf(options_.file,
-              "%#llx%s",
-              code[index],
-              index != code_count - 1 ? ", " : "");
+              "%s %#llx",
+              index != 0 ? "," : "",
+              code[index]);
     }
 
     if (exception == EXC_CRASH) {
       mach_exception_code_t original_code_0;
       int signal;
       exception_type_t original_exception =
           ExcCrashRecoverOriginalException(code[0], &original_code_0, &signal);
       fprintf(options_.file,
               ", original exception %s, original code[0] %lld, signal %s",
               ExceptionToString(original_exception,
@@ skipping 15 matching lines @@
     fflush(options_.file);
 
     if (exception != EXC_CRASH && exception != kMachExceptionSimulated) {
       // Find another handler.
       return KERN_FAILURE;
     }
 
     ExcServerCopyState(
         behavior, old_state, old_state_count, new_state, new_state_count);
 
-    return ExcServerSuccessfulReturnValue(behavior, false);
+    return ExcServerSuccessfulReturnValue(exception, behavior, false);
   }
 
  private:
   const Options& options_;
   const std::string& me_;
   int* exceptions_handled_;
 };
 
 void Usage(const std::string& me) {
   fprintf(stderr,
@@ skipping 138 matching lines @@
 
   return EXIT_SUCCESS;
 }
 
 }  // namespace
 }  // namespace crashpad
 
 int main(int argc, char* argv[]) {
   return crashpad::CatchExceptionToolMain(argc, argv);
 }