diff options
author | Zhang Changzhong <zhangchangzhong@huawei.com> | 2020-11-17 10:45:05 +0800 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2020-11-18 10:53:16 -0800 |
commit | a5ebcbdf34b65fcc07f38eaf2d60563b42619a59 (patch) | |
tree | 9f744b49e91c52559864c87506e9bf9fa61a42da | |
parent | df8d85d8c69d6837817e54dcb73c84a8b5a13877 (diff) | |
download | lwn-a5ebcbdf34b65fcc07f38eaf2d60563b42619a59.tar.gz lwn-a5ebcbdf34b65fcc07f38eaf2d60563b42619a59.zip |
ah6: fix error return code in ah6_input()
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhang Changzhong <zhangchangzhong@huawei.com>
Link: https://lore.kernel.org/r/1605581105-35295-1-git-send-email-zhangchangzhong@huawei.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
-rw-r--r-- | net/ipv6/ah6.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/ipv6/ah6.c b/net/ipv6/ah6.c index d88d97617f7e..440080da805b 100644 --- a/net/ipv6/ah6.c +++ b/net/ipv6/ah6.c @@ -588,7 +588,8 @@ static int ah6_input(struct xfrm_state *x, struct sk_buff *skb) memcpy(auth_data, ah->auth_data, ahp->icv_trunc_len); memset(ah->auth_data, 0, ahp->icv_trunc_len); - if (ipv6_clear_mutable_options(ip6h, hdr_len, XFRM_POLICY_IN)) + err = ipv6_clear_mutable_options(ip6h, hdr_len, XFRM_POLICY_IN); + if (err) goto out_free; ip6h->priority = 0; |