net: macsec: fix potential resource leak in macsec_add_rxsa() and macsec_add_txsa()
authorJianglei Nie <niejianglei2021@163.com>
Fri, 22 Jul 2022 09:29:02 +0000 (17:29 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 25 Jul 2022 10:51:05 +0000 (11:51 +0100)
commitc7b205fbbf3cffa374721bb7623f7aa8c46074f1
tree35cd754fb9f0af9f0bc82cb8d1d19b35495152be
parent20a854616d384d3210d96fee248a3ea327bab810
net: macsec: fix potential resource leak in macsec_add_rxsa() and macsec_add_txsa()

init_rx_sa() allocates relevant resource for rx_sa->stats and rx_sa->
key.tfm with alloc_percpu() and macsec_alloc_tfm(). When some error
occurs after init_rx_sa() is called in macsec_add_rxsa(), the function
released rx_sa with kfree() without releasing rx_sa->stats and rx_sa->
key.tfm, which will lead to a resource leak.

We should call macsec_rxsa_put() instead of kfree() to decrease the ref
count of rx_sa and release the relevant resource if the refcount is 0.
The same bug exists in macsec_add_txsa() for tx_sa as well. This patch
fixes the above two bugs.

Fixes: 3cf3227a21d1 ("net: macsec: hardware offloading infrastructure")
Signed-off-by: Jianglei Nie <niejianglei2021@163.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/macsec.c