diff options
author | Jakub Kicinski <kuba@kernel.org> | 2021-06-14 15:24:05 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2021-06-15 10:59:46 -0700 |
commit | 475b92f932168a78da8109acd10bfb7578b8f2bb (patch) | |
tree | 1659ca2f045d8cc6295dae645c844e3228326494 /include/net/net_namespace.h | |
parent | 2214fb53006e6cfa6371b706070cb99794c68c3b (diff) | |
download | linux-stable-475b92f932168a78da8109acd10bfb7578b8f2bb.tar.gz linux-stable-475b92f932168a78da8109acd10bfb7578b8f2bb.tar.bz2 linux-stable-475b92f932168a78da8109acd10bfb7578b8f2bb.zip |
ptp: improve max_adj check against unreasonable values
Scaled PPM conversion to PPB may (on 64bit systems) result
in a value larger than s32 can hold (freq/scaled_ppm is a long).
This means the kernel will not correctly reject unreasonably
high ->freq values (e.g. > 4294967295ppb, 281474976645 scaled PPM).
The conversion is equivalent to a division by ~66 (65.536),
so the value of ppb is always smaller than ppm, but not small
enough to assume narrowing the type from long -> s32 is okay.
Note that reasonable user space (e.g. ptp4l) will not use such
high values, anyway, 4289046510ppb ~= 4.3x, so the fix is
somewhat pedantic.
Fixes: d39a743511cd ("ptp: validate the requested frequency adjustment.")
Fixes: d94ba80ebbea ("ptp: Added a brand new class driver for ptp clocks.")
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Acked-by: Richard Cochran <richardcochran@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/net_namespace.h')
0 files changed, 0 insertions, 0 deletions