lkml.org 
[lkml]   [2020]   [Sep]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH] time: Avoid undefined behaviour in timespec64_to_ns
    On Tue, Sep 1, 2020 at 11:32 AM Zeng Tao <prime.zeng@hisilicon.com> wrote:
    >
    > Since commit bd40a175769d ("y2038: itimer: change implementation to timespec64")
    > we have break the time clamping which handles the potential overflow.

    Indeed, good catch!

    And I broke it despite the comment telling me about the problem.

    > In this patch, we fix it in the timespec64_to_ns because there is
    > possiblity to cause the same undefined behaviour on overflow whenever
    > the function is called.

    I checked the most important callers of this function, and I agree
    that truncating at the maximum would be sensible in most cases
    here.

    In timekeeping_init(), there is already a check for
    timespec64_valid_settod() that limits it even further, but that
    wouldn't make sense for most callers.

    > Fixes: bd40a175769d ("y2038: itimer: change implementation to timespec64")

    This one caused the regression, but if we add the check here, it
    may be best to also add it in prior kernels that may have the same
    bug in other callers of the same function. Maybe backport all the
    way to stable kernels that first added timespec64?

    Cc <stable@vger.kernel.org> # v3.17+

    > Signed-off-by: Zeng Tao <prime.zeng@hisilicon.com>

    Reviewed-by: Arnd Bergmann <arnd@arndb.de>

    \
     
     \ /
      Last update: 2020-09-01 14:48    [W:3.198 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site