aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2017-04-08 11:29:05 -0400
committerDavid S. Miller <davem@davemloft.net>2017-04-08 11:29:05 -0400
commit3aecfbb19ca746a62575de90957c14e03b9c5d23 (patch)
tree198566224a6c5b6a6408faafe8dfbfbcd960cbe9 /net
parentcc663f4d4c97b7297fb45135ab23cfd508b35a77 (diff)
parent321a52a39189d5e4af542f7dcdc07bba4545cf5d (diff)
Merge branch 'l2tp-sockopt-errors'
Guillaume Nault says: ==================== l2tp: fix error handling of PPPoL2TP socket options Fix pppol2tp_[gs]etsockopt() so that they don't ignore errors returned by their helper functions. ==================== Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/l2tp/l2tp_ppp.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/net/l2tp/l2tp_ppp.c b/net/l2tp/l2tp_ppp.c
index 861b255a2d51..32ea0f3d868c 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
1388end_put_sess: 1386end_put_sess:
1389 sock_put(sk); 1387 sock_put(sk);
1390end: 1388end:
@@ -1507,8 +1505,13 @@ static int pppol2tp_getsockopt(struct socket *sock, int level, int optname,
1507 1505
1508 err = pppol2tp_tunnel_getsockopt(sk, tunnel, optname, &val); 1506 err = pppol2tp_tunnel_getsockopt(sk, tunnel, optname, &val);
1509 sock_put(ps->tunnel_sock); 1507 sock_put(ps->tunnel_sock);
1510 } else 1508 if (err)
1509 goto end_put_sess;
1510 } else {
1511 err = pppol2tp_session_getsockopt(sk, session, optname, &val); 1511 err = pppol2tp_session_getsockopt(sk, session, optname, &val);
1512 if (err)
1513 goto end_put_sess;
1514 }
1512 1515
1513 err = -EFAULT; 1516 err = -EFAULT;
1514 if (put_user(len, optlen)) 1517 if (put_user(len, optlen))