re: clockctl clock_settime bug

[matthew green <mrg%eterna.com.au@localhost>]

   > !          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.


probably.  i was copying style from above 8-/


.mrg.