-
Notifications
You must be signed in to change notification settings - Fork 721
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
jdk_security4_FAILED IOException: Short read of DER length or Too short #17279
Comments
JDK11 aarch64_linux 0.38 build(
50x grinder - all failed JDK11 aarch64_mac 0.38 50x aarch64_linux_hotspot_2023-04-28 |
Both 50x aarch64_linux_hotspot_2023-04-28 and FYI @pshipton |
Similar
|
Also seen at JDK 20 x86-64_linux head stream
Not seen in JDK8 head stream, and JDK19 0.37. |
Picking one test sun/security/krb5/auto/BasicProc.java still fails when run by itself on jdk11 xlinux, head stream (32634). m2 rc1 (CE build) Changes: |
Confirmed failing (32638) with: ibmruntimes/openj9-openjdk-jdk11#641 seems the likely cause. Creating a 0.38 build without this change for testing. |
sun/security/krb5/auto/BasicProc.java is passing with ibmruntimes/openj9-openjdk-jdk11#641 reverted (32648). |
@jasonkatonica @KostasTsiounis I need to back out ibmruntimes/openj9-openjdk-jdk11#641, and all other versions, from the 0.38 release streams. It also needs to be backed out or fixed in the head stream soon. Do you have a preference? |
I think I can see what the issue is. I am reading a If that is actually the problem, the fix should be quick. I will know by tomorrow. I am, however, ok with either solution (fix or revert). |
Those tests are part of the |
Yes, you can also run jdk_security4 which is a smaller target, or run just the failing tests. Is there any reason we need to have this change in the 0.38 release and should wait for the fix? I figured (wrong) it was a safe change to include. |
No, there is nothing rushing us to get it in for this release. It's a debug message helping us figure Kerberos issues that might arise in the future. |
The change is reverted in 0.38 so I'll move this to 0.40. |
A fix for this issue was put in for Java 11, 17, 20 and Next, where the PRs for the debug message had already been merged. |
@pshipton Do you think we should close this? |
Failure link
From an internal build(
ub18-aarch64-8
):Rerun in Grinder - Change TARGET to run only the failed test targets.
Optional info
Failure output (captured from console output)
50x internal grinder - all failed
The text was updated successfully, but these errors were encountered: