> I was just about to push this: Ah, didn't see your mail. Just pushed a fix.
> proc_set_cred(struct proc *p, struct ucred *newcred)
> {
> - bool success = _proc_set_cred(p, newcred, false);
> +#ifdef INVARIANTS
> + bool success =
> +#endif
> + _proc_set_cred(p, newcred, false);
>
> MPASS(success);
Apparently, the idiom for such a situation is to tag the variable with
`__diagused`, but the change above would have worked the same in the end.
Thanks for watching the build!
(And I now have one more thing on my before-commit list, since I forgot
GENERIC-NODEBUG).
--
Olivier Certner
signature.asc
Description: This is a digitally signed message part.
