summaryrefslogtreecommitdiffstats
path: root/net/ethtool/linkstate.c
diff options
context:
space:
mode:
authorMichal Kubecek <mkubecek@suse.cz>2020-06-25 00:09:08 +0200
committerDavid S. Miller <davem@davemloft.net>2020-06-26 01:17:16 +0200
commit1ae71d997a672739b22572cbe72d61a0eed8c42c (patch)
tree35fbaa75e149c650d646ea595de6a0a76d733bb6 /net/ethtool/linkstate.c
parentMerge branch 'napi_gro_receive-caller-return-value-cleanups' (diff)
downloadlinux-1ae71d997a672739b22572cbe72d61a0eed8c42c.tar.xz
linux-1ae71d997a672739b22572cbe72d61a0eed8c42c.zip
ethtool: fix error handling in linkstate_prepare_data()
When getting SQI or maximum SQI value fails in linkstate_prepare_data(), we must not return without calling ethnl_ops_complete(dev) as that could result in imbalance between ethtool_ops ->begin() and ->complete() calls. Fixes: 806602191592 ("ethtool: provide UAPI for PHY Signal Quality Index (SQI)") Signed-off-by: Michal Kubecek <mkubecek@suse.cz> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
-rw-r--r--net/ethtool/linkstate.c11
1 files changed, 5 insertions, 6 deletions
diff --git a/net/ethtool/linkstate.c b/net/ethtool/linkstate.c
index 7f47ba89054e..afe5ac8a0f00 100644
--- a/net/ethtool/linkstate.c
+++ b/net/ethtool/linkstate.c
@@ -78,19 +78,18 @@ static int linkstate_prepare_data(const struct ethnl_req_info *req_base,
ret = linkstate_get_sqi(dev);
if (ret < 0 && ret != -EOPNOTSUPP)
- return ret;
-
+ goto out;
data->sqi = ret;
ret = linkstate_get_sqi_max(dev);
if (ret < 0 && ret != -EOPNOTSUPP)
- return ret;
-
+ goto out;
data->sqi_max = ret;
+ ret = 0;
+out:
ethnl_ops_complete(dev);
-
- return 0;
+ return ret;
}
static int linkstate_reply_size(const struct ethnl_req_info *req_base,