diff options
author | Edward Cree <ecree@solarflare.com> | 2020-09-01 18:52:32 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2020-09-03 14:59:51 -0700 |
commit | 2adc6edcaec09eea9275915e9632ff916fcd0785 (patch) | |
tree | 3b2e454129b95babe7b6a83fee1c1edbd40bef50 /include | |
parent | 30ae801746ea0e099246318d373691a9e3238d40 (diff) |
ethtool: fix error handling in ethtool_phys_id
If ops->set_phys_id() returned an error, previously we would only break
out of the inner loop, which neither stopped the outer loop nor returned
the error to the user (since 'rc' would be overwritten on the next pass
through the loop).
Thus, rewrite it to use a single loop, so that the break does the right
thing. Use u64 for 'count' and 'i' to prevent overflow in case of
(unreasonably) large values of id.data and n.
Signed-off-by: Edward Cree <ecree@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions