diff options
author | Guillaume Nault <g.nault@alphalink.fr> | 2017-04-06 12:31:20 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-04-08 11:29:04 -0400 |
commit | 364700cf8fd54f54ad08313464105a414e3bccb7 (patch) | |
tree | 6effdb79001ec4069af6f3301fda22d465b9280e | |
parent | cc663f4d4c97b7297fb45135ab23cfd508b35a77 (diff) |
l2tp: don't mask errors in pppol2tp_setsockopt()
pppol2tp_setsockopt() unconditionally overwrites the error value
returned by pppol2tp_tunnel_setsockopt() or
pppol2tp_session_setsockopt(), thus hiding errors from userspace.
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>
-rw-r--r-- | net/l2tp/l2tp_ppp.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/l2tp/l2tp_ppp.c b/net/l2tp/l2tp_ppp.c index 861b255a2d51..973a9185b276 100644 --- a/net/l2tp/l2tp_ppp.c +++ b/net/l2tp/l2tp_ppp.c | |||
@@ -1383,8 +1383,6 @@ static int pppol2tp_setsockopt(struct socket *sock, int level, int optname, | |||
1383 | } else | 1383 | } else |
1384 | err = pppol2tp_session_setsockopt(sk, session, optname, val); | 1384 | err = pppol2tp_session_setsockopt(sk, session, optname, val); |
1385 | 1385 | ||
1386 | err = 0; | ||
1387 | |||
1388 | end_put_sess: | 1386 | end_put_sess: |
1389 | sock_put(sk); | 1387 | sock_put(sk); |
1390 | end: | 1388 | end: |