]> git.saurik.com Git - apple/xnu.git/blobdiff - osfmk/kern/exception.c
xnu-7195.101.1.tar.gz
[apple/xnu.git] / osfmk / kern / exception.c
index a47544027c0cdbdf142f464ebe798da49e2dc1be..2060dc2a15b31d35ee0971884d66a1a80712c57a 100644 (file)
@@ -1,8 +1,8 @@
 /*
- * Copyright (c) 2000-2004 Apple Computer, Inc. All rights reserved.
+ * Copyright (c) 2000-2020 Apple Computer, Inc. All rights reserved.
  *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_START@
- * 
+ *
  * This file contains Original Code and/or Modifications of Original Code
  * as defined in and that are subject to the Apple Public Source License
  * Version 2.0 (the 'License'). You may not use this file except in
  * unlawful or unlicensed copies of an Apple operating system, or to
  * circumvent, violate, or enable the circumvention or violation of, any
  * terms of an Apple operating system software license agreement.
- * 
+ *
  * Please obtain a copy of the License at
  * http://www.opensource.apple.com/apsl/ and read it before using this file.
- * 
+ *
  * The Original Code and all software distributed under the License are
  * distributed on an 'AS IS' basis, WITHOUT WARRANTY OF ANY KIND, EITHER
  * EXPRESS OR IMPLIED, AND APPLE HEREBY DISCLAIMS ALL SUCH WARRANTIES,
  * FITNESS FOR A PARTICULAR PURPOSE, QUIET ENJOYMENT OR NON-INFRINGEMENT.
  * Please see the License for the specific language governing rights and
  * limitations under the License.
- * 
+ *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_END@
  */
 /*
  * @OSF_COPYRIGHT@
  */
-/* 
+/*
  * Mach Operating System
  * Copyright (c) 1991,1990,1989,1988,1987 Carnegie Mellon University
  * All Rights Reserved.
- * 
+ *
  * Permission to use, copy, modify and distribute this software and its
  * documentation is hereby granted, provided that both the copyright
  * notice and this permission notice appear in all copies of the
  * software, derivative works or modified versions, and any portions
  * thereof, and that both notices appear in supporting documentation.
- * 
+ *
  * CARNEGIE MELLON ALLOWS FREE USE OF THIS SOFTWARE IN ITS "AS IS"
  * CONDITION.  CARNEGIE MELLON DISCLAIMS ANY LIABILITY OF ANY KIND FOR
  * ANY DAMAGES WHATSOEVER RESULTING FROM THE USE OF THIS SOFTWARE.
- * 
+ *
  * Carnegie Mellon requests users of this software to return to
- * 
+ *
  *  Software Distribution Coordinator  or  Software.Distribution@CS.CMU.EDU
  *  School of Computer Science
  *  Carnegie Mellon University
  *  Pittsburgh PA 15213-3890
- * 
+ *
  * any improvements or extensions that they make and grant Carnegie Mellon
  * the rights to redistribute these changes.
  */
@@ -67,6 +67,7 @@
 #include <mach/exception_types.h>
 #include <mach/exc.h>
 #include <mach/mach_exc.h>
+
 #include <ipc/port.h>
 #include <ipc/ipc_entry.h>
 #include <ipc/ipc_object.h>
@@ -74,7 +75,7 @@
 #include <ipc/ipc_space.h>
 #include <ipc/ipc_pset.h>
 #include <ipc/ipc_machdep.h>
-#include <kern/counters.h>
+
 #include <kern/ipc_tt.h>
 #include <kern/task.h>
 #include <kern/thread.h>
 #include <kern/sched_prim.h>
 #include <kern/host.h>
 #include <kern/misc_protos.h>
+#include <kern/ux_handler.h>
+
+#include <vm/vm_map.h>
+
+#include <security/mac_mach_internal.h>
 #include <string.h>
+
 #include <pexpert/pexpert.h>
 
-extern int panic_on_exception_triage;
+bool panic_on_exception_triage = false;
 
 unsigned long c_thr_exc_raise = 0;
 unsigned long c_thr_exc_raise_state = 0;
@@ -97,26 +104,53 @@ unsigned long c_tsk_exc_raise_state_id = 0;
 
 /* forward declarations */
 kern_return_t exception_deliver(
-       thread_t                thread,
-       exception_type_t        exception,
-       mach_exception_data_t   code,
+       thread_t                thread,
+       exception_type_t        exception,
+       mach_exception_data_t   code,
        mach_msg_type_number_t  codeCnt,
        struct exception_action *excp,
-       lck_mtx_t                       *mutex);
+       lck_mtx_t                       *mutex);
 
 static kern_return_t
 check_exc_receiver_dependency(
-       exception_type_t exception, 
-       struct exception_action *excp, 
+       exception_type_t exception,
+       struct exception_action *excp,
        lck_mtx_t *mutex);
 
 #ifdef MACH_BSD
 kern_return_t bsd_exception(
-       exception_type_t        exception,
-       mach_exception_data_t   code,
+       exception_type_t        exception,
+       mach_exception_data_t   code,
        mach_msg_type_number_t  codeCnt);
 #endif /* MACH_BSD */
 
+#if __has_feature(ptrauth_calls)
+extern int exit_with_pac_exception(
+       void *proc,
+       exception_type_t         exception,
+       mach_exception_code_t    code,
+       mach_exception_subcode_t subcode);
+
+extern bool proc_is_traced(void *p);
+#endif /* __has_feature(ptrauth_calls) */
+
+/*
+ * Routine: exception_init
+ * Purpose:
+ *   Global initialization of state for exceptions.
+ * Conditions:
+ *   None.
+ */
+void
+exception_init(void)
+{
+       int tmp = 0;
+
+       if (PE_parse_boot_argn("-panic_on_exception_triage", &tmp, sizeof(tmp))) {
+               panic_on_exception_triage = true;
+       }
+}
+
 /*
  *     Routine:        exception_deliver
  *     Purpose:
@@ -129,39 +163,44 @@ kern_return_t bsd_exception(
  *     Returns:
  *             KERN_SUCCESS if the exception was handled
  */
-kern_return_t 
+kern_return_t
 exception_deliver(
-       thread_t                thread,
-       exception_type_t        exception,
-       mach_exception_data_t   code,
+       thread_t                thread,
+       exception_type_t        exception,
+       mach_exception_data_t   code,
        mach_msg_type_number_t  codeCnt,
        struct exception_action *excp,
-       lck_mtx_t                       *mutex)
+       lck_mtx_t                       *mutex)
 {
-       ipc_port_t              exc_port;
-       exception_data_type_t   small_code[EXCEPTION_CODE_MAX];
-       int                     code64;
-       int                     behavior;
-       int                     flavor;
-       kern_return_t           kr;
+       ipc_port_t              exc_port = IPC_PORT_NULL;
+       exception_data_type_t   small_code[EXCEPTION_CODE_MAX];
+       int                     code64;
+       int                     behavior;
+       int                     flavor;
+       kern_return_t           kr;
+       task_t task;
+       ipc_port_t thread_port = IPC_PORT_NULL, task_port = IPC_PORT_NULL;
 
        /*
         *  Save work if we are terminating.
         *  Just go back to our AST handler.
         */
-       if (!thread->active && !thread->inspection)
+       if (!thread->active && !thread->inspection) {
                return KERN_SUCCESS;
+       }
 
        /*
         * If there are no exception actions defined for this entity,
         * we can't deliver here.
         */
-       if (excp == NULL)
+       if (excp == NULL) {
                return KERN_FAILURE;
+       }
 
        assert(exception < EXC_TYPES_COUNT);
-       if (exception >= EXC_TYPES_COUNT)
+       if (exception >= EXC_TYPES_COUNT) {
                return KERN_FAILURE;
+       }
 
        excp = &excp[exception];
 
@@ -184,7 +223,7 @@ exception_deliver(
                lck_mtx_unlock(mutex);
                return KERN_FAILURE;
        }
-       ip_reference(exc_port); 
+       ip_reference(exc_port);
        exc_port->ip_srights++;
        ip_unlock(exc_port);
 
@@ -193,13 +232,39 @@ exception_deliver(
        lck_mtx_unlock(mutex);
 
        code64 = (behavior & MACH_EXCEPTION_CODES);
-       behavior &= ~MACH_EXCEPTION_CODES;
+       behavior &= ~MACH_EXCEPTION_MASK;
 
        if (!code64) {
                small_code[0] = CAST_DOWN_EXPLICIT(exception_data_type_t, code[0]);
                small_code[1] = CAST_DOWN_EXPLICIT(exception_data_type_t, code[1]);
        }
 
+       task = thread->task;
+
+#if CONFIG_MACF
+       /* Now is a reasonably good time to check if the exception action is
+        * permitted for this process, because after this point we will send
+        * the message out almost certainly.
+        * As with other failures, exception_triage_thread will go on
+        * to the next level.
+        */
+
+       /* The global exception-to-signal translation port is safe to be an exception handler. */
+       if (is_ux_handler_port(exc_port) == FALSE &&
+           mac_exc_action_check_exception_send(task, excp) != 0) {
+               kr = KERN_FAILURE;
+               goto out_release_right;
+       }
+#endif
+
+       if (behavior != EXCEPTION_STATE) {
+               task_reference(task);
+               task_port = convert_task_to_port(task);
+               /* task ref consumed */
+               thread_reference(thread);
+               thread_port = convert_thread_to_port(thread);
+               /* thread ref consumed */
+       }
 
        switch (behavior) {
        case EXCEPTION_STATE: {
@@ -208,54 +273,58 @@ exception_deliver(
 
                c_thr_exc_raise_state++;
                state_cnt = _MachineStateCount[flavor];
-               kr = thread_getstatus(thread, flavor, 
-                                     (thread_state_t)state,
-                                     &state_cnt);
+               kr = thread_getstatus_to_user(thread, flavor,
+                   (thread_state_t)state,
+                   &state_cnt);
                if (kr == KERN_SUCCESS) {
                        if (code64) {
-                               kr = mach_exception_raise_state(exc_port, 
-                                               exception,
-                                               code, 
-                                               codeCnt,
-                                               &flavor,
-                                               state, state_cnt,
-                                               state, &state_cnt);
+                               kr = mach_exception_raise_state(exc_port,
+                                   exception,
+                                   code,
+                                   codeCnt,
+                                   &flavor,
+                                   state, state_cnt,
+                                   state, &state_cnt);
                        } else {
                                kr = exception_raise_state(exc_port, exception,
-                                               small_code, 
-                                               codeCnt,
-                                               &flavor,
-                                               state, state_cnt,
-                                               state, &state_cnt);
+                                   small_code,
+                                   codeCnt,
+                                   &flavor,
+                                   state, state_cnt,
+                                   state, &state_cnt);
+                       }
+                       if (kr == KERN_SUCCESS) {
+                               if (exception != EXC_CORPSE_NOTIFY) {
+                                       kr = thread_setstatus_from_user(thread, flavor,
+                                           (thread_state_t)state,
+                                           state_cnt);
+                               }
+                               goto out_release_right;
                        }
-                       if (kr == MACH_MSG_SUCCESS && exception != EXC_CORPSE_NOTIFY)
-                               kr = thread_setstatus(thread, flavor, 
-                                               (thread_state_t)state,
-                                               state_cnt);
                }
 
-               return kr;
+               goto out_release_right;
        }
 
        case EXCEPTION_DEFAULT:
                c_thr_exc_raise++;
                if (code64) {
                        kr = mach_exception_raise(exc_port,
-                                       retrieve_thread_self_fast(thread),
-                                       retrieve_task_self_fast(thread->task),
-                                       exception,
-                                       code, 
-                                       codeCnt);
+                           thread_port,
+                           task_port,
+                           exception,
+                           code,
+                           codeCnt);
                } else {
                        kr = exception_raise(exc_port,
-                                       retrieve_thread_self_fast(thread),
-                                       retrieve_task_self_fast(thread->task),
-                                       exception,
-                                       small_code, 
-                                       codeCnt);
+                           thread_port,
+                           task_port,
+                           exception,
+                           small_code,
+                           codeCnt);
                }
 
-               return kr;
+               goto out_release_right;
 
        case EXCEPTION_STATE_IDENTITY: {
                mach_msg_type_number_t state_cnt;
@@ -263,45 +332,66 @@ exception_deliver(
 
                c_thr_exc_raise_state_id++;
                state_cnt = _MachineStateCount[flavor];
-               kr = thread_getstatus(thread, flavor,
-                                     (thread_state_t)state,
-                                     &state_cnt);
+               kr = thread_getstatus_to_user(thread, flavor,
+                   (thread_state_t)state,
+                   &state_cnt);
                if (kr == KERN_SUCCESS) {
                        if (code64) {
                                kr = mach_exception_raise_state_identity(
-                                               exc_port,
-                                               retrieve_thread_self_fast(thread),
-                                               retrieve_task_self_fast(thread->task),
-                                               exception,
-                                               code, 
-                                               codeCnt,
-                                               &flavor,
-                                               state, state_cnt,
-                                               state, &state_cnt);
+                                       exc_port,
+                                       thread_port,
+                                       task_port,
+                                       exception,
+                                       code,
+                                       codeCnt,
+                                       &flavor,
+                                       state, state_cnt,
+                                       state, &state_cnt);
                        } else {
                                kr = exception_raise_state_identity(exc_port,
-                                               retrieve_thread_self_fast(thread),
-                                               retrieve_task_self_fast(thread->task),
-                                               exception,
-                                               small_code, 
-                                               codeCnt,
-                                               &flavor,
-                                               state, state_cnt,
-                                               state, &state_cnt);
+                                   thread_port,
+                                   task_port,
+                                   exception,
+                                   small_code,
+                                   codeCnt,
+                                   &flavor,
+                                   state, state_cnt,
+                                   state, &state_cnt);
+                       }
+
+                       if (kr == KERN_SUCCESS) {
+                               if (exception != EXC_CORPSE_NOTIFY) {
+                                       kr = thread_setstatus_from_user(thread, flavor,
+                                           (thread_state_t)state,
+                                           state_cnt);
+                               }
+                               goto out_release_right;
                        }
-                       if (kr == MACH_MSG_SUCCESS && exception != EXC_CORPSE_NOTIFY)
-                               kr = thread_setstatus(thread, flavor,
-                                               (thread_state_t)state,
-                                               state_cnt);
                }
 
-               return kr;
+               goto out_release_right;
        }
 
        default:
-              panic ("bad exception behavior!");
-              return KERN_FAILURE; 
+               panic("bad exception behavior!");
+               return KERN_FAILURE;
        }/* switch */
+
+out_release_right:
+
+       if (task_port) {
+               ipc_port_release_send(task_port);
+       }
+
+       if (thread_port) {
+               ipc_port_release_send(thread_port);
+       }
+
+       if (exc_port) {
+               ipc_port_release_send(exc_port);
+       }
+
+       return kr;
 }
 
 /*
@@ -326,24 +416,27 @@ check_exc_receiver_dependency(
 {
        kern_return_t retval = KERN_SUCCESS;
 
-       if (excp == NULL || exception != EXC_CRASH)
+       if (excp == NULL || exception != EXC_CRASH) {
                return retval;
+       }
 
        task_t task = current_task();
        lck_mtx_lock(mutex);
        ipc_port_t xport = excp[exception].port;
-       if ( IP_VALID(xport)
-                    && ip_active(xport)
-                    && task->itk_space == xport->ip_receiver)
+       if (IP_VALID(xport)
+           && ip_active(xport)
+           && task->itk_space == xport->ip_receiver) {
                retval = KERN_FAILURE;
+       }
        lck_mtx_unlock(mutex);
        return retval;
 }
 
+
 /*
- *     Routine:        exception_triage
+ *     Routine:        exception_triage_thread
  *     Purpose:
- *             The current thread caught an exception.
+ *             The thread caught an exception.
  *             We make an up-call to the thread's exception server.
  *     Conditions:
  *             Nothing locked and no resources held.
@@ -354,16 +447,17 @@ check_exc_receiver_dependency(
  *             KERN_SUCCESS if exception is handled by any of the handlers.
  */
 kern_return_t
-exception_triage(
-       exception_type_t        exception,
-       mach_exception_data_t   code,
-       mach_msg_type_number_t  codeCnt)
+exception_triage_thread(
+       exception_type_t        exception,
+       mach_exception_data_t   code,
+       mach_msg_type_number_t  codeCnt,
+       thread_t                thread)
 {
-       thread_t                thread;
-       task_t                  task;
-       host_priv_t             host_priv;
-       lck_mtx_t               *mutex;
-       kern_return_t   kr = KERN_FAILURE;
+       task_t                  task;
+       host_priv_t             host_priv;
+       lck_mtx_t               *mutex;
+       kern_return_t   kr = KERN_FAILURE;
+
 
        assert(exception != EXC_RPC_ALERT);
 
@@ -379,29 +473,27 @@ exception_triage(
                panic("called exception_triage when it was forbidden by the boot environment");
        }
 
-       thread = current_thread();
-
        /*
         * Try to raise the exception at the activation level.
         */
        mutex = &thread->mutex;
-       if (KERN_SUCCESS == check_exc_receiver_dependency(exception, thread->exc_actions, mutex))
-       {
+       if (KERN_SUCCESS == check_exc_receiver_dependency(exception, thread->exc_actions, mutex)) {
                kr = exception_deliver(thread, exception, code, codeCnt, thread->exc_actions, mutex);
-               if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED)
+               if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED) {
                        goto out;
+               }
        }
 
        /*
         * Maybe the task level will handle it.
         */
-       task = current_task();
-       mutex = &task->lock;
-       if (KERN_SUCCESS == check_exc_receiver_dependency(exception, task->exc_actions, mutex))
-       {
+       task = thread->task;
+       mutex = &task->itk_lock_data;
+       if (KERN_SUCCESS == check_exc_receiver_dependency(exception, task->exc_actions, mutex)) {
                kr = exception_deliver(thread, exception, code, codeCnt, task->exc_actions, mutex);
-               if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED)
+               if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED) {
                        goto out;
+               }
        }
 
        /*
@@ -409,43 +501,98 @@ exception_triage(
         */
        host_priv = host_priv_self();
        mutex = &host_priv->lock;
-       
-       if (KERN_SUCCESS == check_exc_receiver_dependency(exception, host_priv->exc_actions, mutex))
-       {
+
+       if (KERN_SUCCESS == check_exc_receiver_dependency(exception, host_priv->exc_actions, mutex)) {
                kr = exception_deliver(thread, exception, code, codeCnt, host_priv->exc_actions, mutex);
-               if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED)
+               if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED) {
                        goto out;
+               }
        }
 
 out:
        if ((exception != EXC_CRASH) && (exception != EXC_RESOURCE) &&
-           (exception != EXC_GUARD) && (exception != EXC_CORPSE_NOTIFY))
+           (exception != EXC_GUARD) && (exception != EXC_CORPSE_NOTIFY)) {
                thread_exception_return();
+       }
        return kr;
 }
 
+/*
+ *     Routine:        exception_triage
+ *     Purpose:
+ *             The current thread caught an exception.
+ *             We make an up-call to the thread's exception server.
+ *     Conditions:
+ *             Nothing locked and no resources held.
+ *             Called from an exception context, so
+ *             thread_exception_return and thread_kdb_return
+ *             are possible.
+ *     Returns:
+ *             KERN_SUCCESS if exception is handled by any of the handlers.
+ */
+int debug4k_panic_on_exception = 0;
+kern_return_t
+exception_triage(
+       exception_type_t        exception,
+       mach_exception_data_t   code,
+       mach_msg_type_number_t  codeCnt)
+{
+       thread_t thread = current_thread();
+       if (VM_MAP_PAGE_SIZE(thread->task->map) < PAGE_SIZE) {
+               DEBUG4K_EXC("thread %p task %p map %p exception %d codes 0x%llx 0x%llx \n", thread, thread->task, thread->task->map, exception, code[0], code[1]);
+               if (debug4k_panic_on_exception) {
+                       panic("DEBUG4K %s:%d thread %p task %p map %p exception %d codes 0x%llx 0x%llx \n", __FUNCTION__, __LINE__, thread, thread->task, thread->task->map, exception, code[0], code[1]);
+               }
+       }
+#if __has_feature(ptrauth_calls)
+       /*
+        * If it is a ptrauth violation, then check if the task has the TF_PAC_EXC_FATAL
+        * flag set and isn't being ptraced. If so, terminate the task via exit_with_reason
+        */
+       if (exception & EXC_PTRAUTH_BIT) {
+               exception &= ~EXC_PTRAUTH_BIT;
+
+               boolean_t traced_flag = FALSE;
+               task_t task = thread->task;
+               void *proc = task->bsd_info;
+
+               if (task->bsd_info) {
+                       traced_flag = proc_is_traced(proc);
+               }
+
+               if (task_is_pac_exception_fatal(current_task()) && !traced_flag) {
+                       exit_with_pac_exception(proc, exception, code[0], code[1]);
+                       thread_exception_return();
+                       /* NOT_REACHABLE */
+               }
+       }
+#endif /* __has_feature(ptrauth_calls) */
+       return exception_triage_thread(exception, code, codeCnt, thread);
+}
+
 kern_return_t
 bsd_exception(
-       exception_type_t        exception,
-       mach_exception_data_t   code,
+       exception_type_t        exception,
+       mach_exception_data_t   code,
        mach_msg_type_number_t  codeCnt)
 {
-       task_t                  task;
-       lck_mtx_t               *mutex;
-       thread_t                self = current_thread();
-       kern_return_t           kr;
+       task_t                  task;
+       lck_mtx_t               *mutex;
+       thread_t                self = current_thread();
+       kern_return_t           kr;
 
        /*
         * Maybe the task level will handle it.
         */
        task = current_task();
-       mutex = &task->lock;
+       mutex = &task->itk_lock_data;
 
        kr = exception_deliver(self, exception, code, codeCnt, task->exc_actions, mutex);
 
-       if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED)
-               return(KERN_SUCCESS);
-       return(KERN_FAILURE);
+       if (kr == KERN_SUCCESS || kr == MACH_RCV_PORT_DIED) {
+               return KERN_SUCCESS;
+       }
+       return KERN_FAILURE;
 }
 
 
@@ -453,11 +600,12 @@ bsd_exception(
  * Raise an exception on a task.
  * This should tell launchd to launch Crash Reporter for this task.
  */
-kern_return_t task_exception_notify(exception_type_t exception,
-       mach_exception_data_type_t exccode, mach_exception_data_type_t excsubcode)
+kern_return_t
+task_exception_notify(exception_type_t exception,
+    mach_exception_data_type_t exccode, mach_exception_data_type_t excsubcode)
 {
-       mach_exception_data_type_t      code[EXCEPTION_CODE_MAX];
-       wait_interrupt_t                wsave;
+       mach_exception_data_type_t      code[EXCEPTION_CODE_MAX];
+       wait_interrupt_t                wsave;
        kern_return_t kr = KERN_SUCCESS;
 
        code[0] = exccode;
@@ -474,39 +622,38 @@ kern_return_t task_exception_notify(exception_type_t exception,
  *     Handle interface for special performance monitoring
  *     This is a special case of the host exception handler
  */
-kern_return_t sys_perf_notify(thread_t thread, int pid) 
+kern_return_t
+sys_perf_notify(thread_t thread, int pid)
 {
-       host_priv_t             hostp;
-       ipc_port_t              xport;
-       wait_interrupt_t        wsave;
-       kern_return_t           ret;
+       host_priv_t             hostp;
+       ipc_port_t              xport;
+       wait_interrupt_t        wsave;
+       kern_return_t           ret;
 
-       hostp = host_priv_self();       /* Get the host privileged ports */
-       mach_exception_data_type_t      code[EXCEPTION_CODE_MAX];
-       code[0] = 0xFF000001;           /* Set terminate code */
-       code[1] = pid;          /* Pass out the pid */
+       hostp = host_priv_self();       /* Get the host privileged ports */
+       mach_exception_data_type_t      code[EXCEPTION_CODE_MAX];
+       code[0] = 0xFF000001;           /* Set terminate code */
+       code[1] = pid;          /* Pass out the pid */
 
        struct task *task = thread->task;
-       xport = hostp->exc_actions[EXC_RPC_ALERT].port; 
+       xport = hostp->exc_actions[EXC_RPC_ALERT].port;
 
        /* Make sure we're not catching our own exception */
        if (!IP_VALID(xport) ||
-                       !ip_active(xport) ||
-                       task->itk_space == xport->data.receiver) {
-
-               return(KERN_FAILURE);   
+           !ip_active(xport) ||
+           task->itk_space == xport->data.receiver) {
+               return KERN_FAILURE;
        }
 
-       wsave = thread_interrupt_level(THREAD_UNINT);   
+       wsave = thread_interrupt_level(THREAD_UNINT);
        ret = exception_deliver(
-                       thread,
-                       EXC_RPC_ALERT, 
-                       code, 
-                       2, 
-                       hostp->exc_actions,
-                       &hostp->lock);
+               thread,
+               EXC_RPC_ALERT,
+               code,
+               2,
+               hostp->exc_actions,
+               &hostp->lock);
        (void)thread_interrupt_level(wsave);
 
-       return(ret);
+       return ret;
 }
-