8276651: java/lang/ProcessHandle tests fail with "RuntimeException: Input/output error" in java.lang.ProcessHandleImpl$Info.info0

Roger Riggs · Roger Riggs · commit d24b7b7026cf · 2022-08-23T14:36:46.000Z
Reviewed-by: jpai, alanb
diff --git a/src/java.base/macosx/native/libjava/ProcessHandleImpl_macosx.c b/src/java.base/macosx/native/libjava/ProcessHandleImpl_macosx.c
@@ -272,7 +272,8 @@ void os_getCmdlineAndUserInfo(JNIEnv *env, jobject jinfo, pid_t pid) {
         mib[2] = pid;
         size = (size_t) maxargs;
         if (sysctl(mib, 3, args, &size, NULL, 0) == -1) {
-            if (errno != EINVAL) {
+            if (errno != EINVAL && errno != EIO) {
+                // If the pid is invalid, the information returned is empty and no exception
                 JNU_ThrowByNameWithLastError(env,
                     "java/lang/RuntimeException", "sysctl failed");
             }
@@ -300,4 +301,3 @@ void os_getCmdlineAndUserInfo(JNIEnv *env, jobject jinfo, pid_t pid) {
     // Free the arg buffer
     free(args);
 }
-

Original file line number	Diff line number	Diff line change
`@@ -272,7 +272,8 @@ void os_getCmdlineAndUserInfo(JNIEnv *env, jobject jinfo, pid_t pid) {`
`272`	`272`	`mib[2] = pid;`
`273`	`273`	`size = (size_t) maxargs;`
`274`	`274`	`if (sysctl(mib, 3, args, &size, NULL, 0) == -1) {`
`275`		`- if (errno != EINVAL) {`
	`275`	`+ if (errno != EINVAL && errno != EIO) {`
	`276`	`+ // If the pid is invalid, the information returned is empty and no exception`
`276`	`277`	`JNU_ThrowByNameWithLastError(env,`
`277`	`278`	`"java/lang/RuntimeException", "sysctl failed");`
`278`	`279`	`}`
`@@ -300,4 +301,3 @@ void os_getCmdlineAndUserInfo(JNIEnv *env, jobject jinfo, pid_t pid) {`
`300`	`301`	`// Free the arg buffer`
`301`	`302`	`free(args);`
`302`	`303`	`}`
`303`		`-`