On Mon, 5 Jun 2006, Catherine Zhang wrote:

Looks ok to me, except for one minor nit:

> +     if (ctx) {
> +             rc = avc_has_perm(tsec->sid, ctx->ctx_sid,
> +             SECCLASS_ASSOCIATION,
> +             ASSOCIATION__SETCONTEXT, NULL);
> +     }

you don't need the braces here (similar again further in the patch).


- James
-- 
James Morris
<[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to