[v5,0/2] posix-clock: Fix missing timespec64 check for PTP clock

Message

Jinjie Ruan Oct. 8, 2024, 9:10 a.m. UTC

Check timespec64 in pc_clock_settime() for PTP clock as
the man manual of clock_settime() said.

Changes in v5:
- Use timespec64_valid_strict() instead of timespec64_valid()
  as Thomas suggested.
- Add fix tag.
- Update the commit message.

Changes in v4:
- Check it in pc_clock_settime() for PTP clock.
- Update the commit message.

Changes in v3:
- Check it before call clock_set().
- Update the commit message.

Jinjie Ruan (2):
  posix-clock: Fix missing timespec64 check in pc_clock_settime()
  net: lan743x: Remove duplicate check

 drivers/net/ethernet/microchip/lan743x_ptp.c | 35 ++++++++------------
 kernel/time/posix-clock.c                    |  3 ++
 2 files changed, 17 insertions(+), 21 deletions(-)

Comments

Richard Cochran Oct. 8, 2024, 2:58 p.m. UTC | #1

On Tue, Oct 08, 2024 at 05:10:59PM +0800, Jinjie Ruan wrote:
> Check timespec64 in pc_clock_settime() for PTP clock as
> the man manual of clock_settime() said.
> 
> Changes in v5:
> - Use timespec64_valid_strict() instead of timespec64_valid()
>   as Thomas suggested.
> - Add fix tag.
> - Update the commit message.

For the series:

Acked-by: Richard Cochran <[e-mail]>

Richard Cochran Oct. 8, 2024, 3:18 p.m. UTC | #2

On Tue, Oct 08, 2024 at 05:10:59PM +0800, Jinjie Ruan wrote:
> Check timespec64 in pc_clock_settime() for PTP clock as
> the man manual of clock_settime() said.

Adding John Stultz's correct email address onto CC.

Thanks,
Richard