]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
net: Fix unwanted sign extension in netdev_stats_to_stats64()
authorFelix Riemann <felix.riemann@sma.de>
Fri, 10 Feb 2023 12:36:44 +0000 (13:36 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 22 Feb 2023 11:55:56 +0000 (12:55 +0100)
commite2bf52ff159db37b5b60d34069464c00a1f54fd6
tree958a17dfdbf52402b48389f0dc5525592b713c47
parent1933be146c96b5a707ce297eddfc3c49a9eec775
net: Fix unwanted sign extension in netdev_stats_to_stats64()

commit 9b55d3f0a69af649c62cbc2633e6d695bb3cc583 upstream.

When converting net_device_stats to rtnl_link_stats64 sign extension
is triggered on ILP32 machines as 6c1c509778 changed the previous
"ulong -> u64" conversion to "long -> u64" by accessing the
net_device_stats fields through a (signed) atomic_long_t.

This causes for example the received bytes counter to jump to 16EiB after
having received 2^31 bytes. Casting the atomic value to "unsigned long"
beforehand converting it into u64 avoids this.

Fixes: 6c1c5097781f ("net: add atomic_long_t to net_device_stats fields")
Signed-off-by: Felix Riemann <felix.riemann@sma.de>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
net/core/dev.c