On Monday 17 December 2007 2:56:41 pm Stephen Smalley wrote:
> On Fri, 2007-12-14 at 16:50 -0500, Paul Moore wrote:
> > /* Initialize an AVC audit data structure. */
> > #define AVC_AUDIT_DATA_INIT(_d,_t) \
> > - { memset((_d), 0, sizeof(struct avc_audit_data)); (_d)->type =
> > AVC_AUDIT_DATA_##_t; }
> > + { memset((_d), 0, sizeof(struct
> > avc_audit_data)); \
> > + (_d)->type = AVC_AUDIT_DATA_##_t; \
> > + if ((_d)->type == AVC_AUDIT_DATA_NET) \
> > + (_d)->u.net.netif = -1; }
>
> As a minor nit, at the same time you do this, turn this into a static
> inline function please.
Can do ... although when talking about this with some other people I'm not
sure I need to use -1 as a sentinel value, zero should work. If that is the
case I won't need to change this definition.
--
paul moore
linux security @ hp
-
To unsubscribe from this list: send the line "unsubscribe
linux-security-module" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
