Herbert Xu a écrit :
Eric Dumazet <[EMAIL PROTECTED]> wrote:
- C(sp);
-#ifdef CONFIG_INET
- secpath_get(skb->sp);
+#ifdef CONFIG_XFRM
+ n->sp = secpath_get(skb->sp);
#endif
Please move the ifdef into xfrm.h while you're at it. That is, make
secpath_get a no-op if CONFIG_XFRM is not defined.
Thanks,
Hum, but then we need a new macro or prototype, because n->sp is not valid
n->sp = secpath_get(skb->sp);
would still miscompile, even if secpath_get() is a no-op
Maybe a new secpath_copy() declared in include/net/xfrm.h ?
static inline void
secpath_copy(struct sk_buff *dst,
struct_sk_buff *src)
{
#ifdef CONFIG_XFRM
dst->sp = secpath_get(src);
#endif
}
Eric
-
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
