From ebf7a01594eeb8c2897d1b310069df35da112413 Mon Sep 17 00:00:00 2001 From: Konstantin Belousov Date: Mon, 12 Sep 2022 23:24:55 +0300 Subject: [PATCH] libthr: use nitems() for mib length Suggested by: imp Reviewed by: brooks, imp (previous version) Discussed with: markj Sponsored by: The FreeBSD Foundation MFC after: 2 weeks Differential revision: https://reviews.freebsd.org/D36540 --- lib/libc/gen/elf_utils.c | 4 ++-- lib/libthr/thread/thr_init.c | 4 ++-- lib/libthr/thread/thr_stack.c | 5 ++--- 3 files changed, 6 insertions(+), 7 deletions(-) diff --git a/lib/libc/gen/elf_utils.c b/lib/libc/gen/elf_utils.c index fbf3bf17082..ea5cc25f860 100644 --- a/lib/libc/gen/elf_utils.c +++ b/lib/libc/gen/elf_utils.c @@ -28,7 +28,7 @@ * $FreeBSD$ */ -#include +#include #include #include #include @@ -83,7 +83,7 @@ __libc_map_stacks_exec(void) mib[0] = CTL_KERN; mib[1] = KERN_USRSTACK; len = sizeof(usrstack); - if (sysctl(mib, sizeof(mib) / sizeof(mib[0]), &usrstack, &len, NULL, 0) + if (sysctl(mib, nitems(mib), &usrstack, &len, NULL, 0) == -1) return; if (getrlimit(RLIMIT_STACK, &rlim) == -1) diff --git a/lib/libthr/thread/thr_init.c b/lib/libthr/thread/thr_init.c index 82bde10a153..0ab051e5799 100644 --- a/lib/libthr/thread/thr_init.c +++ b/lib/libthr/thread/thr_init.c @@ -37,7 +37,7 @@ __FBSDID("$FreeBSD$"); #include "namespace.h" -#include +#include #include #include #include @@ -466,7 +466,7 @@ init_private(void) mib[0] = CTL_KERN; mib[1] = KERN_USRSTACK; len = sizeof (_usrstack); - if (sysctl(mib, 2, &_usrstack, &len, NULL, 0) == -1) + if (sysctl(mib, nitems(mib), &_usrstack, &len, NULL, 0) == -1) PANIC("Cannot get kern.usrstack from sysctl"); env_bigstack = getenv("LIBPTHREAD_BIGSTACK_MAIN"); env_splitstack = getenv("LIBPTHREAD_SPLITSTACK_MAIN"); diff --git a/lib/libthr/thread/thr_stack.c b/lib/libthr/thread/thr_stack.c index b08bafdd941..af396fe2ba9 100644 --- a/lib/libthr/thread/thr_stack.c +++ b/lib/libthr/thread/thr_stack.c @@ -30,7 +30,7 @@ #include __FBSDID("$FreeBSD$"); -#include +#include #include #include #include @@ -155,8 +155,7 @@ singlethread_map_stacks_exec(void) mib[0] = CTL_KERN; mib[1] = KERN_USRSTACK; len = sizeof(usrstack); - if (sysctl(mib, sizeof(mib) / sizeof(mib[0]), &usrstack, &len, NULL, 0) - == -1) + if (sysctl(mib, nitems(mib), &usrstack, &len, NULL, 0) == -1) return; if (getrlimit(RLIMIT_STACK, &rlim) == -1) return;