On Wednesday 29 October 2008 10:10:36 pm Manish Katiyar wrote: > On Thu, Oct 30, 2008 at 1:13 AM, Paul Moore <paul.moore@xxxxxx> wrote: > > On Wednesday 29 October 2008 2:27:44 pm you wrote: > >> Hi Paul, > >> > >> Does this look better ?? Appreciate your help. Patch compile > >> tested. > >> > >> Enable netlabel auditing functions only when CONFIG_AUDIT is set > >> > >> Signed-off-by: Manish Katiyar <mkatiyar@xxxxxxxxx> > > > > Much better, just some style nits noted below that need to be > > addressed ... > > Hi Paul, > > Updated patch below as per your comments. > > Enable netlabel auditing functions only when CONFIG_AUDIT is set > > Signed-off-by: Manish Katiyar <mkatiyar@xxxxxxxxx> Thanks for sticking with this, I appreciate it. There is still one small style nit (see below) but it is really my fault for not being more specific. Unless you object I'll go ahead and fix it up when I apply the patch. I'll send a git pull request out to netdev later today, including your patch, as there are two other labeled networking patches that need to make the next 2.6.28-rcX release. > --- > net/netlabel/netlabel_addrlist.c | 2 ++ > net/netlabel/netlabel_addrlist.h | 22 ++++++++++++++++++++++ > 2 files changed, 24 insertions(+), 0 deletions(-) > > diff --git a/net/netlabel/netlabel_addrlist.c > b/net/netlabel/netlabel_addrlist.c index b0925a3..3fe73c2 100644 > --- a/net/netlabel/netlabel_addrlist.c > +++ b/net/netlabel/netlabel_addrlist.c > @@ -327,6 +327,7 @@ struct netlbl_af6list > *netlbl_af6list_remove(const struct in6_addr *addr, > * Write the IPv4 address and address mask, if necessary, to > @audit_buf. * > */ > +#ifdef CONFIG_AUDIT > void netlbl_af4list_audit_addr(struct audit_buffer *audit_buf, > int src, const char *dev, > __be32 addr, __be32 mask) When I said to move the #ifdef below the comment, I meant below the "Audit Helper Functions" comment but before the function's kdoc comment header. -- paul moore linux @ hp -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html