ah: Don't return NET_XMIT_DROP on input.
When the ahash driver returns -EBUSY, AH4/6 input functions return NET_XMIT_DROP, presumably copied from the output code path. But returning transmit codes on input doesn't make a lot of sense. Since NET_XMIT_DROP is a positive int, this gets interpreted as the next header type (i.e., success). As that can only end badly, remove the check. Signed-off-by: Nick Bowler <nbowler@elliptictech.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
1e49570171
commit
4b90a603a1
|
@ -369,8 +369,6 @@ static int ah_input(struct xfrm_state *x, struct sk_buff *skb)
|
|||
if (err == -EINPROGRESS)
|
||||
goto out;
|
||||
|
||||
if (err == -EBUSY)
|
||||
err = NET_XMIT_DROP;
|
||||
goto out_free;
|
||||
}
|
||||
|
||||
|
|
|
@ -581,8 +581,6 @@ static int ah6_input(struct xfrm_state *x, struct sk_buff *skb)
|
|||
if (err == -EINPROGRESS)
|
||||
goto out;
|
||||
|
||||
if (err == -EBUSY)
|
||||
err = NET_XMIT_DROP;
|
||||
goto out_free;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue