> ! if (args->tp) { > ! error = copyin(args->tp, &ts, sizeof ts); > ! if (error) > ! return (error); > ! error = clock_settime1(l->l_proc, args->clock_id, > ! &ts, false); > ! } We shouldn't do the `if (args->tp)' test. If NULL is passed, it is an error isn't it? Just let the copyin to DTRT. enami.