[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20071113.213622.169351284.davem@davemloft.net>
Date: Tue, 13 Nov 2007 21:36:22 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: herbert@...dor.apana.org.au
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH 8/24] [IPSEC]: Make sure idev is consistent with dev in
xfrm_dst
From: Herbert Xu <herbert@...dor.apana.org.au>
Date: Wed, 07 Nov 2007 22:08:24 +0800
> [IPSEC]: Make sure idev is consistent with dev in xfrm_dst
>
> Previously we took the device from the bottom route and idev from the top
> route. This is bad because idev may well point to a different device.
> This patch changes it so that we get the idev from the device directly.
>
> It also makes it an error if either dev or idev is NULL. This is consistent
> with the rest of the routing code which also treats these cases as errors.
>
> I've removed the err initialisation in xfrm6_policy.c because it achieves
> no purpose and hid a bug when an initial version of this patch neglected
> to set err to -ENODEV (fortunately the IPv4 version warned about it).
>
> Signed-off-by: Herbert Xu <herbert@...dor.apana.org.au>
Applied to net-2.6.25
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists