From: Joy Latten <[EMAIL PROTECTED]>
Date: Fri, 9 Mar 2007 17:14:54 -0600

> I noticed that in xfrm_state_add we look for the larval SA in a few
> places without checking for protocol match. So when using both 
> AH and ESP, whichever one gets added first, deletes the larval SA. 
> It seems AH always gets added first and ESP is always the larval 
> SA's protocol since the xfrm->tmpl has it first. Thus causing the
> additional km_query()
> 
> Adding the check eliminates the double SA creation. 
> I know this may not seem like a complete solution and I will 
> continue to test and be on the lookout, but isn't having the
> check a good thing? So far I have tested SAs with just ESP, just AH
> and with both and all seems ok. 
> 
> Please let me know if this patch is ok. 
> My kernel was 2.6.20-rc3-git3.
> 
> Signed-off-by: Joy Latten <[EMAIL PROTECTED]>

This patch looks good, I'll push this to Linus and -stable.

Thanks!
-
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