aboutsummaryrefslogtreecommitdiffstats
path: root/net/l2tp/l2tp_netlink.c
diff options
context:
space:
mode:
authorGuillaume Nault <g.nault@alphalink.fr>2017-03-31 13:02:27 +0200
committerDavid S. Miller <davem@davemloft.net>2017-04-01 20:16:41 -0700
commitdbdbc73b44782e22b3b4b6e8b51e7a3d245f3086 (patch)
tree2b4af9a50ad82ed9805c03cde4012d3859647633 /net/l2tp/l2tp_netlink.c
parentl2tp: ensure session can't get removed during pppol2tp_session_ioctl() (diff)
downloadlinux-dev-dbdbc73b44782e22b3b4b6e8b51e7a3d245f3086.tar.xz
linux-dev-dbdbc73b44782e22b3b4b6e8b51e7a3d245f3086.zip
l2tp: fix duplicate session creation
l2tp_session_create() relies on its caller for checking for duplicate sessions. This is racy since a session can be concurrently inserted after the caller's verification. Fix this by letting l2tp_session_create() verify sessions uniqueness upon insertion. Callers need to be adapted to check for l2tp_session_create()'s return code instead of calling l2tp_session_find(). pppol2tp_connect() is a bit special because it has to work on existing sessions (if they're not connected) or to create a new session if none is found. When acting on a preexisting session, a reference must be held or it could go away on us. So we have to use l2tp_session_get() instead of l2tp_session_find() and drop the reference before exiting. Fixes: d9e31d17ceba ("l2tp: Add L2TP ethernet pseudowire support") Fixes: fd558d186df2 ("l2tp: Split pppol2tp patch into separate l2tp and ppp parts") Signed-off-by: Guillaume Nault <g.nault@alphalink.fr> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/l2tp/l2tp_netlink.c')
0 files changed, 0 insertions, 0 deletions