diff options
author | Guillaume Nault <g.nault@alphalink.fr> | 2017-03-31 13:02:27 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-04-01 20:16:41 -0700 |
commit | dbdbc73b44782e22b3b4b6e8b51e7a3d245f3086 (patch) | |
tree | 2b4af9a50ad82ed9805c03cde4012d3859647633 /fs/nfsd | |
parent | 57377d63547861919ee634b845c7caa38de4a452 (diff) |
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 'fs/nfsd')
0 files changed, 0 insertions, 0 deletions