From nobody Thu Mar 19 00:47:26 2026 X-Original-To: dev-commits-src-all@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 4fbn900WXjz6VN5Y for ; Thu, 19 Mar 2026 00:47:32 +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 "R12" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4fbn8z6nTkz3tfb for ; Thu, 19 Mar 2026 00:47:31 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1773881252; 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=pQRK3nVAJWV8+8JRNRVm2OSGAJIFfP3pOm594zaqzD0=; b=mHWjaY6Qke0F1flYv18iGoPeylE3LLeUxVSfFfxYGu8QRHT01W9Plmj0HhhOHXA34qrvUj Mjub3/WuxWifVUc1fDAScTXOBH5SlGlC+dJw73i1/VMrKCXMWJ1u2aUBQHas/wA84wbupV ctQSwjMdARWDQMhV8F4gPzocI4VacOLLxTlzNUz8p1qApyN5+FyfgYQD8PeKhAzEeos6az yLllJK7KOkY9/fgCX5tFUxdpi/qm3/12E6BAt+HGQGMeW41hlJO05ULQRXr5uOSJYXk+N4 mDRMD7vY0FRYv7xcxe/AeNkktwCbLbiZ83OAvuGWjXOkW1ACiMRhp0CAJV++VA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1773881252; a=rsa-sha256; cv=none; b=ujG0IHhj7D+iTZCS8i6D/uAYYrSWopF1aMNx+nOMKE7BXDdeJxD2wwYZeK59zS22pyQYK+ o2jtSD7r03fNDB3ymSiuMSE7bEL+Rgc+40EY9lQQ7M0pN1QDi0q2aYGt/NJpT1cMWqpO5E utnSY68EYxp3gDp0vZEGIFECCFhNIx4SNxUc3TlsFIMm2xQspmxbvWGze+Dov7bB0ldZpe CLD4s48GRIS6hP4Gps7ptaoAPvneVqZ//+IrXH6aIvrxwKGNHJqmaVZkLFfEDeQSwGRMSf G6JVUJWHgKbLRw4MkzqSO2/YmMVGWTOpm9Z5ZZJwPCHyRDgMHxKik6QFL/TRuA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1773881252; 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=pQRK3nVAJWV8+8JRNRVm2OSGAJIFfP3pOm594zaqzD0=; b=oL2lVK8Ffy8ZSFX+VYyfTYN+Dfq6p8T3ElgZzNTMJO/YDbqi85ynHEfMwweuKVSjQkHFt2 K2E2l6CSf0UIQ1I1SEyVG2zNxTym8Q1Ef8j3pXSqedX5uQvBAgsucPeXUip5zUZ/xPlXJi VLnyswMXeWuJScbB8J2BN5PuRWl4qK0GsMVloCYLZowHYleHe7J2Z1Ao/Bci2v/j6kDpdX Qrry4jFO5yJJUHit++Me8ipXNhCxa+L/JG9L9N5rb3iYtcoVv1u2mJSoId7gnTxHFLz15D KZwNJLndf4jsGLvsGOLM7yZK9zJqbr5eXQqTMD7kyD3R2LX1eG9/XST0RgK+AQ== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4fbn8z6JrRzY8D for ; Thu, 19 Mar 2026 00:47:31 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 3bd1f by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Thu, 19 Mar 2026 00:47:26 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Kirk McKusick Subject: git: 51446d33c6fb - main - Delete error-check code that can never happen. List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: mckusick X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 51446d33c6fbc27ce21f54ebb4c27caace48c3be Auto-Submitted: auto-generated Date: Thu, 19 Mar 2026 00:47:26 +0000 Message-Id: <69bb479e.3bd1f.710a9fc1@gitrepo.freebsd.org> The branch main has been updated by mckusick: URL: https://cgit.FreeBSD.org/src/commit/?id=51446d33c6fbc27ce21f54ebb4c27caace48c3be commit 51446d33c6fbc27ce21f54ebb4c27caace48c3be Author: Kirk McKusick AuthorDate: 2026-03-19 00:45:23 +0000 Commit: Kirk McKusick CommitDate: 2026-03-19 00:45:23 +0000 Delete error-check code that can never happen. Near the top of kern_mmap() that implements the mmap(2) system call, it sets prot = PROT_EXTRACT(prot); with So prot can only be the three PROT_ flags. The following test of the user's mmap(2) parameters (near line 275 in vm/vm_mmap.c): if (prot != PROT_NONE && (prot & ~(PROT_READ | PROT_WRITE | PROT_EXEC)) != 0) { return (EXTERROR(EINVAL, "invalid prot %#jx", prot)); } can never fail. This commit deletes it. No functional change intended. Reviewed by: kib MFC-after: 1 week Sponsored by: Netflix --- sys/vm/vm_mmap.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/sys/vm/vm_mmap.c b/sys/vm/vm_mmap.c index 234586893b59..cb5906440f56 100644 --- a/sys/vm/vm_mmap.c +++ b/sys/vm/vm_mmap.c @@ -271,10 +271,6 @@ kern_mmap(struct thread *td, const struct mmap_req *mrp) return (EXTERROR(EINVAL, "both SHARED and PRIVATE set (flags %#jx)", flags)); } - if (prot != PROT_NONE && - (prot & ~(PROT_READ | PROT_WRITE | PROT_EXEC)) != 0) { - return (EXTERROR(EINVAL, "invalid prot %#jx", prot)); - } if ((flags & MAP_GUARD) != 0 && (prot != PROT_NONE || fd != -1 || pos != 0 || (flags & ~(MAP_FIXED | MAP_GUARD | MAP_EXCL | MAP_32BIT | MAP_ALIGNMENT_MASK)) != 0)) {