diff options
| author | Tushar Gohad <[email protected]> | 2011-07-07 15:38:52 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2011-07-08 08:58:42 -0700 |
| commit | 8fcbc63701b01e913e6a13937f765fabf9c969c5 (patch) | |
| tree | eaed81a25f362299b29de4731ca02289043b8cc2 /net/lapb/lapb_timer.c | |
| parent | f8d9605243280f1870dd2c6c37a735b925c15f3c (diff) | |
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 <[email protected]>
Acked-by: Herbert Xu <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'net/lapb/lapb_timer.c')
0 files changed, 0 insertions, 0 deletions