diff options
author | Nikolay Aleksandrov <nikolay@cumulusnetworks.com> | 2015-11-26 09:23:49 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-11-30 15:26:22 -0500 |
commit | 42e6b89ce4e8a4f02a1e906694d81acf60db6f4d (patch) | |
tree | 47e399d185e96f60884ba4efcae6700d256b15f2 /net/ipv4/ipmr.c | |
parent | 1973a4ea6ceaa47671227c3077f90508ea30897b (diff) |
net: ipmr: fix setsockopt error return
We can have both errors and we'll return the second one, fix it to
return an error at a time as it's normal. I've overlooked this in my
previous set.
Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/ipmr.c')
-rw-r--r-- | net/ipv4/ipmr.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/net/ipv4/ipmr.c b/net/ipv4/ipmr.c index 322fdc6ac75b..6c24a16299c7 100644 --- a/net/ipv4/ipmr.c +++ b/net/ipv4/ipmr.c | |||
@@ -1284,12 +1284,14 @@ int ip_mroute_setsockopt(struct sock *sk, int optname, char __user *optval, | |||
1284 | 1284 | ||
1285 | switch (optname) { | 1285 | switch (optname) { |
1286 | case MRT_INIT: | 1286 | case MRT_INIT: |
1287 | if (optlen != sizeof(int)) | 1287 | if (optlen != sizeof(int)) { |
1288 | ret = -EINVAL; | 1288 | ret = -EINVAL; |
1289 | if (rtnl_dereference(mrt->mroute_sk)) | 1289 | break; |
1290 | } | ||
1291 | if (rtnl_dereference(mrt->mroute_sk)) { | ||
1290 | ret = -EADDRINUSE; | 1292 | ret = -EADDRINUSE; |
1291 | if (ret) | ||
1292 | break; | 1293 | break; |
1294 | } | ||
1293 | 1295 | ||
1294 | ret = ip_ra_control(sk, 1, mrtsock_destruct); | 1296 | ret = ip_ra_control(sk, 1, mrtsock_destruct); |
1295 | if (ret == 0) { | 1297 | if (ret == 0) { |