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 /fs/compat.c | |
parent | 8e6ce7ebeb34f0992f56de078c3744fb383657fa (diff) | |
download | op-kernel-dev-1ba5bf993c6a3142e18e68ea6452b347f9cb5635.zip op-kernel-dev-1ba5bf993c6a3142e18e68ea6452b347f9cb5635.tar.gz |
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 'fs/compat.c')
0 files changed, 0 insertions, 0 deletions