Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 29 May 2026 16:01:31 +0000
From:      Olivier Certner <olce@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: d554b89f4096 - main - MAC/do: check_proc(): Remove a superfluous 'if'
Message-ID:  <6a19b85b.35f90.2a7d1a60@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch main has been updated by olce:

URL: https://cgit.FreeBSD.org/src/commit/?id=d554b89f4096589d00abef584552eff0481418db

commit d554b89f4096589d00abef584552eff0481418db
Author:     Olivier Certner <olce@FreeBSD.org>
AuthorDate: 2026-04-28 13:44:53 +0000
Commit:     Olivier Certner <olce@FreeBSD.org>
CommitDate: 2026-05-29 15:21:10 +0000

    MAC/do: check_proc(): Remove a superfluous 'if'
    
    No functional change (intended).
    
    Reviewed by:    bapt
    MFC after:      1 month
    Sponsored by:   The FreeBSD Foundation
    Pull Request:   https://ron-dev.freebsd.org/FreeBSD/src/pulls/38
---
 sys/security/mac_do/mac_do.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/sys/security/mac_do/mac_do.c b/sys/security/mac_do/mac_do.c
index 9da7aaa257d5..5aacce1b4d09 100644
--- a/sys/security/mac_do/mac_do.c
+++ b/sys/security/mac_do/mac_do.c
@@ -2303,12 +2303,11 @@ check_proc(void)
 	conf = find_conf(td_pr, &pr);
 	exec_paths = &conf->exec_paths;
 
-	if (exec_paths->exec_path_count > 0)
-		for (int i = 0; i < exec_paths->exec_path_count; i++)
-			if (strcmp(exec_paths->exec_paths[i], path) == 0) {
-				error = 0;
-				break;
-			}
+	for (int i = 0; i < exec_paths->exec_path_count; i++)
+		if (strcmp(exec_paths->exec_paths[i], path) == 0) {
+			error = 0;
+			break;
+		}
 
 	prison_unlock(pr);
 


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6a19b85b.35f90.2a7d1a60>