XFRM: Fix memory leak in xfrm_state_update
authorTushar Gohad <tgohad@mvista.com>
Thu, 7 Jul 2011 15:38:52 +0000 (15:38 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 8 Jul 2011 15:58:42 +0000 (08:58 -0700)
commit8fcbc63701b01e913e6a13937f765fabf9c969c5
treeeaed81a25f362299b29de4731ca02289043b8cc2
parentf8d9605243280f1870dd2c6c37a735b925c15f3c
XFRM: Fix memory leak in xfrm_state_update

Upon "ip xfrm state update ..", xfrm_add_sa() takes an extra reference on
the user-supplied SA and forgets to drop the reference when
xfrm_state_update() returns 0.  This leads to a memory leak as the
parameter SA is never freed.  This change attempts to fix the leak by
calling __xfrm_state_put() when xfrm_state_update() updates a valid SA
(err = 0).  The parameter SA is added to the gc list when the final
reference is dropped by xfrm_add_sa() upon completion.

Signed-off-by: Tushar Gohad <tgohad@mvista.com>
Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/xfrm/xfrm_state.c