diff options
| author | Vegard Nossum <vegard.nossum@oracle.com> | 2016-07-05 10:18:08 +0200 |
|---|---|---|
| committer | Steffen Klassert <steffen.klassert@secunet.com> | 2016-07-18 09:37:02 +0200 |
| commit | 1ba5bf993c6a3142e18e68ea6452b347f9cb5635 (patch) | |
| tree | 50acfc1c0fc36a81964f600f505ec93be0e34f10 /net/rxrpc | |
| parent | net: cavium: liquidio: Avoid dma_unmap_single on uninitialized ndata (diff) | |
| download | linux-1ba5bf993c6a3142e18e68ea6452b347f9cb5635.tar.xz linux-1ba5bf993c6a3142e18e68ea6452b347f9cb5635.zip | |
xfrm: fix crash in XFRM_MSG_GETSA netlink handler
If we hit any of the error conditions inside xfrm_dump_sa(), then
xfrm_state_walk_init() never gets called. However, we still call
xfrm_state_walk_done() from xfrm_dump_sa_done(), which will crash
because the state walk was never initialized properly.
We can fix this by setting cb->args[0] only after we've processed the
first element and checking this before calling xfrm_state_walk_done().
Fixes: d3623099d3 ("ipsec: add support of limited SA dump")
Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Cc: Steffen Klassert <steffen.klassert@secunet.com>
Signed-off-by: Vegard Nossum <vegard.nossum@oracle.com>
Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Diffstat (limited to 'net/rxrpc')
0 files changed, 0 insertions, 0 deletions