]> git.baikalelectronics.ru Git - kernel.git/commit
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)
commit2d58f44134c870ce599ea5bf40e17d7054a783c6
tree35cd754fb9f0af9f0bc82cb8d1d19b35495152be
parent82820ce22bcb3899ebefb29db8f744a8d0f27e8f
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: 37ce3272dda3 ("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