From nobody Mon Apr 25 07:23:41 2022 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id CDC3D1A90FB2; Mon, 25 Apr 2022 07:23:41 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4KmxMK5QNPz3FSZ; Mon, 25 Apr 2022 07:23:41 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1650871421; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=l/V+Aai3QRbaztRUJAe9JiTtuhorS1+QTuWcKPlMSkI=; b=HzIdZKMB14i1PPqGSxiBRVJUB0qc93vmEc691Hi5dzwn+4RFYl3qoWN4DCaEXkXDbzxinW GWJcDBidmiYQjtx8jaTUU6sNE/7nJy7iI81EEP3gxm49i6s8bHx1SymyF84bQpXwGwkN2Y OptzE6TpI/Z19bFf93QjYcz6xLVLY2QKXk5HEn3WfpYctajtVSYj9jqutLwpSkPE3ZraDw iP1h0eEM3OCXZz7KV4s8m6/Lb5J9kw3zR76qBc00oyDb195too73syTaD9zxyprIX0SkMZ mmqB2pQm94qfXW9HQMEACkwXXKBNwwf/5M+6Nw9i+zifiu8iBzDPOkZG/IIDmA== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 99E3816ABF; Mon, 25 Apr 2022 07:23:41 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 23P7NfSi065677; Mon, 25 Apr 2022 07:23:41 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 23P7NfIf065676; Mon, 25 Apr 2022 07:23:41 GMT (envelope-from git) Date: Mon, 25 Apr 2022 07:23:41 GMT Message-Id: <202204250723.23P7NfIf065676@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Dmitry Chagin Subject: git: 91e7bdcdcf10 - main - Add timespecvalid_interval macro and use it. List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-main@freebsd.org X-BeenThere: dev-commits-src-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: dchagin X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 91e7bdcdcf10684098e213e311cdafcd7a0ac983 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1650871421; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=l/V+Aai3QRbaztRUJAe9JiTtuhorS1+QTuWcKPlMSkI=; b=MZSbVDiV7f7EHDOkrLAKwZQERd3dYej7hVZNcYsQ3YqcIh3GJ5oZNsBx9ReMf3YSOp9Xu4 cPDFdk+NvCWbW5dqyruE84d7bn43Xl3HdaqAljlnB3QG/k9B2DdXthKN0+tHvhxM52BmcI at5duPBWqoDE7mMfxMYv/Dsr4l6DOPeAUDDZjHr4IuGM6jjKvE4T6U5Mp7lak2/M8n0O0j OnUiBtIn8OvAoap4Rgkd6m7klvET2YsSeUIdug6YiY71DBVjk0O3rv5p2+1+Bu3nc4Kc10 kHpLEDyxR9MFVBytAPKz9vmc093rb/XnAApj7WB9p/4AaUvEMqBTjuKxfdPasA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1650871421; a=rsa-sha256; cv=none; b=gKez0kS3jpzo99YmGUPYlKpOJSarqSdErNbLsSG9cTbklYJEyy1bhrM4/FLIXRGMNTRC7k K9dGWzrubOehgZRrwUr4WOtkvgkUCad1dJbXcfySQ9CqIcFAlHLkj5fG3NkXv2RJAB0ykA PXrDaJkDC10CaJj4hLpkdxpzf3urjC/ti7dUAXbODKbPFTzof10Ht/Bc0qdZPHAcK1iCsd zmyxMVhvWMrJ0NsPHOQ3Z15qdJ0Uas1O5BU9A7JSIgV5c1SHnT3J2mkF9GyVlM9oiZ63hZ Qb3OpijG5Pb3qsjkGGur5uZGuS7sU40gjl/pz1KGpjePGOwDz5MEinhi0jHOlQ== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch main has been updated by dchagin: URL: https://cgit.FreeBSD.org/src/commit/?id=91e7bdcdcf10684098e213e311cdafcd7a0ac983 commit 91e7bdcdcf10684098e213e311cdafcd7a0ac983 Author: Dmitry Chagin AuthorDate: 2022-04-25 07:20:54 +0000 Commit: Dmitry Chagin CommitDate: 2022-04-25 07:20:54 +0000 Add timespecvalid_interval macro and use it. Reviewed by: jhb, imp (early rev) Differential revision: https://reviews.freebsd.org/D34848 MFC after: 2 weeks --- sys/compat/linux/linux_time.c | 4 ++-- sys/kern/kern_event.c | 3 +-- sys/kern/kern_time.c | 4 ++-- sys/kern/kern_umtx.c | 21 ++++++--------------- sys/kern/sys_generic.c | 4 +--- sys/sys/time.h | 2 ++ 6 files changed, 14 insertions(+), 24 deletions(-) diff --git a/sys/compat/linux/linux_time.c b/sys/compat/linux/linux_time.c index f43e4d3985aa..739480342ad9 100644 --- a/sys/compat/linux/linux_time.c +++ b/sys/compat/linux/linux_time.c @@ -142,7 +142,7 @@ int linux_to_native_timespec(struct timespec *ntp, struct l_timespec *ltp) { - if (ltp->tv_sec < 0 || ltp->tv_nsec < 0 || ltp->tv_nsec > 999999999) + if (!timespecvalid_interval(ltp)) return (EINVAL); ntp->tv_sec = ltp->tv_sec; ntp->tv_nsec = ltp->tv_nsec; @@ -165,7 +165,7 @@ int linux_to_native_timespec64(struct timespec *ntp, struct l_timespec64 *ltp64) { - if (ltp64->tv_sec < 0 || ltp64->tv_nsec < 0 || ltp64->tv_nsec > 999999999) + if (!timespecvalid_interval(ltp64)) return (EINVAL); ntp->tv_sec = ltp64->tv_sec; ntp->tv_nsec = ltp64->tv_nsec; diff --git a/sys/kern/kern_event.c b/sys/kern/kern_event.c index 0dc715f96670..a49762d9453c 100644 --- a/sys/kern/kern_event.c +++ b/sys/kern/kern_event.c @@ -1930,8 +1930,7 @@ kqueue_scan(struct kqueue *kq, int maxevents, struct kevent_copyops *k_ops, rsbt = 0; if (tsp != NULL) { - if (tsp->tv_sec < 0 || tsp->tv_nsec < 0 || - tsp->tv_nsec >= 1000000000) { + if (!timespecvalid_interval(tsp)) { error = EINVAL; goto done_nl; } diff --git a/sys/kern/kern_time.c b/sys/kern/kern_time.c index f052c4b6d698..de1b4ea7bc02 100644 --- a/sys/kern/kern_time.c +++ b/sys/kern/kern_time.c @@ -411,7 +411,7 @@ kern_clock_settime(struct thread *td, clockid_t clock_id, struct timespec *ats) return (error); if (clock_id != CLOCK_REALTIME) return (EINVAL); - if (ats->tv_nsec < 0 || ats->tv_nsec >= NS_PER_SEC || ats->tv_sec < 0) + if (!timespecvalid_interval(ats)) return (EINVAL); if (!allow_insane_settime && (ats->tv_sec > 8000ULL * 365 * 24 * 60 * 60 || @@ -1644,7 +1644,7 @@ static int itimespecfix(struct timespec *ts) { - if (ts->tv_sec < 0 || ts->tv_nsec < 0 || ts->tv_nsec >= NS_PER_SEC) + if (!timespecvalid_interval(ts)) return (EINVAL); if ((UINT64_MAX - ts->tv_nsec) / NS_PER_SEC < ts->tv_sec) return (EINVAL); diff --git a/sys/kern/kern_umtx.c b/sys/kern/kern_umtx.c index 7f74ba68b59c..2f666bdcdc7b 100644 --- a/sys/kern/kern_umtx.c +++ b/sys/kern/kern_umtx.c @@ -3772,9 +3772,7 @@ umtx_copyin_timeout(const void *uaddr, struct timespec *tsp) error = copyin(uaddr, tsp, sizeof(*tsp)); if (error == 0) { - if (tsp->tv_sec < 0 || - tsp->tv_nsec >= 1000000000 || - tsp->tv_nsec < 0) + if (!timespecvalid_interval(tsp)) error = EINVAL; } return (error); @@ -3793,8 +3791,7 @@ umtx_copyin_umtx_time(const void *uaddr, size_t size, struct _umtx_time *tp) error = copyin(uaddr, tp, sizeof(*tp)); if (error != 0) return (error); - if (tp->_timeout.tv_sec < 0 || - tp->_timeout.tv_nsec >= 1000000000 || tp->_timeout.tv_nsec < 0) + if (!timespecvalid_interval(&tp->_timeout)) return (EINVAL); return (0); } @@ -4640,9 +4637,7 @@ umtx_copyin_timeouti386(const void *uaddr, struct timespec *tsp) error = copyin(uaddr, &ts32, sizeof(ts32)); if (error == 0) { - if (ts32.tv_sec < 0 || - ts32.tv_nsec >= 1000000000 || - ts32.tv_nsec < 0) + if (!timespecvalid_interval(&ts32)) error = EINVAL; else { CP(ts32, *tsp, tv_sec); @@ -4666,8 +4661,7 @@ umtx_copyin_umtx_timei386(const void *uaddr, size_t size, struct _umtx_time *tp) error = copyin(uaddr, &t32, sizeof(t32)); if (error != 0) return (error); - if (t32._timeout.tv_sec < 0 || - t32._timeout.tv_nsec >= 1000000000 || t32._timeout.tv_nsec < 0) + if (!timespecvalid_interval(&t32._timeout)) return (EINVAL); TS_CP(t32, *tp, _timeout); CP(t32, *tp, _flags); @@ -4704,9 +4698,7 @@ umtx_copyin_timeoutx32(const void *uaddr, struct timespec *tsp) error = copyin(uaddr, &ts32, sizeof(ts32)); if (error == 0) { - if (ts32.tv_sec < 0 || - ts32.tv_nsec >= 1000000000 || - ts32.tv_nsec < 0) + if (!timespecvalid_interval(&ts32)) error = EINVAL; else { CP(ts32, *tsp, tv_sec); @@ -4730,8 +4722,7 @@ umtx_copyin_umtx_timex32(const void *uaddr, size_t size, struct _umtx_time *tp) error = copyin(uaddr, &t32, sizeof(t32)); if (error != 0) return (error); - if (t32._timeout.tv_sec < 0 || - t32._timeout.tv_nsec >= 1000000000 || t32._timeout.tv_nsec < 0) + if (!timespecvalid_interval(&t32._timeout)) return (EINVAL); TS_CP(t32, *tp, _timeout); CP(t32, *tp, _flags); diff --git a/sys/kern/sys_generic.c b/sys/kern/sys_generic.c index dd33d3dbe53e..b8cf1ff45610 100644 --- a/sys/kern/sys_generic.c +++ b/sys/kern/sys_generic.c @@ -1501,9 +1501,7 @@ kern_poll_kfds(struct thread *td, struct pollfd *kfds, u_int nfds, precision = 0; if (tsp != NULL) { - if (tsp->tv_sec < 0) - return (EINVAL); - if (tsp->tv_nsec < 0 || tsp->tv_nsec >= 1000000000) + if (!timespecvalid_interval(tsp)) return (EINVAL); if (tsp->tv_sec == 0 && tsp->tv_nsec == 0) sbt = 0; diff --git a/sys/sys/time.h b/sys/sys/time.h index f3a3bc99a0f2..7520478cf3ed 100644 --- a/sys/sys/time.h +++ b/sys/sys/time.h @@ -404,6 +404,8 @@ tvtosbt(struct timeval _tv) (vsp)->tv_nsec += 1000000000L; \ } \ } while (0) +#define timespecvalid_interval(tsp) ((tsp)->tv_sec >= 0 && \ + (tsp)->tv_nsec >= 0 && (tsp)->tv_nsec < 1000000000L) #ifdef _KERNEL