]> www.infradead.org Git - users/willy/xarray.git/commit
net: esp: cleanup esp_output_tail_tcp() in case of unsupported ESPINTCP
authorHagar Hemdan <hagarhem@amazon.com>
Sat, 18 May 2024 13:04:39 +0000 (13:04 +0000)
committerSteffen Klassert <steffen.klassert@secunet.com>
Thu, 23 May 2024 06:46:03 +0000 (08:46 +0200)
commit96f887a612e4cda89efc3f54bc10c1997e3ab0e9
treee35c53690edd25aaa0a92ab852ca43cc2b8eddb1
parent9c91c7fadb1771dcc2815c5271d14566366d05c5
net: esp: cleanup esp_output_tail_tcp() in case of unsupported ESPINTCP

xmit() functions should consume skb or return error codes in error
paths.
When the configuration "CONFIG_INET_ESPINTCP" is not set, the
implementation of the function "esp_output_tail_tcp" violates this rule.
The function frees the skb and returns the error code.
This change removes the kfree_skb from both functions, for both
esp4 and esp6.
WARN_ON is added because esp_output_tail_tcp() should never be called if
CONFIG_INET_ESPINTCP is not set.

This bug was discovered and resolved using Coverity Static Analysis
Security Testing (SAST) by Synopsys, Inc.

Fixes: e27cca96cd68 ("xfrm: add espintcp (RFC 8229)")
Signed-off-by: Hagar Hemdan <hagarhem@amazon.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
net/ipv4/esp4.c
net/ipv6/esp6.c