From 57d5fd7ef5b4d121a5e5c43d84d4b50fc93a32dc Mon Sep 17 00:00:00 2001
From: Paul Hummer <paul@eventuallyanyway.com>
Date: Sun, 7 Sep 2025 13:02:31 -0600
Subject: [PATCH] fix: adjust max pid on macos

`PID_MAX` on macOS is `99999`, but we can never be equal to
`PID_MAX`, so instead of `<=`, we should use `<` to avoid the
potential off-by-one validation error.
---
 src/lib.rs | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/lib.rs b/src/lib.rs
index 1754f19..b5e6641 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -328,8 +328,9 @@ fn validate_pid(pid: i32) -> bool {
 
     #[cfg(target_os = "macos")]
     {
-        // macOS uses 32-bit PIDs but typically keeps them much lower
-        pid <= 99999
+        // macOS defines PID_MAX as 99999, but process IDs are not assigned
+        // to PID_MAX, so max pid is 99998.
+        pid < 99999
     }
 
     #[cfg(target_os = "windows")]
@@ -1626,8 +1627,8 @@ mod tests {
 
         #[cfg(target_os = "macos")]
         {
-            assert!(validate_pid(99999)); // Should be valid on macOS
-            assert!(!validate_pid(100000)); // Should be invalid
+            assert!(validate_pid(99998)); // Should be valid on macOS
+            assert!(!validate_pid(99999)); // Should be invalid
         }
     }