• Herbert Xu's avatar
    [ICMP]: Ensure that ICMP relookup maintains status quo · af268182
    Herbert Xu authored
    The ICMP relookup path is only meant to modify behaviour when
    appropriate IPsec policies are in place and marked as requiring
    relookups.  It is certainly not meant to modify behaviour when
    IPsec policies don't exist at all.
    
    However, due to an oversight on the error paths existing behaviour
    may in fact change should one of the relookup steps fail.
    
    This patch corrects this by redirecting all errors on relookup
    failures to the previous code path.  That is, if the initial
    xfrm_lookup let the packet pass, we will stand by that decision
    should the relookup fail due to an error.
    
    This should be safe from a security point-of-view because compliant
    systems must install a default deny policy so the packet would'nt
    have passed in that case.
    
    Many thanks to Julian Anastasov for pointing out this error.
    Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    af268182
icmp.c 20.7 KB