Matthew Dillon wrote:

> -     mtx_lock(&Giant);
> -     td->td_retval[0] = p->p_ucred->cr_ruid;
> +     s = mtx_lock_giant(kern_giant_ucred);
> +     td->td_retval[0] = td->td_ucred->cr_ruid;
>  #if defined(COMPAT_43) || defined(COMPAT_SUNOS)
> -     td->td_retval[1] = p->p_ucred->cr_uid;
> +     td->td_retval[1] = td->td_ucred->cr_uid;
>  #endif
> -     mtx_unlock(&Giant);
> +     mtx_unlock_giant(s);
>       return (0);

What a waste.. John has already done all this stuff already (using
td_ucred instead of p_ucred) over the entire tree.

Cheers,
-Peter
--
Peter Wemm - [EMAIL PROTECTED]; [EMAIL PROTECTED]; [EMAIL PROTECTED]
"All of this is for nothing if we don't go to the stars" - JMS/B5


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to